[libvirt] [PATCH 3/5] virfile: Avoid Coverity IDENTICAL_BRANCHES error
Laine Stump
laine at laine.org
Fri Sep 4 17:43:24 UTC 2015
On 09/04/2015 10:31 AM, John Ferlan wrote:
> 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 at 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;
> }
> }
ACK.
More information about the libvir-list
mailing list