On Wed, Mar 25, 2015 at 05:37:16PM -0500, Tyler Hicks wrote: > The two internal aa_features objects weren't being unreferenced when the > aa_policy_cache object was being freed. > > Signed-off-by: Tyler Hicks <tyhi...@canonical.com> Acked-by: Steve Beattie <st...@nxnw.org>
Thanks.
> ---
> libraries/libapparmor/src/policy_cache.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/libraries/libapparmor/src/policy_cache.c
> b/libraries/libapparmor/src/policy_cache.c
> index 98155a6..a9e43bb 100644
> --- a/libraries/libapparmor/src/policy_cache.c
> +++ b/libraries/libapparmor/src/policy_cache.c
> @@ -203,6 +203,8 @@ aa_policy_cache *aa_policy_cache_ref(aa_policy_cache
> *policy_cache)
> void aa_policy_cache_unref(aa_policy_cache *policy_cache)
> {
> if (policy_cache && atomic_dec_and_test(&policy_cache->ref_count)) {
> + aa_features_unref(policy_cache->kernel_features);
> + aa_features_unref(policy_cache->features);
> free(policy_cache->features_path);
> free(policy_cache->path);
> free(policy_cache);
> --
> 2.1.4
>
>
> --
> AppArmor mailing list
> AppArmor@lists.ubuntu.com
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/apparmor
--
Steve Beattie
<sbeat...@ubuntu.com>
http://NxNW.org/~steve/
signature.asc
Description: Digital signature
-- AppArmor mailing list AppArmor@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
