apparmor: fix cast in format string DEBUG statement

jrjohansen · jrjohansen · commit 1b51bd761599 · 2026-02-03T04:47:32.000-08:00
if debugging is enabled the DEBUG statement will fail do to a bad fat fingered cast. Fixes: 102ada7 ("apparmor: fix fmt string type error in process_strs_entry") Signed-off-by: John Johansen <john.johansen@canonical.com>
diff --git a/security/apparmor/policy_unpack.c b/security/apparmor/policy_unpack.c
@@ -546,7 +546,7 @@ static int unpack_strs_table(struct aa_ext *e, const char *name, bool multi,
 			if (c <= 0) {
 				AA_DEBUG(DEBUG_UNPACK, "process_strs %d i %d pos %ld",
 					 c, i,
-					 (unsigned_long) e->pos - saved_pos);
+					 (unsigned long)(e->pos - saved_pos));
 				goto fail;
 			}
 			if (!multi && c > 1) {

Original file line number	Diff line number	Diff line change
`@@ -546,7 +546,7 @@ static int unpack_strs_table(struct aa_ext e, const char name, bool multi,`
`546`	`546`	`if (c <= 0) {`
`547`	`547`	`AA_DEBUG(DEBUG_UNPACK, "process_strs %d i %d pos %ld",`
`548`	`548`	`c, i,`
`549`		`- (unsigned_long) e->pos - saved_pos);`
	`549`	`+ (unsigned long)(e->pos - saved_pos));`
`550`	`550`	`goto fail;`
`551`	`551`	`}`
`552`	`552`	`if (!multi && c > 1) {`