[Spice-devel] [vdagent-linux] vdagent: Return '1' when virtio device cannot be opened

Frediano Ziglio fziglio at redhat.com
Mon Sep 7 08:52:54 PDT 2015


Acked!

> The vdagent process currently exits with an error code set to 0 whenn
> the virtio device cannot be opened (for example because it's missing).
> This is not consistent with the other failures to startup which set the
> exit code to 1. This commit ensures 1 is returned in this situation too.
> 
> Resolves: rhbz#1256704
> ---
>  src/vdagent.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/src/vdagent.c b/src/vdagent.c
> index 7600b67..6c4ee8c 100644
> --- a/src/vdagent.c
> +++ b/src/vdagent.c
> @@ -303,7 +303,8 @@ int main(int argc, char *argv[])
>              LOG_USER);
>  
>      if (file_test(portdev) != 0) {
> -        return 0;
> +        syslog(LOG_ERR, "Cannot access vdagent virtio channel %s", portdev);
> +        return 1;
>      }
>  
>      if (do_daemonize)
> --
> 2.4.3
> 


More information about the Spice-devel mailing list