linux 2.6.16.38 w/ vs2.0.3-rc1
[linux-2.6.git] / kernel / audit.c
index 963fd15..0a813d2 100644 (file)
 #include <linux/audit.h>
 
 #include <net/sock.h>
-#include <net/netlink.h>
 #include <linux/skbuff.h>
 #include <linux/netlink.h>
-#include <linux/selinux.h>
-#include <linux/inotify.h>
-
-#include "audit.h"
 
 /* No auditing will take place until audit_initialized != 0.
  * (Initialization happens after skb_init is called.) */
@@ -77,7 +72,7 @@ static int    audit_failure = AUDIT_FAIL_PRINTK;
  * contains the (non-zero) pid. */
 int            audit_pid;
 
-/* If audit_rate_limit is non-zero, limit the rate of sending audit records
+/* If audit_limit is non-zero, limit the rate of sending audit records
  * to that number per second.  This prevents DoS attacks, but results in
  * audit records being dropped. */
 static int     audit_rate_limit;
@@ -90,7 +85,6 @@ static int    audit_backlog_wait_overflow = 0;
 /* The identity of the user shutting down the audit system. */
 uid_t          audit_sig_uid = -1;
 pid_t          audit_sig_pid = -1;
-u32            audit_sig_sid = 0;
 
 /* Records can be lost in several ways:
    0) [suppressed in audit_alloc]
@@ -104,17 +98,11 @@ static atomic_t    audit_lost = ATOMIC_INIT(0);
 /* The netlink socket. */
 static struct sock *audit_sock;
 
-/* Inotify handle. */
-struct inotify_handle *audit_ih;
-
-/* Hash for inode-based rules */
-struct list_head audit_inode_hash[AUDIT_INODE_BUCKETS];
-
 /* The audit_freelist is a list of pre-allocated audit buffers (if more
  * than AUDIT_MAXFREE are in use, the audit buffer is freed instead of
  * being placed on the freelist). */
 static DEFINE_SPINLOCK(audit_freelist_lock);
-static int        audit_freelist_count;
+static int        audit_freelist_count = 0;
 static LIST_HEAD(audit_freelist);
 
 static struct sk_buff_head audit_skb_queue;
@@ -122,8 +110,10 @@ static struct task_struct *kauditd_task;
 static DECLARE_WAIT_QUEUE_HEAD(kauditd_wait);
 static DECLARE_WAIT_QUEUE_HEAD(audit_backlog_wait);
 
-/* Serialize requests from userspace. */
-static DEFINE_MUTEX(audit_cmd_mutex);
+/* The netlink socket is only to be read by 1 CPU, which lets us assume
+ * that list additions and deletions never happen simultaneously in
+ * auditsc.c */
+DECLARE_MUTEX(audit_netlink_sem);
 
 /* AUDIT_BUFSIZ is the size of the temporary buffer used for formatting
  * audit records.  Since printk uses a 1024 byte buffer, this buffer
@@ -152,7 +142,7 @@ static void audit_set_pid(struct audit_buffer *ab, pid_t pid)
        nlh->nlmsg_pid = pid;
 }
 
-void audit_panic(const char *message)
+static void audit_panic(const char *message)
 {
        switch (audit_failure)
        {
@@ -196,14 +186,8 @@ static inline int audit_rate_check(void)
        return retval;
 }
 
-/**
- * audit_log_lost - conditionally log lost audit message event
- * @message: the message stating reason for lost audit message
- *
- * Emit at least 1 message per second, even if audit_rate_check is
- * throttling.
- * Always increment the lost messages counter.
-*/
+/* Emit at least 1 message per second, even if audit_rate_check is
+ * throttling. */
 void audit_log_lost(const char *message)
 {
        static unsigned long    last_msg = 0;
@@ -234,106 +218,53 @@ void audit_log_lost(const char *message)
                       audit_backlog_limit);
                audit_panic(message);
        }
+
 }
 
