[libvirt] [PATCH 3/3] qemu_capabilities: Resolve Coverity RESOURCE_LEAK
Wang Rui
moon.wangrui at huawei.com
Thu Aug 28 10:20:58 UTC 2014
In function virQEMUCapsParseMachineTypesStr, VIR_STRNDUP allocates
memory for 'name' in {do,while} loop. If 'name' isn't freed before
'continue', its memory will be allocated again in the next loop.
In this case the memory allocated for 'name' in privious loop is
useless and not freed. Free it before continue this loop to fix that.
Signed-off-by: Wang Rui <moon.wangrui at huawei.com>
---
src/qemu/qemu_capabilities.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
index ce899f2..4a540ee 100644
--- a/src/qemu/qemu_capabilities.c
+++ b/src/qemu/qemu_capabilities.c
@@ -433,8 +433,10 @@ virQEMUCapsParseMachineTypesStr(const char *output,
if ((t = strstr(p, "(alias of ")) && (!next || t < next)) {
p = t + strlen("(alias of ");
- if (!(t = strchr(p, ')')) || (next && t >= next))
+ if (!(t = strchr(p, ')')) || (next && t >= next)) {
+ VIR_FREE(name);
continue;
+ }
if (VIR_STRNDUP(canonical, p, t - p) < 0) {
VIR_FREE(name);
--
1.7.12.4
More information about the libvir-list
mailing list