On 2016-03-18 10:29:07, [email protected] wrote: > 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]>
Acked-by: Tyler Hicks <[email protected]> Thanks! > --- > 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
signature.asc
Description: PGP signature
-- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
