diff options
author | Markus Armbruster <armbru@redhat.com> | 2020-06-30 11:03:28 +0200 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2020-07-02 06:25:28 +0200 |
commit | 9261ef5e32b0559642ccb70565836e1bc023937e (patch) | |
tree | c05577c4c52758094879bad94e4b2af7b0a4302a /ui | |
parent | 5a79d10c953c9eebc69e4ad7c18cd8476782cf6f (diff) | |
download | qemu-9261ef5e32b0559642ccb70565836e1bc023937e.zip qemu-9261ef5e32b0559642ccb70565836e1bc023937e.tar.gz qemu-9261ef5e32b0559642ccb70565836e1bc023937e.tar.bz2 |
Clean up some calls to ignore Error objects the right way
Receiving the error in a local variable only to free it is less clear
(and also less efficient) than passing NULL. Clean up.
Cc: Daniel P. Berrange <berrange@redhat.com>
Cc: Jerome Forissier <jerome@forissier.org>
CC: Greg Kurz <groug@kaod.org>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Greg Kurz <groug@kaod.org>
Message-Id: <20200630090351.1247703-4-armbru@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Diffstat (limited to 'ui')
-rw-r--r-- | ui/vnc.c | 3 |
1 files changed, 1 insertions, 2 deletions
@@ -458,9 +458,8 @@ static VncServerInfo2List *qmp_query_server_entry(QIOChannelSocket *ioc, Error *err = NULL; SocketAddress *addr; - addr = qio_channel_socket_get_local_address(ioc, &err); + addr = qio_channel_socket_get_local_address(ioc, NULL); if (!addr) { - error_free(err); return prev; } |