On 24.04.2018 19:58, Ian Jackson wrote:
> Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
> CC: Paolo Bonzini <pbonzini@redhat.com>
> CC: Markus Armbruster <armbru@redhat.com>
> CC: Daniel P. Berrange <berrange@redhat.com>
> CC: Michael Tokarev <mjt@tls.msk.ru>
> Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> v8: Remove one remaining spurious "\n"
> 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..a2ba50d 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");
> 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("/")) {
>
Maybe merge this with patch 9/16?
Anyway:
Reviewed-by: Thomas Huth <thuth@redhat.com>