-static int audit_set_rate_limit(int limit, uid_t loginuid, u32 sid)
+static int audit_set_rate_limit(int limit, uid_t loginuid)
 {
-       int old = audit_rate_limit;
-
-       if (sid) {
-               char *ctx = NULL;
-               u32 len;
-               int rc;
-               if ((rc = selinux_ctxid_to_string(sid, &ctx, &len)))
-                       return rc;
-               else
-                       audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
-                               "audit_rate_limit=%d old=%d by auid=%u subj=%s",
-                               limit, old, loginuid, ctx);
-               kfree(ctx);
-       } else
-               audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
-                       "audit_rate_limit=%d old=%d by auid=%u",
-                       limit, old, loginuid);
+       int old          = audit_rate_limit;
        audit_rate_limit = limit;
-       return 0;
+       audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE, 
+                       "audit_rate_limit=%d old=%d by auid=%u",
+                       audit_rate_limit, old, loginuid);
+       return old;
 }
 
-static int audit_set_backlog_limit(int limit, uid_t loginuid, u32 sid)
+static int audit_set_backlog_limit(int limit, uid_t loginuid)
 {
-       int old = audit_backlog_limit;
-
-       if (sid) {
-               char *ctx = NULL;
-               u32 len;
-               int rc;
-               if ((rc = selinux_ctxid_to_string(sid, &ctx, &len)))
-                       return rc;
-               else
-                       audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
-                           "audit_backlog_limit=%d old=%d by auid=%u subj=%s",
-                               limit, old, loginuid, ctx);
-               kfree(ctx);
-       } else
-               audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
-                       "audit_backlog_limit=%d old=%d by auid=%u",
-                       limit, old, loginuid);
+       int old          = audit_backlog_limit;
        audit_backlog_limit = limit;
-       return 0;
+       audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
+                       "audit_backlog_limit=%d old=%d by auid=%u",
+                       audit_backlog_limit, old, loginuid);
+       return old;
 }
 
-static int audit_set_enabled(int state, uid_t loginuid, u32 sid)
+static int audit_set_enabled(int state, uid_t loginuid)
 {
-       int old = audit_enabled;
-
+       int old          = audit_enabled;
        if (state != 0 && state != 1)
                return -EINVAL;
-
-       if (sid) {
-               char *ctx = NULL;
-               u32 len;
-               int rc;
-               if ((rc = selinux_ctxid_to_string(sid, &ctx, &len)))
-                       return rc;
-               else
-                       audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
-                               "audit_enabled=%d old=%d by auid=%u subj=%s",
-                               state, old, loginuid, ctx);
-               kfree(ctx);
-       } else
-               audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
-                       "audit_enabled=%d old=%d by auid=%u",
-                       state, old, loginuid);
        audit_enabled = state;
-       return 0;
+       audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
+                       "audit_enabled=%d old=%d by auid=%u",
+                       audit_enabled, old, loginuid);
+       return old;
 }
 
-static int audit_set_failure(int state, uid_t loginuid, u32 sid)
+static int audit_set_failure(int state, uid_t loginuid)
 {
-       int old = audit_failure;
-
+       int old          = audit_failure;
        if (state != AUDIT_FAIL_SILENT
            && state != AUDIT_FAIL_PRINTK
            && state != AUDIT_FAIL_PANIC)
                return -EINVAL;
-
-       if (sid) {
-               char *ctx = NULL;
-               u32 len;
-               int rc;
-               if ((rc = selinux_ctxid_to_string(sid, &ctx, &len)))
-                       return rc;
-               else
-                       audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
-                               "audit_failure=%d old=%d by auid=%u subj=%s",
-                               state, old, loginuid, ctx);
-               kfree(ctx);
-       } else
-               audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
-                       "audit_failure=%d old=%d by auid=%u",
-                       state, old, loginuid);
        audit_failure = state;
-       return 0;
+       audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
+                       "audit_failure=%d old=%d by auid=%u",
+                       audit_failure, old, loginuid);
+       return old;
 }
 
 static int kauditd_thread(void *dummy)
@@ -371,26 +302,8 @@ static int kauditd_thread(void *dummy)
        }
 }
 
