External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-01-24 21:45:21 +00:00
Code Issues Packages Projects Releases Wiki Activity

storage: scsi: invert logic in createVport

Browse Source 
Check whether the alloc result is negative (which is
cannot happen with current code) to reduce churn in
the following commit.

Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Peter Krempa <pkrempa@redhat.com>
This commit is contained in:
Ján Tomko 2020-09-24 20:55:24 +02:00
parent 6e9a2351d4
commit ff146d0953
1 changed files with 10 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/storage/storage_backend_scsi.c
View File

@ -329,7 +329,9 @@ createVport(virStoragePoolDefPtr def,
* retry logic set to true. If the thread isn't created, then no big * retry logic set to true. If the thread isn't created, then no big
* deal since it's still possible to refresh the pool later. * deal since it's still possible to refresh the pool later.
*/ */
if (VIR_ALLOC(cbdata) == 0) { if (VIR_ALLOC(cbdata) < 0)
return -1;
memcpy(cbdata->pool_uuid, def->uuid, VIR_UUID_BUFLEN); memcpy(cbdata->pool_uuid, def->uuid, VIR_UUID_BUFLEN);
cbdata->fchost_name = g_steal_pointer(&name); cbdata->fchost_name = g_steal_pointer(&name);
@ -339,7 +341,6 @@ createVport(virStoragePoolDefPtr def,
VIR_DEBUG("Failed to create FC Pool Refresh Thread"); VIR_DEBUG("Failed to create FC Pool Refresh Thread");
virStoragePoolFCRefreshDataFree(cbdata); virStoragePoolFCRefreshDataFree(cbdata);
} }
}
return 0; return 0;
} }