SELinux: do not set automatic i_ino in selinuxfs
authorEric Paris <eparis@redhat.com>
Mon, 29 Nov 2010 20:46:39 +0000 (15:46 -0500)
committerEric Paris <eparis@redhat.com>
Tue, 30 Nov 2010 22:28:57 +0000 (17:28 -0500)
selinuxfs carefully uses i_ino to figure out what the inode refers to.  The
VFS used to generically set this value and we would reset it to something
useable.  After 85fe4025c616 each filesystem sets this value to a default
if needed.  Since selinuxfs doesn't use the default value and it can only
lead to problems (I'd rather have 2 inodes with i_ino == 0 than one
pointing to the wrong data) lets just stop setting a default.

Signed-off-by: Eric Paris <eparis@redhat.com>
Acked-by: James Morris <jmorris@namei.org>
security/selinux/selinuxfs.c

index 8bae68e21af9ed70c079ef8dab41b3c8923b5dff..45d35e629fc6bec0bce943c33287bfb46e640a0e 100644 (file)
@@ -989,7 +989,6 @@ static struct inode *sel_make_inode(struct super_block *sb, int mode)
        struct inode *ret = new_inode(sb);
 
        if (ret) {
-               ret->i_ino = get_next_ino();
                ret->i_mode = mode;
                ret->i_atime = ret->i_mtime = ret->i_ctime = CURRENT_TIME;
        }