From bb673117d5823030a47dda70712bafbf62fe302a Mon Sep 17 00:00:00 2001 From: Dmitry Frolov Date: Tue, 10 Oct 2023 11:54:14 +0300 Subject: [PATCH] util: vircommand: fix redundant if Comparison "if (ret == -1)" is always false. This statement was forgotten during switching to g_new0() Found by Linux Verification Center (linuxtesting.org) with SVACE. Fixes: 0275b06a55fc7b1ec6a9e93f7fb73bea7388f634 ("util: command: use g_new0") Signed-off-by: Dmitry Frolov Reviewed-by: Martin Kletzander --- src/util/vircommand.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/src/util/vircommand.c b/src/util/vircommand.c index 9d4ba1e97c..81e74deee0 100644 --- a/src/util/vircommand.c +++ b/src/util/vircommand.c @@ -2227,7 +2227,7 @@ virCommandProcessIO(virCommand *cmd) int outfd = -1, errfd = -1; size_t inlen = 0, outlen = 0, errlen = 0; size_t inoff = 0; - int ret = 0; + int ret = -1; g_autofree struct pollfd *fds = NULL; if (dryRunBuffer || dryRunCallback) { @@ -2254,9 +2254,6 @@ virCommandProcessIO(virCommand *cmd) VIR_FREE(*cmd->errbuf); *cmd->errbuf = g_new0(char, 1); } - if (ret == -1) - goto cleanup; - ret = -1; fds = g_new0(struct pollfd, 3 + virCommandGetNumSendBuffers(cmd));