-int audit_send_list(void *_dest)
-{
-       struct audit_netlink_list *dest = _dest;
-       int pid = dest->pid;
-       struct sk_buff *skb;
-
-       /* wait for parent to finish and send an ACK */
-       mutex_lock(&audit_cmd_mutex);
-       mutex_unlock(&audit_cmd_mutex);
-
-       while ((skb = __skb_dequeue(&dest->q)) != NULL)
-               netlink_unicast(audit_sock, skb, pid, 0);
-
-       kfree(dest);
-
-       return 0;
-}
-
-struct sk_buff *audit_make_reply(int pid, int seq, int type, int done,
-                                int multi, void *payload, int size)
+void audit_send_reply(int pid, int seq, int type, int done, int multi,
+                     void *payload, int size)
 {
        struct sk_buff  *skb;
        struct nlmsghdr *nlh;
@@ -401,71 +314,44 @@ struct sk_buff *audit_make_reply(int pid, int seq, int type, int done,
 
        skb = alloc_skb(len, GFP_KERNEL);
        if (!skb)
-               return NULL;
+               return;
 
        nlh              = NLMSG_PUT(skb, pid, seq, t, size);
        nlh->nlmsg_flags = flags;
        data             = NLMSG_DATA(nlh);
        memcpy(data, payload, size);
-       return skb;
-
-nlmsg_failure:                 /* Used by NLMSG_PUT */
-       if (skb)
-               kfree_skb(skb);
-       return NULL;
-}
 
-/**
- * audit_send_reply - send an audit reply message via netlink
- * @pid: process id to send reply to
- * @seq: sequence number
- * @type: audit message type
- * @done: done (last) flag
- * @multi: multi-part message flag
- * @payload: payload data
- * @size: payload size
- *
- * Allocates an skb, builds the netlink message, and sends it to the pid.
- * No failure notifications.
- */
-void audit_send_reply(int pid, int seq, int type, int done, int multi,
-                     void *payload, int size)
-{
-       struct sk_buff  *skb;
-       skb = audit_make_reply(pid, seq, type, done, multi, payload, size);
-       if (!skb)
-               return;
        /* Ignore failure. It'll only happen if the sender goes away,
           because our timeout is set to infinite. */
        netlink_unicast(audit_sock, skb, pid, 0);
        return;
+
+nlmsg_failure:                 /* Used by NLMSG_PUT */
+       if (skb)
+               kfree_skb(skb);
 }
 
 /*
  * Check for appropriate CAP_AUDIT_ capabilities on incoming audit
  * control messages.
  */
-static int audit_netlink_ok(struct sk_buff *skb, u16 msg_type)
+static int audit_netlink_ok(kernel_cap_t eff_cap, u16 msg_type)
 {
        int err = 0;
 
        switch (msg_type) {
        case AUDIT_GET:
        case AUDIT_LIST:
-       case AUDIT_LIST_RULES:
        case AUDIT_SET:
        case AUDIT_ADD:
-       case AUDIT_ADD_RULE:
        case AUDIT_DEL:
-       case AUDIT_DEL_RULE:
        case AUDIT_SIGNAL_INFO:
-               if (security_netlink_recv(skb, CAP_AUDIT_CONTROL))
+               if (!cap_raised(eff_cap, CAP_AUDIT_CONTROL))
                        err = -EPERM;
                break;
        case AUDIT_USER:
        case AUDIT_FIRST_USER_MSG...AUDIT_LAST_USER_MSG:
-       case AUDIT_FIRST_USER_MSG2...AUDIT_LAST_USER_MSG2:
-               if (security_netlink_recv(skb, CAP_AUDIT_WRITE))
+               if (!cap_raised(eff_cap, CAP_AUDIT_WRITE))
                        err = -EPERM;
                break;
        default:  /* bad msg */
@@ -477,23 +363,20 @@ static int audit_netlink_ok(struct sk_buff *skb, u16 msg_type)
 
 static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh)
 {
-       u32                     uid, pid, seq, sid;
+       u32                     uid, pid, seq;
        void                    *data;
        struct audit_status     *status_get, status_set;
        int                     err;
        struct audit_buffer     *ab;
        u16                     msg_type = nlh->nlmsg_type;
        uid_t                   loginuid; /* loginuid of sender */
-       struct audit_sig_info   *sig_data;
-       char                    *ctx;
-       u32                     len;
+       struct audit_sig_info   sig_data;
 
-       err = audit_netlink_ok(skb, msg_type);
+       err = audit_netlink_ok(NETLINK_CB(skb).eff_cap, msg_type);
        if (err)
                return err;
 
-       /* As soon as there's any sign of userspace auditd,
-        * start kauditd to talk to it */
+       /* As soon as there's any sign of userspace auditd, start kauditd to talk to it */
        if (!kauditd_task)
                kauditd_task = kthread_run(kauditd_thread, NULL, "kauditd");
        if (IS_ERR(kauditd_task)) {
@@ -505,7 +388,6 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh)
        pid  = NETLINK_CREDS(skb)->pid;
        uid  = NETLINK_CREDS(skb)->uid;
        loginuid = NETLINK_CB(skb).loginuid;
-       sid  = NETLINK_CB(skb).sid;
        seq  = nlh->nlmsg_seq;
        data = NLMSG_DATA(nlh);
 
@@ -526,44 +408,28 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh)
                        return -EINVAL;
                status_get   = (struct audit_status *)data;
                if (status_get->mask & AUDIT_STATUS_ENABLED) {
-                       err = audit_set_enabled(status_get->enabled,
-                                                       loginuid, sid);
+                       err = audit_set_enabled(status_get->enabled, loginuid);
                        if (err < 0) return err;
                }
                if (status_get->mask & AUDIT_STATUS_FAILURE) {
-                       err = audit_set_failure(status_get->failure,
-                                                        loginuid, sid);
+                       err = audit_set_failure(status_get->failure, loginuid);
                        if (err < 0) return err;
                }
                if (status_get->mask & AUDIT_STATUS_PID) {
                        int old   = audit_pid;
-                       if (sid) {
-                               if ((err = selinux_ctxid_to_string(
-                                               sid, &ctx, &len)))
-                                       return err;
-                               else
-                                       audit_log(NULL, GFP_KERNEL,
-                                               AUDIT_CONFIG_CHANGE,
-                                               "audit_pid=%d old=%d by auid=%u subj=%s",
-                                               status_get->pid, old,
-                                               loginuid, ctx);
-                               kfree(ctx);
-                       } else
-                               audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
-                                       "audit_pid=%d old=%d by auid=%u",
-                                         status_get->pid, old, loginuid);
                        audit_pid = status_get->pid;
+                       audit_log(NULL, GFP_KERNEL, AUDIT_CONFIG_CHANGE,
+                               "audit_pid=%d old=%d by auid=%u",
+                                 audit_pid, old, loginuid);
                }
                if (status_get->mask & AUDIT_STATUS_RATE_LIMIT)
-                       err = audit_set_rate_limit(status_get->rate_limit,
-                                                        loginuid, sid);
+                       audit_set_rate_limit(status_get->rate_limit, loginuid);
                if (status_get->mask & AUDIT_STATUS_BACKLOG_LIMIT)
-                       err = audit_set_backlog_limit(status_get->backlog_limit,
-                                                       loginuid, sid);
+                       audit_set_backlog_limit(status_get->backlog_limit,
+                                                       loginuid);
                break;
        case AUDIT_USER:
        case AUDIT_FIRST_USER_MSG...AUDIT_LAST_USER_MSG:
-       case AUDIT_FIRST_USER_MSG2...AUDIT_LAST_USER_MSG2:
                if (!audit_enabled && msg_type != AUDIT_USER_AVC)
                        return 0;
 
@@ -573,21 +439,8 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh)
                        ab = audit_log_start(NULL, GFP_KERNEL, msg_type);
                        if (ab) {
                                audit_log_format(ab,
-                                                "user pid=%d uid=%u auid=%u",
-                                                pid, uid, loginuid);
-                               if (sid) {
-                                       if (selinux_ctxid_to_string(
-                                                       sid, &ctx, &len)) {
-                                               audit_log_format(ab, 
-                                                       " ssid=%u", sid);
-                                               /* Maybe call audit_panic? */
-                                       } else
-                                               audit_log_format(ab, 
-                                                       " subj=%s", ctx);
-                                       kfree(ctx);
-                               }
-                               audit_log_format(ab, " msg='%.1024s'",
-                                        (char *)data);
+                                                "user pid=%d uid=%u auid=%u msg='%.1024s'",
+                                                pid, uid, loginuid, (char *)data);
                                audit_set_pid(ab, pid);
                                audit_log_end(ab);
                        }
