7 files changed, 148 insertions, 126 deletions

diff --git a/security/dummy.c b/security/dummy.c
index 649326bf64ea..78d8f92310a4 100644
--- a/security/dummy.c
+++ b/security/dummy.c
@@ -181,8 +181,7 @@ static void dummy_sb_free_security (struct super_block *sb)
         return;
 }
 
-static int dummy_sb_copy_data (struct file_system_type *type,
-                               void *orig, void *copy)
+static int dummy_sb_copy_data (char *orig, char *copy)
 {
         return 0;
 }
@@ -245,19 +244,17 @@ static void dummy_sb_post_pivotroot (struct nameidata *old_nd, struct nameidata
         return;
 }
 
-static int dummy_sb_get_mnt_opts(const struct super_block *sb, char ***mount_options,
-                                 int **flags, int *num_opts)
+static int dummy_sb_get_mnt_opts(const struct super_block *sb,
+                                 struct security_mnt_opts *opts)
 {
-        *mount_options = NULL;
-        *flags = NULL;
-        *num_opts = 0;
+        security_init_mnt_opts(opts);
         return 0;
 }
 
-static int dummy_sb_set_mnt_opts(struct super_block *sb, char **mount_options,
-                                 int *flags, int num_opts)
+static int dummy_sb_set_mnt_opts(struct super_block *sb,
+                                 struct security_mnt_opts *opts)
 {
-        if (unlikely(num_opts))
+        if (unlikely(opts->num_mnt_opts))
                 return -EOPNOTSUPP;
         return 0;
 }
@@ -268,6 +265,11 @@ static void dummy_sb_clone_mnt_opts(const struct super_block *oldsb,
         return;
 }
 
+static int dummy_sb_parse_opts_str(char *options, struct security_mnt_opts *opts)
+{
+        return 0;
+}
+
 static int dummy_inode_alloc_security (struct inode *inode)
 {
         return 0;
@@ -1028,6 +1030,7 @@ void security_fixup_ops (struct security_operations *ops)
         set_to_dummy_if_null(ops, sb_get_mnt_opts);
         set_to_dummy_if_null(ops, sb_set_mnt_opts);
         set_to_dummy_if_null(ops, sb_clone_mnt_opts);
+        set_to_dummy_if_null(ops, sb_parse_opts_str);
         set_to_dummy_if_null(ops, inode_alloc_security);
         set_to_dummy_if_null(ops, inode_free_security);
         set_to_dummy_if_null(ops, inode_init_security);
diff --git a/security/security.c b/security/security.c
index d15e56cbaade..b1387a6b416d 100644
--- a/security/security.c
+++ b/security/security.c
@@ -244,10 +244,11 @@ void security_sb_free(struct super_block *sb)
         security_ops->sb_free_security(sb);
 }
 
-int security_sb_copy_data(struct file_system_type *type, void *orig, void *copy)
+int security_sb_copy_data(char *orig, char *copy)
 {
-        return security_ops->sb_copy_data(type, orig, copy);
+        return security_ops->sb_copy_data(orig, copy);
 }
+EXPORT_SYMBOL(security_sb_copy_data);
 
 int security_sb_kern_mount(struct super_block *sb, void *data)
 {
@@ -306,24 +307,30 @@ void security_sb_post_pivotroot(struct nameidata *old_nd, struct nameidata *new_
 }
 
 int security_sb_get_mnt_opts(const struct super_block *sb,
-                              char ***mount_options,
-                              int **flags, int *num_opts)
+                                struct security_mnt_opts *opts)
 {
-        return security_ops->sb_get_mnt_opts(sb, mount_options, flags, num_opts);
+        return security_ops->sb_get_mnt_opts(sb, opts);
 }
 
 int security_sb_set_mnt_opts(struct super_block *sb,
-                              char **mount_options,
-                              int *flags, int num_opts)
+                                struct security_mnt_opts *opts)
 {
-        return security_ops->sb_set_mnt_opts(sb, mount_options, flags, num_opts);
+        return security_ops->sb_set_mnt_opts(sb, opts);
 }
