There are two variables that can be freed automatically: @cmd
(which allows us to drop explicit virCommandFree() call at the
end of the function) and @help which was never freed (and thus
leaked).
Signed-off-by: Michal Privoznik <mprivozn(a)redhat.com>
---
src/ch/ch_conf.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/ch/ch_conf.c b/src/ch/ch_conf.c
index b2812de7ad..c67c815d45 100644
--- a/src/ch/ch_conf.c
+++ b/src/ch/ch_conf.c
@@ -196,10 +196,10 @@ chExtractVersion(virCHDriver *driver)
{
int ret = -1;
unsigned long version;
- char *help = NULL;
+ g_autofree char *help = NULL;
char *tmp = NULL;
g_autofree char *ch_cmd = g_find_program_in_path(CH_CMD);
- virCommand *cmd = NULL;
+ g_autoptr(virCommand) cmd = NULL;
if (!ch_cmd)
return -2;
@@ -236,6 +236,5 @@ chExtractVersion(virCHDriver *driver)
ret = 0;
cleanup:
- virCommandFree(cmd);
return ret;
}
--
2.31.1