apparmor: get_suspend_policy must return NULL in error cases

Before this fix, it could return MAP_FAILED which is ((void *) -1). Signed-off-by: Andrei Vagin <[email protected]>
adrianreber · Jul 3, 2024 · c6c83f1 · c6c83f1
1 parent c2f101a
commit c6c83f1
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/criu/apparmor.c b/criu/apparmor.c
@@ -469,6 +469,7 @@ static void *get_suspend_policy(char *name, off_t *len)
 	ret = mmap(NULL, sb.st_size, PROT_READ, MAP_PRIVATE, fd, 0);
 	if (ret == MAP_FAILED) {
 		pr_perror("mmap of %s failed", file);
+		ret = NULL;
 		goto out;
 	}