From b1613c26518abb55ae5fc970dccfb7e3c97398d1 Mon Sep 17 00:00:00 2001 From: Dan McGee Date: Fri, 5 Oct 2007 00:13:36 -0500 Subject: Clean up the scriptlet fork code a bit, honor the child return value Signed-off-by: Dan McGee --- lib/libalpm/trans.c | 23 ++++++++++++++++++++--- 1 file changed, 20 insertions(+), 3 deletions(-) diff --git a/lib/libalpm/trans.c b/lib/libalpm/trans.c index 462b4d8b..22780899 100644 --- a/lib/libalpm/trans.c +++ b/lib/libalpm/trans.c @@ -608,6 +608,7 @@ int _alpm_runscriptlet(const char *root, const char *installfn, } _alpm_log(PM_LOG_DEBUG, "%s\n", cmdline); + /* fork- parent and child each have seperate code blocks below */ pid = fork(); if(pid == -1) { _alpm_log(PM_LOG_ERROR, _("could not fork a new process (%s)\n"), strerror(errno)); @@ -616,13 +617,16 @@ int _alpm_runscriptlet(const char *root, const char *installfn, } if(pid == 0) { + /* this code runs for the child only (the actual chroot/exec) */ _alpm_log(PM_LOG_DEBUG, "chrooting in %s\n", root); if(chroot(root) != 0) { - _alpm_log(PM_LOG_ERROR, _("could not change the root directory (%s)\n"), strerror(errno)); + _alpm_log(PM_LOG_ERROR, _("could not change the root directory (%s)\n"), + strerror(errno)); exit(1); } if(chdir("/") != 0) { - _alpm_log(PM_LOG_ERROR, _("could not change directory to / (%s)\n"), strerror(errno)); + _alpm_log(PM_LOG_ERROR, _("could not change directory to / (%s)\n"), + strerror(errno)); exit(1); } umask(0022); @@ -630,11 +634,24 @@ int _alpm_runscriptlet(const char *root, const char *installfn, execl("/bin/sh", "sh", "-c", cmdline, (char *)NULL); exit(0); } else { - if(waitpid(pid, 0, 0) == -1) { + /* this code runs for the parent only (wait on the child) */ + pid_t retpid; + int status; + retpid = waitpid(pid, &status, 0); + if(retpid == -1) { _alpm_log(PM_LOG_ERROR, _("call to waitpid failed (%s)\n"), strerror(errno)); retval = 1; goto cleanup; + } else { + /* check the return status, make sure it is 0 (success) */ + if(WIFEXITED(status)) { + _alpm_log(PM_LOG_DEBUG, "call to waitpid succeeded\n"); + if(WEXITSTATUS(status) != 0) { + _alpm_log(PM_LOG_ERROR, _("scriptlet failed to execute correctly\n")); + retval = 1; + } + } } } -- cgit v1.2.3-70-g09d2