@@ -595,40 +448,18 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh)
                break;
        case AUDIT_ADD:
        case AUDIT_DEL:
-               if (nlmsg_len(nlh) < sizeof(struct audit_rule))
+               if (nlh->nlmsg_len < sizeof(struct audit_rule))
                        return -EINVAL;
                /* fallthrough */
        case AUDIT_LIST:
                err = audit_receive_filter(nlh->nlmsg_type, NETLINK_CB(skb).pid,
-                                          uid, seq, data, nlmsg_len(nlh),
-                                          loginuid, sid);
-               break;
-       case AUDIT_ADD_RULE:
-       case AUDIT_DEL_RULE:
-               if (nlmsg_len(nlh) < sizeof(struct audit_rule_data))
-                       return -EINVAL;
-               /* fallthrough */
-       case AUDIT_LIST_RULES:
-               err = audit_receive_filter(nlh->nlmsg_type, NETLINK_CB(skb).pid,
-                                          uid, seq, data, nlmsg_len(nlh),
-                                          loginuid, sid);
+                                          uid, seq, data, loginuid);
                break;
        case AUDIT_SIGNAL_INFO:
-               err = selinux_ctxid_to_string(audit_sig_sid, &ctx, &len);
-               if (err)
-                       return err;
-               sig_data = kmalloc(sizeof(*sig_data) + len, GFP_KERNEL);
-               if (!sig_data) {
-                       kfree(ctx);
-                       return -ENOMEM;
-               }
-               sig_data->uid = audit_sig_uid;
-               sig_data->pid = audit_sig_pid;
-               memcpy(sig_data->ctx, ctx, len);
-               kfree(ctx);
+               sig_data.uid = audit_sig_uid;
+               sig_data.pid = audit_sig_pid;
                audit_send_reply(NETLINK_CB(skb).pid, seq, AUDIT_SIGNAL_INFO, 
-                               0, 0, sig_data, sizeof(*sig_data) + len);
-               kfree(sig_data);
+                               0, 0, &sig_data, sizeof(sig_data));
                break;
        default:
                err = -EINVAL;
