On Thu, Dec 20, 2018 at 01:28:38PM -0800, Eric Chiang wrote:
> --- a/security/apparmor/policy_unpack.c
> +++ b/security/apparmor/policy_unpack.c
> @@ -535,6 +535,24 @@ static bool unpack_xattrs(struct aa_ext *e, struct 
> aa_profile *profile)
>                       goto fail;
>       }
>  
> +     if (unpack_nameX(e, AA_STRUCT, "xattr_keys")) {
> +             int i, size;
> +
> +             size = unpack_array(e, NULL);
> +             profile->xattr_keys_count = size;
> +             profile->xattr_keys = kcalloc(size, sizeof(char *), GFP_KERNEL);

Do we need to worry about a zero-size array here?

Thanks

Attachment: signature.asc
Description: PGP signature

-- 
AppArmor mailing list
AppArmor@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to