diff options
author | Eduardo Habkost <ehabkost@redhat.com> | 2016-06-13 18:57:56 -0300 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2016-06-20 16:38:13 +0200 |
commit | 621ff94d5074d88253a5818c6b9c4db718fbfc65 (patch) | |
tree | 471a14faadcea9b57a834801610e2079264e4920 /qga | |
parent | da002526ac30f679b0c797b0d71102dcdf200fe6 (diff) | |
download | qemu-621ff94d5074d88253a5818c6b9c4db718fbfc65.zip qemu-621ff94d5074d88253a5818c6b9c4db718fbfc65.tar.gz qemu-621ff94d5074d88253a5818c6b9c4db718fbfc65.tar.bz2 |
error: Remove NULL checks on error_propagate() calls
error_propagate() already ignores local_err==NULL, so there's no
need to check it before calling.
Coccinelle patch used to perform the changes added to
scripts/coccinelle/error_propagate_null.cocci.
Reviewed-by: Eric Blake <eblake@redhat.com>
Acked-by: Cornelia Huck <cornelia.huck@de.ibm.com>
Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
Message-Id: <1465855078-19435-2-git-send-email-ehabkost@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'qga')
-rw-r--r-- | qga/commands-win32.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/qga/commands-win32.c b/qga/commands-win32.c index c1a8588..ea23797 100644 --- a/qga/commands-win32.c +++ b/qga/commands-win32.c @@ -247,9 +247,7 @@ out: if (token) { CloseHandle(token); } - if (local_err) { - error_propagate(errp, local_err); - } + error_propagate(errp, local_err); } static void execute_async(DWORD WINAPI (*func)(LPVOID), LPVOID opaque, @@ -882,9 +880,7 @@ static void check_suspend_mode(GuestSuspendMode mode, Error **errp) } out: - if (local_err) { - error_propagate(errp, local_err); - } + error_propagate(errp, local_err); } static DWORD WINAPI do_suspend(LPVOID opaque) |