@@ -638,11 +469,9 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh)
        return err < 0 ? err : 0;
 }
 
-/*
- * Get message from skb (based on rtnetlink_rcv_skb).  Each message is
+/* Get message from skb (based on rtnetlink_rcv_skb).  Each message is
  * processed by audit_receive_msg.  Malformed skbs with wrong length are
- * discarded silently.
- */
+ * discarded silently.  */
 static void audit_receive_skb(struct sk_buff *skb)
 {
        int             err;
@@ -670,56 +499,32 @@ static void audit_receive(struct sock *sk, int length)
        struct sk_buff  *skb;
        unsigned int qlen;
 
-       mutex_lock(&audit_cmd_mutex);
+       down(&audit_netlink_sem);
 
        for (qlen = skb_queue_len(&sk->sk_receive_queue); qlen; qlen--) {
                skb = skb_dequeue(&sk->sk_receive_queue);
                audit_receive_skb(skb);
                kfree_skb(skb);
        }
-       mutex_unlock(&audit_cmd_mutex);
+       up(&audit_netlink_sem);
 }
 
-#ifdef CONFIG_AUDITSYSCALL
-static const struct inotify_operations audit_inotify_ops = {
-       .handle_event   = audit_handle_ievent,
-       .destroy_watch  = audit_free_parent,
-};
-#endif
 
 /* Initialize audit support at boot time. */
 static int __init audit_init(void)
 {
-       int i;
-
        printk(KERN_INFO "audit: initializing netlink socket (%s)\n",
               audit_default ? "enabled" : "disabled");
        audit_sock = netlink_kernel_create(NETLINK_AUDIT, 0, audit_receive,
                                           THIS_MODULE);
        if (!audit_sock)
                audit_panic("cannot initialize netlink socket");
-       else
-               audit_sock->sk_sndtimeo = MAX_SCHEDULE_TIMEOUT;
 
+       audit_sock->sk_sndtimeo = MAX_SCHEDULE_TIMEOUT;
        skb_queue_head_init(&audit_skb_queue);
        audit_initialized = 1;
        audit_enabled = audit_default;
-
-       /* Register the callback with selinux.  This callback will be invoked
-        * when a new policy is loaded. */
-       selinux_audit_set_callback(&selinux_audit_rule_update);
-
        audit_log(NULL, GFP_KERNEL, AUDIT_KERNEL, "initialized");
-
-#ifdef CONFIG_AUDITSYSCALL
-       audit_ih = inotify_init(&audit_inotify_ops);
-       if (IS_ERR(audit_ih))
-               audit_panic("cannot initialize inotify handle");
-#endif
-
-       for (i = 0; i < AUDIT_INODE_BUCKETS; i++)
-               INIT_LIST_HEAD(&audit_inode_hash[i]);
-
        return 0;
 }
 __initcall(audit_init);
