From: John Johansen Date: Fri, 14 Apr 2023 07:24:47 +0000 (-0700) Subject: apparmor: add missing failure check in compute_xmatch_perms X-Git-Tag: v6.6.17~4419^2~5 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=6600e9f692e36e265ef0828f08337fa294bb330f;p=platform%2Fkernel%2Flinux-rpi.git apparmor: add missing failure check in compute_xmatch_perms Add check for failure to allocate the permission table. Fixes: caa9f579ca72 ("apparmor: isolate policy backwards compatibility to its own file") Signed-off-by: John Johansen --- diff --git a/security/apparmor/policy_compat.c b/security/apparmor/policy_compat.c index 9e52e21..b2ec7fb 100644 --- a/security/apparmor/policy_compat.c +++ b/security/apparmor/policy_compat.c @@ -180,6 +180,8 @@ static struct aa_perms *compute_xmatch_perms(struct aa_dfa *xmatch) state_count = xmatch->tables[YYTD_ID_BASE]->td_lolen; /* DFAs are restricted from having a state_count of less than 2 */ perms = kvcalloc(state_count, sizeof(struct aa_perms), GFP_KERNEL); + if (!perms) + return NULL; /* zero init so skip the trap state (state == 0) */ for (state = 1; state < state_count; state++)