diff options
author | Eric Paris <eparis@redhat.com> | 2010-11-29 15:46:39 -0500 |
---|---|---|
committer | Eric Paris <eparis@redhat.com> | 2010-11-30 17:28:57 -0500 |
commit | c9e86a9b95f198d7df49b25fcd808ee39cba218f (patch) | |
tree | 0e62d348103f25a612d649c796cab225db2372c3 /security | |
parent | 7ae9f23cbd3ef9daff7f768da4bfd4c56b19300d (diff) |
SELinux: do not set automatic i_ino in selinuxfs
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>
Diffstat (limited to 'security')
-rw-r--r-- | security/selinux/selinuxfs.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/security/selinux/selinuxfs.c b/security/selinux/selinuxfs.c index 8bae68e21af..45d35e629fc 100644 --- a/security/selinux/selinuxfs.c +++ b/security/selinux/selinuxfs.c | |||
@@ -989,7 +989,6 @@ static struct inode *sel_make_inode(struct super_block *sb, int mode) | |||
989 | struct inode *ret = new_inode(sb); | 989 | struct inode *ret = new_inode(sb); |
990 | 990 | ||
991 | if (ret) { | 991 | if (ret) { |
992 | ret->i_ino = get_next_ino(); | ||
993 | ret->i_mode = mode; | 992 | ret->i_mode = mode; |
994 | ret->i_atime = ret->i_mtime = ret->i_ctime = CURRENT_TIME; | 993 | ret->i_atime = ret->i_mtime = ret->i_ctime = CURRENT_TIME; |
995 | } | 994 | } |