NACK: [SRU][Bionic][PATCH 0/1] Fix memory leak on profile removal

[Stefan Bader]

On 13.08.21 21:07, Georgia Garcia wrote:
> BugLink: https://bugs.launchpad.net/bugs/1939915
> 
> SRU Justification:
> 
> [Impact]
> There's a memory leak on AppArmor when removing a profile. When the
> proxy isn't replaced and the profile is removed, the proxy is leaked.
> 
> [Fix]
> 
> Upstream commit 3622ad25d4d fixes the leak by cleaning up the label
> structure within the profile when the profile is getting freed. The
> proxy is freed correctly when cleaning up the label.
> 
> [Test Plan]
> 
> /sys/kernel/debug/kmemleak should not return a memleak when removing
> a profile.
> 
> root at ubuntu:~# echo "profile foo {}" > profile
> root at ubuntu:~# apparmor_parser profile
> root at ubuntu:~# echo scan > /sys/kernel/debug/kmemleak
> root at ubuntu:~# cat /sys/kernel/debug/kmemleak
> 
> [Where problems could occur]
> Low probability of any problem. There's no longer a leak.
> 
> 
> John Johansen (1):
>    apparmor: Fix memory leak of profile proxy
> 
>   security/apparmor/include/label.h |  1 +
>   security/apparmor/label.c         | 17 +++++++----------
>   security/apparmor/policy.c        |  1 +
>   3 files changed, 9 insertions(+), 10 deletions(-)
> 
Comment on Xenial part

-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20210816/c7e67377/attachment.sig>