diff options
author | Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp> | 2013-05-27 07:11:27 -0400 |
---|---|---|
committer | Casey Schaufler <casey@schaufler-ca.com> | 2013-05-28 13:15:35 -0400 |
commit | 8cd77a0bd4b4a7d02c2a6926a69585d8088ee721 (patch) | |
tree | d74853403421414232b56b9f0d7e61a1733c6bf9 /security/smack/smack_access.c | |
parent | e830b39412ca2bbedd7508243f21c04d57ad543c (diff) |
Smack: Fix possible NULL pointer dereference at smk_netlbl_mls()
netlbl_secattr_catmap_alloc(GFP_ATOMIC) can return NULL.
Signed-off-by: Tetsuo Handa <penguin-kernel@I-love.SAKURA.ne.jp>
Diffstat (limited to 'security/smack/smack_access.c')
-rw-r--r-- | security/smack/smack_access.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/security/smack/smack_access.c b/security/smack/smack_access.c index 53f2327a592f..6a0377f38620 100644 --- a/security/smack/smack_access.c +++ b/security/smack/smack_access.c | |||
@@ -403,6 +403,8 @@ int smk_netlbl_mls(int level, char *catset, struct netlbl_lsm_secattr *sap, | |||
403 | sap->flags |= NETLBL_SECATTR_MLS_CAT; | 403 | sap->flags |= NETLBL_SECATTR_MLS_CAT; |
404 | sap->attr.mls.lvl = level; | 404 | sap->attr.mls.lvl = level; |
405 | sap->attr.mls.cat = netlbl_secattr_catmap_alloc(GFP_ATOMIC); | 405 | sap->attr.mls.cat = netlbl_secattr_catmap_alloc(GFP_ATOMIC); |
406 | if (!sap->attr.mls.cat) | ||
407 | return -ENOMEM; | ||
406 | sap->attr.mls.cat->startbit = 0; | 408 | sap->attr.mls.cat->startbit = 0; |
407 | 409 | ||
408 | for (cat = 1, cp = catset, byte = 0; byte < len; cp++, byte++) | 410 | for (cat = 1, cp = catset, byte = 0; byte < len; cp++, byte++) |