On 11/22/23 03:56, Ben Dooks wrote:
The stack_msg is not exported out of security/apparmor/domain.c so
make it static to avoid the following sparse warning:

security/apparmor/domain.c:1314:12: warning: symbol 'stack_msg' was not 
declared. Should it be static?

Signed-off-by: Ben Dooks <[email protected]>

thank you for your contribution, unfortunately this fix has already landed in 
apparmor-next on Nov 9 with commit

3c49ce0e2209 apparmor: declare stack_msg as static


---
  security/apparmor/domain.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/security/apparmor/domain.c b/security/apparmor/domain.c
index 89fbeab4b33b..571158ec6188 100644
--- a/security/apparmor/domain.c
+++ b/security/apparmor/domain.c
@@ -1311,7 +1311,7 @@ static int change_profile_perms_wrapper(const char *op, 
const char *name,
        return error;
  }
-const char *stack_msg = "change_profile unprivileged unconfined converted to stacking";
+static const char *stack_msg = "change_profile unprivileged unconfined converted to 
stacking";
/**
   * aa_change_profile - perform a one-way profile transition


Reply via email to