[libvirt] [PATCH 3/5] virfile: Avoid Coverity IDENTICAL_BRANCHES error
In virFileNBDDeviceFindUnused if virFileNBDDeviceIsBusy returns 0,
then both branches jumped to cleanup, so just use ignore_value
since the function returns NULL or some memory and the caller
handles the error.
Signed-off-by: John Ferlan <jferlan(a)redhat.com>
---
src/util/virfile.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/util/virfile.c b/src/util/virfile.c
index e3c00ef..408d2d9 100644
--- a/src/util/virfile.c
+++ b/src/util/virfile.c
@@ -797,8 +797,7 @@ virFileNBDDeviceFindUnused(void)
if (rv < 0)
goto cleanup;
if (rv == 0) {
- if (virAsprintf(&ret, "/dev/%s", de->d_name) < 0)
- goto cleanup;
+ ignore_value(virAsprintf(&ret, "/dev/%s", de->d_name));
goto cleanup;
}
}
--
2.1.0