X-Git-Url: http://git.annexia.org/?p=libguestfs.git;a=blobdiff_plain;f=daemon%2Fcommand.c;h=b33f2dbe8a332a1e54c7387666ccd02f8e37039c;hp=589ca76931ae5fdb2596933b3af782592fa36044;hb=ad8a256f54a6cb99f89bb444c8597a152a793dce;hpb=5365ebd501850ea10d9a5b28fc6480ea34dbe16d diff --git a/daemon/command.c b/daemon/command.c index 589ca76..b33f2db 100644 --- a/daemon/command.c +++ b/daemon/command.c @@ -27,10 +27,11 @@ #include "actions.h" char * -do_command (char * const * const argv) +do_command (char **argv) { char *out, *err; int r; + int proc_ok, dev_ok, dev_pts_ok, sys_ok; /* We need a root filesystem mounted to do this. */ NEED_ROOT (NULL); @@ -44,10 +45,32 @@ do_command (char * const * const argv) return NULL; } + /* While running the command, bind-mount /dev, /proc, /sys + * into the chroot. However we must be careful to unmount them + * afterwards because otherwise they would interfere with + * future mount and unmount operations. + * + * We deliberately allow these commands to fail silently, BUT + * if a mount fails, don't unmount the corresponding mount. + */ + r = command (NULL, NULL, "mount", "--bind", "/dev", "/sysroot/dev", NULL); + dev_ok = r != -1; + r = command (NULL, NULL, "mount", "--bind", "/dev/pts", "/sysroot/dev/pts", NULL); + dev_pts_ok = r != -1; + r = command (NULL, NULL, "mount", "--bind", "/proc", "/sysroot/proc", NULL); + proc_ok = r != -1; + r = command (NULL, NULL, "mount", "--bind", "/sys", "/sysroot/sys", NULL); + sys_ok = r != -1; + CHROOT_IN; r = commandv (&out, &err, argv); CHROOT_OUT; + if (sys_ok) command (NULL, NULL, "umount", "/sysroot/sys", NULL); + if (proc_ok) command (NULL, NULL, "umount", "/sysroot/proc", NULL); + if (dev_pts_ok) command (NULL, NULL, "umount", "/sysroot/dev/pts", NULL); + if (dev_ok) command (NULL, NULL, "umount", "/sysroot/dev", NULL); + if (r == -1) { reply_with_error ("%s", err); free (out); @@ -61,41 +84,36 @@ do_command (char * const * const argv) } char ** -do_command_lines (char * const * const argv) +do_command_lines (char **argv) { char *out; - char **lines = NULL; - int size = 0, alloc = 0; - char *p, *pend; + char **lines; out = do_command (argv); if (out == NULL) return NULL; - /* Now convert the output to a list of lines. */ - p = out; - while (p) { - pend = strchr (p, '\n'); - if (pend) { - *pend = '\0'; - pend++; - } + lines = split_lines (out); + free (out); - /* Final \n? Don't return an empty final element. */ - if (pend && *pend == '\0') break; + if (lines == NULL) + return NULL; - if (add_string (&lines, &size, &alloc, p) == -1) { - free (out); - return NULL; - } + return lines; /* Caller frees. */ +} - p = pend; - } +char * +do_sh (char *command) +{ + char *argv[] = { "/bin/sh", "-c", command, NULL }; - free (out); + return do_command (argv); +} - if (add_string (&lines, &size, &alloc, NULL) == -1) - return NULL; +char ** +do_sh_lines (char *command) +{ + char *argv[] = { "/bin/sh", "-c", command, NULL }; - return lines; + return do_command_lines (argv); }