userns: Convert debugfs to use kuid/kgid where appropriate.
authorEric W. Biederman <ebiederm@xmission.com>
Tue, 3 Apr 2012 21:01:31 +0000 (14:01 -0700)
committerEric W. Biederman <ebiederm@xmission.com>
Fri, 7 Sep 2012 02:02:52 +0000 (19:02 -0700)
Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Acked-by: Serge Hallyn <serge.hallyn@canonical.com>
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
fs/debugfs/inode.c
init/Kconfig

index 4733eab34a237faa58f57ea14bce96c9659cb2d7..36e2b667e82280e62c108113cd2c02724b14a5ed 100644 (file)
@@ -128,8 +128,8 @@ static inline int debugfs_positive(struct dentry *dentry)
 }
 
 struct debugfs_mount_opts {
-       uid_t uid;
-       gid_t gid;
+       kuid_t uid;
+       kgid_t gid;
        umode_t mode;
 };
 
@@ -156,6 +156,8 @@ static int debugfs_parse_options(char *data, struct debugfs_mount_opts *opts)
        substring_t args[MAX_OPT_ARGS];
        int option;
        int token;
+       kuid_t uid;
+       kgid_t gid;
        char *p;
 
        opts->mode = DEBUGFS_DEFAULT_MODE;
@@ -169,12 +171,18 @@ static int debugfs_parse_options(char *data, struct debugfs_mount_opts *opts)
                case Opt_uid:
                        if (match_int(&args[0], &option))
                                return -EINVAL;
-                       opts->uid = option;
+                       uid = make_kuid(current_user_ns(), option);
+                       if (!uid_valid(uid))
+                               return -EINVAL;
+                       opts->uid = uid;
                        break;
                case Opt_gid:
                        if (match_octal(&args[0], &option))
                                return -EINVAL;
-                       opts->gid = option;
+                       gid = make_kgid(current_user_ns(), option);
+                       if (!gid_valid(gid))
+                               return -EINVAL;
+                       opts->gid = gid;
                        break;
                case Opt_mode:
                        if (match_octal(&args[0], &option))
@@ -226,10 +234,12 @@ static int debugfs_show_options(struct seq_file *m, struct dentry *root)
        struct debugfs_fs_info *fsi = root->d_sb->s_fs_info;
        struct debugfs_mount_opts *opts = &fsi->mount_opts;
 
-       if (opts->uid != 0)
-               seq_printf(m, ",uid=%u", opts->uid);
-       if (opts->gid != 0)
-               seq_printf(m, ",gid=%u", opts->gid);
+       if (!uid_eq(opts->uid, GLOBAL_ROOT_UID))
+               seq_printf(m, ",uid=%u",
+                          from_kuid_munged(&init_user_ns, opts->uid));
+       if (!gid_eq(opts->gid, GLOBAL_ROOT_GID))
+               seq_printf(m, ",gid=%u",
+                          from_kgid_munged(&init_user_ns, opts->gid));
        if (opts->mode != DEBUGFS_DEFAULT_MODE)
                seq_printf(m, ",mode=%o", opts->mode);
 
index fdabc5160cdffb1cc66cee8633197aedf8c282af..071dbb4928ef5961fd7c507de24f6abddb0c8564 100644 (file)
@@ -964,7 +964,6 @@ config UIDGID_CONVERTED
        depends on CODA_FS = n
        depends on CONFIGFS_FS = n
        depends on CRAMFS = n
-       depends on DEBUG_FS = n
        depends on ECRYPT_FS = n
        depends on EFS_FS = n
        depends on EXOFS_FS = n