Hi, While researching https://bugs.launchpad.net/ubuntu/+source/apparmor/+bug/1661030 I discovered that when the exec() of the child process, we don't report FAIL to stdout, so the regression tests consider it an error rather than a failure and abort, short-circuiting the test script.
This patch fixes this by emitting the FAIL message when the result from the wait() syscall indicates the child process did not succeed. Nominated for 2.11, 2.10, and 2.9. Signed-off-by: Steve Beattie <st...@nxnw.org> --- tests/regression/apparmor/environ.c | 2 ++ 1 file changed, 2 insertions(+) Index: b/tests/regression/apparmor/environ.c =================================================================== --- a/tests/regression/apparmor/environ.c +++ b/tests/regression/apparmor/environ.c @@ -63,6 +63,8 @@ int main(int argc, char *argv[]) if (retval == RET_CHLD_SUCCESS) { printf("PASS\n"); retval = 0; + } else { + printf("FAIL: Child failed\n"); } } else if (pid == 0) { -- Steve Beattie <sbeat...@ubuntu.com> http://NxNW.org/~steve/
signature.asc
Description: PGP signature
-- AppArmor mailing list AppArmor@lists.ubuntu.com Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor