]> git.proxmox.com Git - mirror_qemu.git/commit
qga/vss-win32: Document the DLL requires non-null errp
authorMarkus Armbruster <armbru@redhat.com>
Sat, 20 Jun 2015 07:33:56 +0000 (09:33 +0200)
committerMarkus Armbruster <armbru@redhat.com>
Thu, 10 Sep 2015 11:48:06 +0000 (13:48 +0200)
commit08e64640357cd9517aa30fd49840f05f0f2ee3a4
tree1d4de3c1c8df39ca0d1d20d65eecdfa8a5d6dfd8
parente7cf59e84767e30b507b6bd7c1347072ec12b636
qga/vss-win32: Document the DLL requires non-null errp

requester.cpp uses this pattern to receive an error and pass it on to
the caller (err_is_set() macro peeled off for clarity):

    ... code that may set errset->errp ...
    if (errset->errp && *errset->errp) {
        ... handle error ...
    }

This breaks when errset->errp is null.  As far as I can tell, it
currently isn't, so this is merely fragile, not actually broken.

The robust way to do this is to receive the error in a local variable,
then propagate it up, like this:

    Error *err = NULL;

    ... code that may set err ...
    if (err)
        ... handle error ...
        error_propagate(errset->errp, err);
    }

See also commit 5e547690f230bfa903f40.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
qga/vss-win32.c
qga/vss-win32/requester.cpp
qga/vss-win32/requester.h