diff options
author | Michael Tokarev <mjt@tls.msk.ru> | 2012-01-16 18:37:44 +0400 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2012-02-17 12:41:48 +0100 |
commit | 9faf31b68f049ca6432a1ef086a76344ff2a2357 (patch) | |
tree | 1998d8c9943f6a50063833062ac8c4062a40e287 /qemu-nbd.c | |
parent | 9de36b1a7cf61aa8be365f13c81668b3e19fbc7f (diff) | |
download | hqemu-9faf31b68f049ca6432a1ef086a76344ff2a2357.zip hqemu-9faf31b68f049ca6432a1ef086a76344ff2a2357.tar.gz |
do not chdir(/) in qemu-nbd before opening all files
When qemu-nbd becomes a daemon it calls daemon(3) with
nochdir=0, so daemon(3) changes current directory to /.
But at this time, qemu-nbd did not open any user-specified
files yet, so by changing current directory, all non-absolute
paths becomes wrong. The solution is to pass nochdir=1 to
daemon(3) function, and to chdir("/") after all init has
been performed, before entering the main loop, -- just like
a good daemon should do.
This patch is applicable for -stable.
Signed-off-by: Michael Tokarev <mjt@tls.msk.ru>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'qemu-nbd.c')
-rw-r--r-- | qemu-nbd.c | 8 |
1 files changed, 7 insertions, 1 deletions
@@ -429,7 +429,7 @@ int main(int argc, char **argv) pid = fork(); if (pid == 0) { close(stderr_fd[0]); - ret = qemu_daemon(0, 0); + ret = qemu_daemon(1, 0); /* Temporarily redirect stderr to the parent's pipe... */ dup2(stderr_fd[1], STDERR_FILENO); @@ -527,6 +527,12 @@ int main(int argc, char **argv) qemu_set_fd_handler2(fd, nbd_can_accept, nbd_accept, NULL, (void *)(uintptr_t)fd); + /* now when the initialization is (almost) complete, chdir("/") + * to free any busy filesystems */ + if (chdir("/") < 0) { + err(EXIT_FAILURE, "Could not chdir to root directory"); + } + do { main_loop_wait(false); } while (!sigterm_reported && (persistent || !nbd_started || nb_fds > 0)); |