On 24.04.2018 19:58, 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>
> Reviewed-by: Philippe Mathieu-Daudé <f4...@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 <th...@redhat.com>
