diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2008-12-09 09:34:39 -0500 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2009-01-05 11:54:28 -0500 |
| commit | 56ff5efad96182f4d3cb3dc6b07396762c658f16 (patch) | |
| tree | cb91f93aa2324573527165d56d230b606a3111ed /security | |
| parent | acfa4380efe77e290d3a96b11cd4c9f24f4fbb18 (diff) | |
| download | op-kernel-dev-56ff5efad96182f4d3cb3dc6b07396762c658f16.zip op-kernel-dev-56ff5efad96182f4d3cb3dc6b07396762c658f16.tar.gz | |
zero i_uid/i_gid on inode allocation
... and don't bother in callers. Don't bother with zeroing i_blocks,
while we are at it - it's already been zeroed.
i_mode is not worth the effort; it has no common default value.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'security')
| -rw-r--r-- | security/inode.c | 3 | ||||
| -rw-r--r-- | security/selinux/selinuxfs.c | 2 |
2 files changed, 0 insertions, 5 deletions
diff --git a/security/inode.c b/security/inode.c index efea5a6..007ef25 100644 --- a/security/inode.c +++ b/security/inode.c @@ -61,9 +61,6 @@ static struct inode *get_inode(struct super_block *sb, int mode, dev_t dev) if (inode) { inode->i_mode = mode; - inode->i_uid = 0; - inode->i_gid = 0; - inode->i_blocks = 0; inode->i_atime = inode->i_mtime = inode->i_ctime = CURRENT_TIME; switch (mode & S_IFMT) { default: diff --git a/security/selinux/selinuxfs.c b/security/selinux/selinuxfs.c index e552099..8f612c8 100644 --- a/security/selinux/selinuxfs.c +++ b/security/selinux/selinuxfs.c @@ -847,8 +847,6 @@ static struct inode *sel_make_inode(struct super_block *sb, int mode) if (ret) { ret->i_mode = mode; - ret->i_uid = ret->i_gid = 0; - ret->i_blocks = 0; ret->i_atime = ret->i_mtime = ret->i_ctime = CURRENT_TIME; } return ret; |
