[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 13/25] vnc: add error propagation to vnc_display
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 13/25] vnc: add error propagation to vnc_display_open |
Date: |
Wed, 17 Oct 2012 17:48:20 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20121009 Thunderbird/16.0 |
Il 17/10/2012 17:17, Markus Armbruster ha scritto:
>> > +fail:
>> > + if (!error_is_set(errp)) {
>> > + error_set(errp, QERR_VNC_SERVER_FAILED, display);
> How can we get here with no error set?
>
> 1. !vnc_display (first goto fail).
This can be fixed up to give a separate error.
> 2. unit_connect() or inet_listen() return failure, but don't set error.
>
> 3. unix_listen() or inet_listen() return failure, but don't set error.
>
> Can 2. or 3. happen?
>
> If yes, these functions suck. If no, let's fix up 1. to set a suitable
> error, and drop the uninformative generic error here.
>
It can at this point in the series, but not at the end.
I tried to split this one into many commits, but I wasn't sure it was
worth to make a mini-series out of one function. In retrospect
it was.
Paolo
[Qemu-devel] [PATCH 10/25] qemu-char: ask and print error information from qemu-sockets, Paolo Bonzini, 2012/10/10
[Qemu-devel] [PATCH 11/25] nbd: ask and print error information from qemu-sockets, Paolo Bonzini, 2012/10/10
[Qemu-devel] [PATCH 16/25] qemu-sockets: add error propagation to inet_dgram_opts, Paolo Bonzini, 2012/10/10
[Qemu-devel] [PATCH 22/25] qemu-sockets: add socket_listen, socket_connect, socket_parse, Paolo Bonzini, 2012/10/10