smackfs: check for allocation failures in smk_set_access()
authorSergio Luis <sergio@larces.uece.br>
Mon, 22 Dec 2008 04:16:15 +0000 (01:16 -0300)
committerJames Morris <jmorris@namei.org>
Thu, 25 Dec 2008 01:14:55 +0000 (12:14 +1100)
commit81ea714bf148fce35e931edcbdfd3aedda20d1dc
treee3cabfd2ce35bc8be542910bffc4b9b99288a7f4
parent74192246910ff4fb95309ba1a683215644beeb62
smackfs: check for allocation failures in smk_set_access()

smackfs: check for allocation failures in smk_set_access()

 While adding a new subject/object pair to smack_list, smk_set_access()
 didn't check the return of kzalloc().

 This patch changes smk_set_access() to return 0 or -ENOMEM, based on
 kzalloc()'s return. It also updates its caller, smk_write_load(), to
 check for smk_set_access()'s return, given it is no longer a void
 return function.

Signed-off-by: Sergio Luis <sergio@larces.uece.br>
 To: Casey Schaufler <casey@schaufler-ca.com>
Cc: Ahmed S. Darwish <darwish.07@gmail.com>
Cc: LSM <linux-security-module@vger.kernel.org>
Cc: LKLM <linux-kernel@vger.kernel.org>
Acked-by: Casey Schaufler <casey@schaufler-ca.com>
security/smack/smackfs.c