Hi Ian, On 04/19/2018 01:45 PM, Ian Jackson wrote: > Signed-off-by: Ian Jackson <ian.jack...@eu.citrix.com> > CC: Paolo Bonzini <pbonz...@redhat.com> > CC: Markus Armbruster <arm...@redhat.com> > CC: Daniel P. Berrange <berra...@redhat.com> > CC: Michael Tokarev <m...@tls.msk.ru> > --- > v7: New patch > --- > os-posix.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/os-posix.c b/os-posix.c > index 0f59566..d4cf466 100644 > --- a/os-posix.c > +++ b/os-posix.c > @@ -129,7 +129,7 @@ void os_set_proc_name(const char *s) > exit(1); > } > #else > - fprintf(stderr, "Change of process name not supported by your OS\n"); > + error_report("Change of process name not supported by your OS\n");
removing the trailing "\n": Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> > exit(1); > #endif > } > @@ -243,7 +243,7 @@ static void change_root(void) > { > if (chroot_dir) { > if (chroot(chroot_dir) < 0) { > - fprintf(stderr, "chroot failed\n"); > + error_report("chroot failed"); > exit(1); > } > if (chdir("/")) { >