diff options
| author | Daniel P. Berrange <berrange@redhat.com> | 2014-10-27 16:38:06 +0000 |
|---|---|---|
| committer | Daniel P. Berrange <berrange@redhat.com> | 2014-10-27 16:38:06 +0000 |
| commit | bd6fd964f9ebf8c0bf4fd3cd6413eb85b3ab16a9 (patch) | |
| tree | 5b0e97720efb8d35decdfa30d71708c5b19e694a /src/remote-viewer.c | |
| parent | f0d662994fcc22c5dcb97cb0fb68c1c213171d20 (diff) | |
| download | virt-viewer-bd6fd964f9ebf8c0bf4fd3cd6413eb85b3ab16a9.tar.gz virt-viewer-bd6fd964f9ebf8c0bf4fd3cd6413eb85b3ab16a9.tar.xz virt-viewer-bd6fd964f9ebf8c0bf4fd3cd6413eb85b3ab16a9.zip | |
Remove useless 'if (...)' before free()
The 'free()' function accepts NULL, so you should not
check for NULL before calling it.
Diffstat (limited to 'src/remote-viewer.c')
| -rw-r--r-- | src/remote-viewer.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/remote-viewer.c b/src/remote-viewer.c index 112ef8b..7030c3e 100644 --- a/src/remote-viewer.c +++ b/src/remote-viewer.c @@ -1148,9 +1148,7 @@ choose_vm(char **vm_name, OvirtCollection *vms_collection, GError **error) OvirtVm *vm; g_return_val_if_fail(vm_name != NULL, NULL); - if (*vm_name != NULL) { - free(*vm_name); - } + free(*vm_name); model = gtk_list_store_new(1, G_TYPE_STRING); |