On a Thursday in 2020, Matt Coleman wrote:
Signed-off-by: Matt Coleman <matt(a)datto.com>
---
src/conf/domain_conf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index 25baa44a92..d71bd68682 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -31972,7 +31972,7 @@ virDomainDiskSetBlockIOTune(virDomainDiskDefPtr disk,
tmp_group = g_strdup(info->group_name);
The tmp_group variable is not needed here. It's an artifact from back
when strdup could fail, to avoid the situation of freeing the old group
name without being able to copy the new one.
Jano
- VIR_FREE(disk->blkdeviotune.group_name);
+ g_free(disk->blkdeviotune.group_name);
disk->blkdeviotune = *info;
disk->blkdeviotune.group_name = g_steal_pointer(&tmp_group);
}
--
2.27.0