The onexec test was ignoring errors from the kill() call, so it didn't
notice when it had unsuccessfully sent SIGSTOP to itself.

Signed-off-by: Steve Beattie <[email protected]>
---
 tests/regression/apparmor/onexec.c |    7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

Index: b/tests/regression/apparmor/onexec.c
===================================================================
--- a/tests/regression/apparmor/onexec.c
+++ b/tests/regression/apparmor/onexec.c
@@ -45,7 +45,12 @@ int main(int argc, char *argv[])
 
        /* stop after onexec and wait to for continue before exec so
         * caller can introspect task */
-       (void)kill(getpid(), SIGSTOP);
+       rc = kill(getpid(), SIGSTOP);
+       if (rc == -1){
+               fprintf(stderr, "FAIL: signal to self failed - %s\n",
+                       strerror(errno));
+               exit(errno);
+       }
 
        (void)execve(argv[2], &argv[2], environ);
        /* exec failed, kill outselves to flag parent */


-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to