@@ -733,7 +538,7 @@ static int __init audit_enable(char *str)
               audit_initialized ? "" : " (after initialization)");
        if (audit_initialized)
                audit_enabled = audit_default;
-       return 1;
+       return 0;
 }
 
 __setup("audit=", audit_enable);
@@ -749,12 +554,10 @@ static void audit_buffer_free(struct audit_buffer *ab)
                kfree_skb(ab->skb);
 
        spin_lock_irqsave(&audit_freelist_lock, flags);
-       if (audit_freelist_count > AUDIT_MAXFREE)
+       if (++audit_freelist_count > AUDIT_MAXFREE)
                kfree(ab);
-       else {
-               audit_freelist_count++;
+       else
                list_add(&ab->list, &audit_freelist);
-       }
        spin_unlock_irqrestore(&audit_freelist_lock, flags);
 }
 
@@ -797,10 +600,7 @@ err:
        return NULL;
 }
 
-/**
- * audit_serial - compute a serial number for the audit record
- *
- * Compute a serial number for the audit record.  Audit records are
+/* Compute a serial number for the audit record.  Audit records are
  * written to user-space as soon as they are generated, so a complete
  * audit record may be written in several pieces.  The timestamp of the
  * record and this serial number are used by the user-space tools to
@@ -812,11 +612,11 @@ err:
  * audit context (for those records that have a context), and emit them
  * all at syscall exit.  However, this could delay the reporting of
  * significant errors until syscall exit (or never, if the system
- * halts).
- */
+ * halts). */
+
 unsigned int audit_serial(void)
 {
-       static DEFINE_SPINLOCK(serial_lock);
+       static spinlock_t serial_lock = SPIN_LOCK_UNLOCKED;
        static unsigned int serial = 0;
 
        unsigned long flags;
@@ -849,21 +649,6 @@ static inline void audit_get_stamp(struct audit_context *ctx,
  * will be written at syscall exit.  If there is no associated task, tsk
  * should be NULL. */
 
-/**
- * audit_log_start - obtain an audit buffer
- * @ctx: audit_context (may be NULL)
- * @gfp_mask: type of allocation
- * @type: audit message type
- *
- * Returns audit_buffer pointer on success or NULL on error.
- *
- * Obtain an audit buffer.  This routine does locking to obtain the
- * audit buffer, but then no locking is required for calls to
- * audit_log_*format.  If the task (ctx) is a task that is currently in a
- * syscall, then the syscall is marked as auditable and an audit record
- * will be written at syscall exit.  If there is no associated task, then
- * task context (ctx) should be NULL.
- */
 struct audit_buffer *audit_log_start(struct audit_context *ctx, gfp_t gfp_mask,
                                     int type)
 {
@@ -876,9 +661,6 @@ struct audit_buffer *audit_log_start(struct audit_context *ctx, gfp_t gfp_mask,
        if (!audit_initialized)
                return NULL;
 
-       if (unlikely(audit_filter_type(type)))
-               return NULL;
-
        if (gfp_mask & __GFP_WAIT)
                reserve = 0;
        else
@@ -931,7 +713,6 @@ struct audit_buffer *audit_log_start(struct audit_context *ctx, gfp_t gfp_mask,
 /**
  * audit_expand - expand skb in the audit buffer
  * @ab: audit_buffer
- * @extra: space to add at tail of the skb
  *
  * Returns 0 (no space) on failed expansion, or available space if
  * successful.
@@ -948,12 +729,10 @@ static inline int audit_expand(struct audit_buffer *ab, int extra)
        return skb_tailroom(skb);
 }
 
-/*
- * Format an audit message into the audit buffer.  If there isn't enough
+/* Format an audit message into the audit buffer.  If there isn't enough
  * room in the audit buffer, more room will be allocated and vsnprint
  * will be called a second time.  Currently, we assume that a printk
- * can't format message larger than 1024 bytes, so we don't either.
- */
+ * can't format message larger than 1024 bytes, so we don't either. */
 static void audit_log_vformat(struct audit_buffer *ab, const char *fmt,
                              va_list args)
 {
@@ -978,8 +757,7 @@ static void audit_log_vformat(struct audit_buffer *ab, const char *fmt,
                /* The printk buffer is 1024 bytes long, so if we get
                 * here and AUDIT_BUFSIZ is at least 1024, then we can
                 * log everything that printk could have logged. */
-               avail = audit_expand(ab,
-                       max_t(unsigned, AUDIT_BUFSIZ, 1+len-avail));
+               avail = audit_expand(ab, max_t(unsigned, AUDIT_BUFSIZ, 1+len-avail));
                if (!avail)
                        goto out;
                len = vsnprintf(skb->tail, avail, fmt, args2);
@@ -990,14 +768,8 @@ out:
        return;
 }
 
-/**
- * audit_log_format - format a message into the audit buffer.
- * @ab: audit_buffer
- * @fmt: format string
- * @...: optional parameters matching @fmt string
- *
- * All the work is done in audit_log_vformat.
- */
+/* Format a message into the audit buffer.  All the work is done in
+ * audit_log_vformat. */
 void audit_log_format(struct audit_buffer *ab, const char *fmt, ...)
 {
        va_list args;
@@ -1009,18 +781,9 @@ void audit_log_format(struct audit_buffer *ab, const char *fmt, ...)
        va_end(args);
 }
 
-/**
- * audit_log_hex - convert a buffer to hex and append it to the audit skb
- * @ab: the audit_buffer
- * @buf: buffer to convert to hex
- * @len: length of @buf to be converted
- *
- * No return value; failure to expand is silently ignored.
- *
- * This function will take the passed buf and convert it into a string of
- * ascii hex digits. The new string is placed onto the skb.
- */
-void audit_log_hex(struct audit_buffer *ab, const unsigned char *buf,
+/* This function will take the passed buf and convert it into a string of
+ * ascii hex digits. The new string is placed onto the skb. */
+void audit_log_hex(struct audit_buffer *ab, const unsigned char *buf, 
                size_t len)
 {
        int i, avail, new_len;
@@ -1028,9 +791,6 @@ void audit_log_hex(struct audit_buffer *ab, const unsigned char *buf,
        struct sk_buff *skb;
        static const unsigned char *hex = "0123456789ABCDEF";
 
-       if (!ab)
-               return;
-
        BUG_ON(!ab->skb);
        skb = ab->skb;
        avail = skb_tailroom(skb);
@@ -1052,79 +812,22 @@ void audit_log_hex(struct audit_buffer *ab, const unsigned char *buf,
        skb_put(skb, len << 1); /* new string is twice the old string */
 }
 
-/*
- * Format a string of no more than slen characters into the audit buffer,
- * enclosed in quote marks.
- */
-static void audit_log_n_string(struct audit_buffer *ab, size_t slen,
-                              const char *string)
-{
-       int avail, new_len;
-       unsigned char *ptr;
-       struct sk_buff *skb;
-
-       if (!ab)
-               return;
-
-       BUG_ON(!ab->skb);
-       skb = ab->skb;
-       avail = skb_tailroom(skb);
-       new_len = slen + 3;     /* enclosing quotes + null terminator */
-       if (new_len > avail) {
-               avail = audit_expand(ab, new_len);
-               if (!avail)
-                       return;
-       }
-       ptr = skb->tail;
-       *ptr++ = '"';
-       memcpy(ptr, string, slen);
-       ptr += slen;
-       *ptr++ = '"';
-       *ptr = 0;
-       skb_put(skb, slen + 2); /* don't include null terminator */
-}
-
-/**
- * audit_log_n_unstrustedstring - log a string that may contain random characters
- * @ab: audit_buffer
- * @len: lenth of string (not including trailing null)
- * @string: string to be logged
- *
- * This code will escape a string that is passed to it if the string
- * contains a control character, unprintable character, double quote mark,
+/* This code will escape a string that is passed to it if the string
+ * contains a control character, unprintable character, double quote mark, 
  * or a space. Unescaped strings will start and end with a double quote mark.
- * Strings that are escaped are printed in hex (2 digits per char).
- *
- * The caller specifies the number of characters in the string to log, which may
- * or may not be the entire string.
- */
-const char *audit_log_n_untrustedstring(struct audit_buffer *ab, size_t len,
-                                       const char *string)
+ * Strings that are escaped are printed in hex (2 digits per char). */
+void audit_log_untrustedstring(struct audit_buffer *ab, const char *string)
 {
        const unsigned char *p = string;
 
        while (*p) {
                if (*p == '"' || *p < 0x21 || *p > 0x7f) {
-                       audit_log_hex(ab, string, len);
-                       return string + len + 1;
+                       audit_log_hex(ab, string, strlen(string));
+                       return;
                }
                p++;
        }
-       audit_log_n_string(ab, len, string);
-       return p + 1;
-}
-
-/**
- * audit_log_unstrustedstring - log a string that may contain random characters
- * @ab: audit_buffer
- * @string: string to be logged
- *
- * Same as audit_log_n_unstrustedstring(), except that strlen is used to
- * determine string length.
- */
-const char *audit_log_untrustedstring(struct audit_buffer *ab, const char *string)
-{
-       return audit_log_n_untrustedstring(ab, strlen(string), string);
+       audit_log_format(ab, "\"%s\"", string);
 }
 
 /* This is a helper-function to print the escaped d_path */
@@ -1151,15 +854,10 @@ void audit_log_d_path(struct audit_buffer *ab, const char *prefix,
        kfree(path);
 }
 
-/**
- * audit_log_end - end one audit record
- * @ab: the audit_buffer
- *
- * The netlink_* functions cannot be called inside an irq context, so
- * the audit buffer is placed on a queue and a tasklet is scheduled to
+/* The netlink_* functions cannot be called inside an irq context, so
+ * the audit buffer is places on a queue and a tasklet is scheduled to
  * remove them from the queue outside the irq context.  May be called in
- * any context.
- */
+ * any context. */
 void audit_log_end(struct audit_buffer *ab)
 {
        if (!ab)
@@ -1180,18 +878,9 @@ void audit_log_end(struct audit_buffer *ab)
        audit_buffer_free(ab);
 }
 
-/**
- * audit_log - Log an audit record
- * @ctx: audit context
- * @gfp_mask: type of allocation
- * @type: audit message type
- * @fmt: format string to use
- * @...: variable parameters matching the format string
- *
- * This is a convenience function that calls audit_log_start,
- * audit_log_vformat, and audit_log_end.  It may be called
- * in any context.
- */
+/* Log an audit record.  This is a convenience function that calls
+ * audit_log_start, audit_log_vformat, and audit_log_end.  It may be
+ * called in any context. */
 void audit_log(struct audit_context *ctx, gfp_t gfp_mask, int type, 
               const char *fmt, ...)
 {
@@ -1206,8 +895,3 @@ void audit_log(struct audit_context *ctx, gfp_t gfp_mask, int type,
                audit_log_end(ab);
        }
 }
-
-EXPORT_SYMBOL(audit_log_start);
-EXPORT_SYMBOL(audit_log_end);
-EXPORT_SYMBOL(audit_log_format);
-EXPORT_SYMBOL(audit_log);