+EXPORT_SYMBOL(security_sb_set_mnt_opts);
 
 void security_sb_clone_mnt_opts(const struct super_block *oldsb,
                                 struct super_block *newsb)
 {
         security_ops->sb_clone_mnt_opts(oldsb, newsb);
 }
+EXPORT_SYMBOL(security_sb_clone_mnt_opts);
+
+int security_sb_parse_opts_str(char *options, struct security_mnt_opts *opts)
+{
+        return security_ops->sb_parse_opts_str(options, opts);
+}
+EXPORT_SYMBOL(security_sb_parse_opts_str);
 
 int security_inode_alloc(struct inode *inode)
 {
diff --git a/security/selinux/hooks.c b/security/selinux/hooks.c
index 75c2e99bfb81..4bf4807f2d44 100644
--- a/security/selinux/hooks.c
+++ b/security/selinux/hooks.c
@@ -443,8 +443,7 @@ out:
  * mount options, or whatever.
  */
 static int selinux_get_mnt_opts(const struct super_block *sb,
-                                char ***mount_options, int **mnt_opts_flags,
-                                int *num_opts)
+                                struct security_mnt_opts *opts)
 {
         int rc = 0, i;
         struct superblock_security_struct *sbsec = sb->s_security;
@@ -452,9 +451,7 @@ static int selinux_get_mnt_opts(const struct super_block *sb,
         u32 len;
         char tmp;
 
-        *num_opts = 0;
-        *mount_options = NULL;
-        *mnt_opts_flags = NULL;
+        security_init_mnt_opts(opts);
 
         if (!sbsec->initialized)
                 return -EINVAL;
@@ -470,18 +467,18 @@ static int selinux_get_mnt_opts(const struct super_block *sb,
         /* count the number of mount options for this sb */
         for (i = 0; i < 8; i++) {
                 if (tmp & 0x01)
-                        (*num_opts)++;
+                        opts->num_mnt_opts++;
                 tmp >>= 1;
         }
 
-        *mount_options = kcalloc(*num_opts, sizeof(char *), GFP_ATOMIC);
-        if (!*mount_options) {
+        opts->mnt_opts = kcalloc(opts->num_mnt_opts, sizeof(char *), GFP_ATOMIC);
+        if (!opts->mnt_opts) {
                 rc = -ENOMEM;
                 goto out_free;
         }
 
-        *mnt_opts_flags = kcalloc(*num_opts, sizeof(int), GFP_ATOMIC);
-        if (!*mnt_opts_flags) {
+        opts->mnt_opts_flags = kcalloc(opts->num_mnt_opts, sizeof(int), GFP_ATOMIC);
+        if (!opts->mnt_opts_flags) {
                 rc = -ENOMEM;
                 goto out_free;
         }
@@ -491,22 +488,22 @@ static int selinux_get_mnt_opts(const struct super_block *sb,
                 rc = security_sid_to_context(sbsec->sid, &context, &len);
                 if (rc)
                         goto out_free;
-                (*mount_options)[i] = context;
-                (*mnt_opts_flags)[i++] = FSCONTEXT_MNT;
+                opts->mnt_opts[i] = context;
+                opts->mnt_opts_flags[i++] = FSCONTEXT_MNT;
         }
         if (sbsec->flags & CONTEXT_MNT) {
                 rc = security_sid_to_context(sbsec->mntpoint_sid, &context, &len);
                 if (rc)
                         goto out_free;
-                (*mount_options)[i] = context;
-                (*mnt_opts_flags)[i++] = CONTEXT_MNT;
+                opts->mnt_opts[i] = context;
+                opts->mnt_opts_flags[i++] = CONTEXT_MNT;
         }
         if (sbsec->flags & DEFCONTEXT_MNT) {
                 rc = security_sid_to_context(sbsec->def_sid, &context, &len);
                 if (rc)
                         goto out_free;
-                (*mount_options)[i] = context;
-                (*mnt_opts_flags)[i++] = DEFCONTEXT_MNT;
+                opts->mnt_opts[i] = context;
+                opts->mnt_opts_flags[i++] = DEFCONTEXT_MNT;
         }
         if (sbsec->flags & ROOTCONTEXT_MNT) {
                 struct inode *root = sbsec->sb->s_root->d_inode;
@@ -515,24 +512,16 @@ static int selinux_get_mnt_opts(const struct super_block *sb,
                 rc = security_sid_to_context(isec->sid, &context, &len);
                 if (rc)
                         goto out_free;
-                (*mount_options)[i] = context;
-                (*mnt_opts_flags)[i++] = ROOTCONTEXT_MNT;
+                opts->mnt_opts[i] = context;
+                opts->mnt_opts_flags[i++] = ROOTCONTEXT_MNT;
         }
 
-        BUG_ON(i != *num_opts);
+        BUG_ON(i != opts->num_mnt_opts);
 
         return 0;
 
 out_free:
-        /* don't leak context string if security_sid_to_context had an error */
-        if (*mount_options && i)
-                for (; i > 0; i--)
-                        kfree((*mount_options)[i-1]);
-        kfree(*mount_options);
-        *mount_options = NULL;
-        kfree(*mnt_opts_flags);
-        *mnt_opts_flags = NULL;
-        *num_opts = 0;
+        security_free_mnt_opts(opts);
         return rc;
 }
 
@@ -553,12 +542,13 @@ static int bad_option(struct superblock_security_struct *sbsec, char flag,
                         return 1;
         return 0;
 }
+
 /*
  * Allow filesystems with binary mount data to explicitly set mount point
  * labeling information.
  */
-static int selinux_set_mnt_opts(struct super_block *sb, char **mount_options,
-                                int *flags, int num_opts)
+static int selinux_set_mnt_opts(struct super_block *sb,
+                                struct security_mnt_opts *opts)
 {
         int rc = 0, i;
         struct task_security_struct *tsec = current->security;
@@ -568,6 +558,9 @@ static int selinux_set_mnt_opts(struct super_block *sb, char **mount_options,
         struct inode_security_struct *root_isec = inode->i_security;
         u32 fscontext_sid = 0, context_sid = 0, rootcontext_sid = 0;
         u32 defcontext_sid = 0;
+        char **mount_options = opts->mnt_opts;
+        int *flags = opts->mnt_opts_flags;
+        int num_opts = opts->num_mnt_opts;
 
         mutex_lock(&sbsec->lock);
 
@@ -589,6 +582,21 @@ static int selinux_set_mnt_opts(struct super_block *sb, char **mount_options,
         }
 
         /*
+         * Binary mount data FS will come through this function twice.  Once
+         * from an explicit call and once from the generic calls from the vfs.
+         * Since the generic VFS calls will not contain any security mount data
+         * we need to skip the double mount verification.
+         *
+         * This does open a hole in which we will not notice if the first
+         * mount using this sb set explict options and a second mount using
+         * this sb does not set any security options.  (The first options
+         * will be used for both mounts)
+         */
+        if (sbsec->initialized && (sb->s_type->fs_flags & FS_BINARY_MOUNTDATA)
+            && (num_opts == 0))
+                goto out;
+
+        /*
          * parse the mount options, check if they are valid sids.
          * also check if someone is trying to mount the same sb more
          * than once with different security options.
@@ -792,43 +800,14 @@ static void selinux_sb_clone_mnt_opts(const struct super_block *oldsb,
         mutex_unlock(&newsbsec->lock);
 }
 
-/*
- * string mount options parsing and call set the sbsec
- */
-static int superblock_doinit(struct super_block *sb, void *data)
+int selinux_parse_opts_str(char *options, struct security_mnt_opts *opts)
 {
+        char *p;
         char *context = NULL, *defcontext = NULL;
         char *fscontext = NULL, *rootcontext = NULL;
-        int rc = 0;
-        char *p, *options = data;
-        /* selinux only know about a fixed number of mount options */
-        char *mnt_opts[NUM_SEL_MNT_OPTS];
-        int mnt_opts_flags[NUM_SEL_MNT_OPTS], num_mnt_opts = 0;
-
-        if (!data)
-                goto out;
+        int rc, num_mnt_opts = 0;
 
-        /* with the nfs patch this will become a goto out; */
-        if (sb->s_type->fs_flags & FS_BINARY_MOUNTDATA) {
-                const char *name = sb->s_type->name;
-                /* NFS we understand. */
-                if (!strcmp(name, "nfs")) {
-                        struct nfs_mount_data *d = data;
-
-                        if (d->version !=  NFS_MOUNT_VERSION)
-                                goto out;
-
-                        if (d->context[0]) {
-                                context = kstrdup(d->context, GFP_KERNEL);
-                                if (!context) {
-                                        rc = -ENOMEM;
-                                        goto out;
-                                }
-                        }
-                        goto build_flags;
-                } else
-                        goto out;
-        }
+        opts->num_mnt_opts = 0;
 
         /* Standard string-based options. */
         while ((p = strsep(&options, "|")) != NULL) {
@@ -901,26 +880,37 @@ static int superblock_doinit(struct super_block *sb, void *data)
                 }
         }
 
-build_flags:
+        rc = -ENOMEM;
+        opts->mnt_opts = kcalloc(NUM_SEL_MNT_OPTS, sizeof(char *), GFP_ATOMIC);
+        if (!opts->mnt_opts)
+                goto out_err;
+
+        opts->mnt_opts_flags = kcalloc(NUM_SEL_MNT_OPTS, sizeof(int), GFP_ATOMIC);
+        if (!opts->mnt_opts_flags) {
+                kfree(opts->mnt_opts);
+                goto out_err;
+        }
+
         if (fscontext) {
-                mnt_opts[num_mnt_opts] = fscontext;
-                mnt_opts_flags[num_mnt_opts++] = FSCONTEXT_MNT;
+                opts->mnt_opts[num_mnt_opts] = fscontext;
+                opts->mnt_opts_flags[num_mnt_opts++] = FSCONTEXT_MNT;
         }
         if (context) {
-                mnt_opts[num_mnt_opts] = context;
-                mnt_opts_flags[num_mnt_opts++] = CONTEXT_MNT;
+                opts->mnt_opts[num_mnt_opts] = context;
+                opts->mnt_opts_flags[num_mnt_opts++] = CONTEXT_MNT;
         }
         if (rootcontext) {
-                mnt_opts[num_mnt_opts] = rootcontext;
-                mnt_opts_flags[num_mnt_opts++] = ROOTCONTEXT_MNT;
+                opts->mnt_opts[num_mnt_opts] = rootcontext;
+                opts->mnt_opts_flags[num_mnt_opts++] = ROOTCONTEXT_MNT;
         }
         if (defcontext) {
-                mnt_opts[num_mnt_opts] = defcontext;
-                mnt_opts_flags[num_mnt_opts++] = DEFCONTEXT_MNT;
+                opts->mnt_opts[num_mnt_opts] = defcontext;
+                opts->mnt_opts_flags[num_mnt_opts++] = DEFCONTEXT_MNT;
         }
 
-out:
-        rc = selinux_set_mnt_opts(sb, mnt_opts, mnt_opts_flags, num_mnt_opts);
+        opts->num_mnt_opts = num_mnt_opts;
+        return 0;
+
 out_err:
         kfree(context);
         kfree(defcontext);
@@ -928,6 +918,33 @@ out_err:
         kfree(rootcontext);
         return rc;
 }
+/*
+ * string mount options parsing and call set the sbsec
+ */
+static int superblock_doinit(struct super_block *sb, void *data)
+{
+        int rc = 0;
+        char *options = data;
+        struct security_mnt_opts opts;
+
+        security_init_mnt_opts(&opts);
+
+        if (!data)
+                goto out;
+
+        BUG_ON(sb->s_type->fs_flags & FS_BINARY_MOUNTDATA);
+
+        rc = selinux_parse_opts_str(options, &opts);
+        if (rc)
+                goto out_err;
+
+out:
+        rc = selinux_set_mnt_opts(sb, &opts);
+
+out_err:
+        security_free_mnt_opts(&opts);
+        return rc;
+}
 
 static inline u16 inode_mode_to_security_class(umode_t mode)
 {
@@ -2253,7 +2270,7 @@ static inline void take_selinux_option(char **to, char *from, int *first,
         }
 }
 
-static int selinux_sb_copy_data(struct file_system_type *type, void *orig, void *copy)
+static int selinux_sb_copy_data(char *orig, char *copy)
 {
         int fnosec, fsec, rc = 0;
         char *in_save, *in_curr, *in_end;
@@ -2263,12 +2280,6 @@ static int selinux_sb_copy_data(struct file_system_type *type, void *orig, void
         in_curr = orig;
         sec_curr = copy;
 
-        /* Binary mount data: just copy */
-        if (type->fs_flags & FS_BINARY_MOUNTDATA) {
-                copy_page(sec_curr, in_curr);
-                goto out;
-        }
-
         nosec = (char *)get_zeroed_page(GFP_KERNEL);
         if (!nosec) {
                 rc = -ENOMEM;
@@ -5251,6 +5262,8 @@ static struct security_operations selinux_ops = {
         .sb_get_mnt_opts =              selinux_get_mnt_opts,
         .sb_set_mnt_opts =              selinux_set_mnt_opts,
         .sb_clone_mnt_opts =            selinux_sb_clone_mnt_opts,
+        .sb_parse_opts_str =            selinux_parse_opts_str,
+
 
         .inode_alloc_security =         selinux_inode_alloc_security,
         .inode_free_security =          selinux_inode_free_security,
diff --git a/security/selinux/include/security.h b/security/selinux/include/security.h
index 837ce420d2f6..f7d2f03781f2 100644
--- a/security/selinux/include/security.h
+++ b/security/selinux/include/security.h
@@ -35,6 +35,11 @@
 #define POLICYDB_VERSION_MAX    POLICYDB_VERSION_POLCAP
 #endif
 
+#define CONTEXT_MNT     0x01
+#define FSCONTEXT_MNT   0x02
+#define ROOTCONTEXT_MNT 0x04
+#define DEFCONTEXT_MNT  0x08
+
 struct netlbl_lsm_secattr;
 
 extern int selinux_enabled;
diff --git a/security/smack/smack.h b/security/smack/smack.h
index a21a0e907ab3..62c1e982849d 100644
--- a/security/smack/smack.h
+++ b/security/smack/smack.h
@@ -26,14 +26,6 @@
 #define SMK_MAXLEN      23
 #define SMK_LABELLEN    (SMK_MAXLEN+1)
 
-/*
- * How many kinds of access are there?
- * Here's your answer.
- */
-#define SMK_ACCESSDASH  '-'
-#define SMK_ACCESSLOW   "rwxa"
-#define SMK_ACCESSKINDS (sizeof(SMK_ACCESSLOW) - 1)
-
 struct superblock_smack {
         char            *smk_root;
         char            *smk_floor;
diff --git a/security/smack/smack_lsm.c b/security/smack/smack_lsm.c
index 770eb067e165..0241fd359675 100644
--- a/security/smack/smack_lsm.c
+++ b/security/smack/smack_lsm.c
@@ -189,17 +189,10 @@ static void smack_sb_free_security(struct super_block *sb)
  * Copy the Smack specific mount options out of the mount
  * options list.
  */
-static int smack_sb_copy_data(struct file_system_type *type, void *orig,
-                              void *smackopts)
+static int smack_sb_copy_data(char *orig, char *smackopts)
 {
         char *cp, *commap, *otheropts, *dp;
 
-        /* Binary mount data: just copy */
-        if (type->fs_flags & FS_BINARY_MOUNTDATA) {
-                copy_page(smackopts, orig);
-                return 0;
-        }
-
         otheropts = (char *)get_zeroed_page(GFP_KERNEL);
         if (otheropts == NULL)
                 return -ENOMEM;
diff --git a/security/smack/smackfs.c b/security/smack/smackfs.c
index 358c92c1a153..afe7c9b0732a 100644
--- a/security/smack/smackfs.c
+++ b/security/smack/smackfs.c
@@ -81,10 +81,23 @@ static struct semaphore smack_write_sem;
 /*
  * Values for parsing cipso rules
  * SMK_DIGITLEN: Length of a digit field in a rule.
- * SMK_CIPSOMEN: Minimum possible cipso rule length.
+ * SMK_CIPSOMIN: Minimum possible cipso rule length.
+ * SMK_CIPSOMAX: Maximum possible cipso rule length.
  */
 #define SMK_DIGITLEN 4
-#define SMK_CIPSOMIN (SMK_MAXLEN + 2 * SMK_DIGITLEN)
+#define SMK_CIPSOMIN (SMK_LABELLEN + 2 * SMK_DIGITLEN)
+#define SMK_CIPSOMAX (SMK_CIPSOMIN + SMACK_CIPSO_MAXCATNUM * SMK_DIGITLEN)
+
+/*
+ * Values for parsing MAC rules
+ * SMK_ACCESS: Maximum possible combination of access permissions
+ * SMK_ACCESSLEN: Maximum length for a rule access field
+ * SMK_LOADLEN: Smack rule length
+ */
+#define SMK_ACCESS    "rwxa"
+#define SMK_ACCESSLEN (sizeof(SMK_ACCESS) - 1)
+#define SMK_LOADLEN   (SMK_LABELLEN + SMK_LABELLEN + SMK_ACCESSLEN)
+
 
 /*
  * Seq_file read operations for /smack/load
@@ -229,14 +242,10 @@ static void smk_set_access(struct smack_rule *srp)
  * The format is exactly:
  *     char subject[SMK_LABELLEN]
  *     char object[SMK_LABELLEN]
- *     char access[SMK_ACCESSKINDS]
- *
- *     Anything following is commentary and ignored.
+ *     char access[SMK_ACCESSLEN]
  *
- * writes must be SMK_LABELLEN+SMK_LABELLEN+4 bytes.
+ * writes must be SMK_LABELLEN+SMK_LABELLEN+SMK_ACCESSLEN bytes.
  */
-#define MINIMUM_LOAD (SMK_LABELLEN + SMK_LABELLEN + SMK_ACCESSKINDS)
-
 static ssize_t smk_write_load(struct file *file, const char __user *buf,
                               size_t count, loff_t *ppos)
 {
@@ -253,7 +262,7 @@ static ssize_t smk_write_load(struct file *file, const char __user *buf,
                 return -EPERM;
         if (*ppos != 0)
                 return -EINVAL;
-        if (count < MINIMUM_LOAD)
+        if (count != SMK_LOADLEN)
                 return -EINVAL;
 
         data = kzalloc(count, GFP_KERNEL);
@@ -513,7 +522,7 @@ static ssize_t smk_write_cipso(struct file *file, const char __user *buf,
                 return -EPERM;
         if (*ppos != 0)
                 return -EINVAL;
-        if (count <= SMK_CIPSOMIN)
+        if (count < SMK_CIPSOMIN || count > SMK_CIPSOMAX)
                 return -EINVAL;
 
         data = kzalloc(count + 1, GFP_KERNEL);
@@ -547,7 +556,7 @@ static ssize_t smk_write_cipso(struct file *file, const char __user *buf,
         if (ret != 1 || catlen > SMACK_CIPSO_MAXCATNUM)
                 goto out;
 
-        if (count <= (SMK_CIPSOMIN + catlen * SMK_DIGITLEN))
+        if (count != (SMK_CIPSOMIN + catlen * SMK_DIGITLEN))
                 goto out;
 
         memset(mapcatset, 0, sizeof(mapcatset));