Currently, ret is only used in the 'cleanup' section and initialized right before the jump. Switch to the customary initialization to -1 and only leave in the 'ret = 0' statement on an empty list. Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> --- src/interface/interface_backend_udev.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/interface/interface_backend_udev.c b/src/interface/interface_backend_udev.c index ce007fd29e..31246dde74 100644 --- a/src/interface/interface_backend_udev.c +++ b/src/interface/interface_backend_udev.c @@ -303,7 +303,7 @@ udevConnectListAllInterfaces(virConnectPtr conn, int tmp_count; int count = 0; int status = 0; - int ret; + int ret = -1; virCheckFlags(VIR_CONNECT_LIST_INTERFACES_FILTERS_ACTIVE, -1); @@ -320,7 +320,6 @@ udevConnectListAllInterfaces(virConnectPtr conn, virReportError(VIR_ERR_INTERNAL_ERROR, _("failed to get list of %s interfaces on host"), virUdevStatusString(status)); - ret = -1; goto cleanup; } -- 2.26.2