Smack: fix smack_new_inode bogosities
authorCasey Schaufler <casey@schaufler-ca.com>
Thu, 24 May 2012 00:46:58 +0000 (17:46 -0700)
committerCasey Schaufler <casey@schaufler-ca.com>
Fri, 13 Jul 2012 22:49:23 +0000 (15:49 -0700)
commiteb982cb4cf6405b97ea1f9e1d10864981f269d46
tree8d89448e0ef96d587ea8052021a721e632b4b318
parent417c6c8ee2eb6975f357d8975af94ba5fbeaf82d
Smack: fix smack_new_inode bogosities

In January of 2012 Al Viro pointed out three bits of code that
he titled "new_inode_smack bogosities". This patch repairs these
errors.

1. smack_sb_kern_mount() included a NULL check that is impossible.
   The check and NULL case are removed.
2. smack_kb_kern_mount() included pointless locking. The locking is
   removed. Since this is the only place that lock was used the lock
   is removed from the superblock_smack structure.
3. smk_fill_super() incorrectly and unnecessarily set the Smack label
   for the smackfs root inode. The assignment has been removed.

Targeted for git://gitorious.org/smack-next/kernel.git

Signed-off-by: Casey Schaufler <casey@schaufler-ca.com>
security/smack/smack.h
security/smack/smack_lsm.c
security/smack/smackfs.c