diff options
author | Peter Zijlstra <peterz@infradead.org> | 2023-06-09 09:48:59 +0200 |
---|---|---|
committer | Peter Zijlstra <peterz@infradead.org> | 2023-06-26 11:14:18 +0200 |
commit | 9a1f37ebcfe061721564042254719dc8fd5c9fa0 (patch) | |
tree | 07911d9c97e45740014b44f525b84af37fb15c37 /security/apparmor | |
parent | dmaengine: ioat: Free up __cleanup() name (diff) | |
download | linux-9a1f37ebcfe061721564042254719dc8fd5c9fa0.tar.xz linux-9a1f37ebcfe061721564042254719dc8fd5c9fa0.zip |
apparmor: Free up __cleanup() name
In order to use __cleanup for __attribute__((__cleanup__(func))) the
name must not be used for anything else. Avoid the conflict.
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Acked-by: John Johansen <john.johansen@canonical.com>
Link: https://lkml.kernel.org/r/20230612093537.536441207%40infradead.org
Diffstat (limited to 'security/apparmor')
-rw-r--r-- | security/apparmor/include/lib.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/security/apparmor/include/lib.h b/security/apparmor/include/lib.h index f1a29ab7ea1b..73c8a32c6861 100644 --- a/security/apparmor/include/lib.h +++ b/security/apparmor/include/lib.h @@ -232,7 +232,7 @@ void aa_policy_destroy(struct aa_policy *policy); */ #define fn_label_build(L, P, GFP, FN) \ ({ \ - __label__ __cleanup, __done; \ + __label__ __do_cleanup, __done; \ struct aa_label *__new_; \ \ if ((L)->size > 1) { \ @@ -250,7 +250,7 @@ void aa_policy_destroy(struct aa_policy *policy); __new_ = (FN); \ AA_BUG(!__new_); \ if (IS_ERR(__new_)) \ - goto __cleanup; \ + goto __do_cleanup; \ __lvec[__j++] = __new_; \ } \ for (__j = __count = 0; __j < (L)->size; __j++) \ @@ -272,7 +272,7 @@ void aa_policy_destroy(struct aa_policy *policy); vec_cleanup(profile, __pvec, __count); \ } else \ __new_ = NULL; \ -__cleanup: \ +__do_cleanup: \ vec_cleanup(label, __lvec, (L)->size); \ } else { \ (P) = labels_profile(L); \ |