On Mon, Jul 17, 2023 at 04:55:42PM +0200, Denis V. Lunev wrote: > We are trying to temporary redirect stderr of daemonized process to
temporarily > a pipe to report a error and get failed. In that case we could not > use error_report() helper, but should write the message directly into > the problematic pipe. > > Signed-off-by: Denis V. Lunev <d...@openvz.org> > CC: Eric Blake <ebl...@redhat.com> > CC: Vladimir Sementsov-Ogievskiy <vsement...@yandex-team.ru> > --- > qemu-nbd.c | 15 ++++++++++++++- > 1 file changed, 14 insertions(+), 1 deletion(-) > > diff --git a/qemu-nbd.c b/qemu-nbd.c > index 186ce9474c..4450cc826b 100644 > --- a/qemu-nbd.c > +++ b/qemu-nbd.c > @@ -937,7 +937,20 @@ int main(int argc, char **argv) > ret = qemu_daemon(1, 0); > errno from qemu_daemon... > /* Temporarily redirect stderr to the parent's pipe... */ > - dup2(stderr_fd[1], STDERR_FILENO); > + if (dup2(stderr_fd[1], STDERR_FILENO) < 0) { ...is still potentially lost here... > + char str[256]; > + snprintf(str, sizeof(str), > + "%s: Failed to link stderr to the pipe: %s\n", > + g_get_prgname(), strerror(errno)); > + /* > + * We are unable to use error_report() here as we need to get > + * stderr pointed to the parent's pipe. Write to that pipe > + * manually. > + */ > + ret = write(stderr_fd[1], str, strlen(str)); > + exit(EXIT_FAILURE); > + } > + > if (ret < 0) { > error_report("Failed to daemonize: %s", strerror(errno)); ...before use here. Patch 4/5 addresses that, but I'm inclined to just swap the order of the two patches when applying the series in my NBD tree. Reviewed-by: Eric Blake <ebl...@redhat.com> -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org