Applied. Thanks! Tom
On Sat, Oct 11, 2014 at 10:13 AM, <[email protected]> wrote: > From: Michael Scherer <[email protected]> > > Since aa_change_onexec return the error code in errno, and return > -1, the current code do not give any useful information when > something fail. This make apparmor easier to debug, as seen on > https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=760526 > --- > src/core/execute.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/core/execute.c b/src/core/execute.c > index 8b9bb27..b165b33 100644 > --- a/src/core/execute.c > +++ b/src/core/execute.c > @@ -1698,7 +1698,7 @@ static int exec_child(ExecCommand *command, > err = aa_change_onexec(context->apparmor_profile); > if (err < 0 && !context->apparmor_profile_ignore) { > *error = EXIT_APPARMOR_PROFILE; > - return err; > + return -errno; > } > } > #endif > -- > 1.8.3.1 > > _______________________________________________ > systemd-devel mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/systemd-devel _______________________________________________ systemd-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/systemd-devel
