[libvirt] [PATCH] qemu_monitor_json.c: avoid many unconditional leaks

Jim Meyering jim at meyering.net
Wed Jan 27 09:38:55 UTC 2010


Actually, the preceding patch fixed only the one leak that had been
introduced in the last month or so.  Looking at the many other functions
that do the same sort of thing (call qemuMonitorJSONMakeCommand, and
later virJSONValueFree), I saw that they all had exactly the same leak.
So this amended patch fixes all of them:

>From 28f820354dcae9950cad042ea78a893fd9475830 Mon Sep 17 00:00:00 2001
From: Jim Meyering <meyering at redhat.com>
Date: Wed, 27 Jan 2010 09:58:12 +0100
Subject: [PATCH] qemu_monitor_json.c: avoid many unconditional leaks

* src/qemu/qemu_monitor_json.c (qemuMonitorJSONAttachDrive):
Don't leak the buffer behind a virJSONValuePtr.
(qemuMonitorJSONStartCPUs): Likewise.
(qemuMonitorJSONStopCPUs, qemuMonitorJSONSystemPowerdown): Likewise.
(qemuMonitorJSONGetCPUInfo, qemuMonitorJSONGetBalloonInfo): Likewise.
(qemuMonitorJSONGetBlockStatsInfo): Likewise.
(qemuMonitorJSONSetVNCPassword): Likewise.
(qemuMonitorJSONSetBalloon, qemuMonitorJSONEjectMedia): Likewise.
(qemuMonitorJSONChangeMedia, qemuMonitorJSONSaveMemory): Likewise.
(qemuMonitorJSONSetMigrationSpeed): Likewise.
(qemuMonitorJSONGetMigrationStatus, qemuMonitorJSONMigrate): Likewise.
(qemuMonitorJSONMigrateCancel, qemuMonitorJSONAddUSB): Likewise.
(qemuMonitorJSONAddPCIHostDevice, qemuMonitorJSONAddPCIDisk): Likewise.
(qemuMonitorJSONAddPCINetwork, qemuMonitorJSONRemovePCIDevice): Likewise.
(qemuMonitorJSONSendFileHandle): Likewise.
(qemuMonitorJSONCloseFileHandle): Likewise.
(qemuMonitorJSONAddHostNetwork): Likewise.
(qemuMonitorJSONRemoveHostNetwork): Likewise.
(qemuMonitorJSONGetPtyPaths): Likewise.
(qemuMonitorJSONAttachPCIDiskController): Likewise.
---
 src/qemu/qemu_monitor_json.c |   27 +++++++++++++++++++++++++++
 1 files changed, 27 insertions(+), 0 deletions(-)

diff --git a/src/qemu/qemu_monitor_json.c b/src/qemu/qemu_monitor_json.c
index 8e88c7e..b6c3449 100644
--- a/src/qemu/qemu_monitor_json.c
+++ b/src/qemu/qemu_monitor_json.c
@@ -492,6 +492,7 @@ qemuMonitorJSONStartCPUs(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -512,6 +513,7 @@ qemuMonitorJSONStopCPUs(qemuMonitorPtr mon)
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -531,6 +533,7 @@ int qemuMonitorJSONSystemPowerdown(qemuMonitorPtr mon)
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -637,6 +640,7 @@ int qemuMonitorJSONGetCPUInfo(qemuMonitorPtr mon,
         ret = qemuMonitorJSONExtractCPUInfo(reply, pids);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -696,6 +700,7 @@ int qemuMonitorJSONGetBalloonInfo(qemuMonitorPtr mon,

 cleanup:
     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -800,6 +805,7 @@ int qemuMonitorJSONGetBlockStatsInfo(qemuMonitorPtr mon,

 cleanup:
     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -824,6 +830,7 @@ int qemuMonitorJSONSetVNCPassword(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -861,6 +868,7 @@ int qemuMonitorJSONSetBalloon(qemuMonitorPtr mon,

 cleanup:
     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -884,6 +892,7 @@ int qemuMonitorJSONEjectMedia(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -918,6 +927,7 @@ int qemuMonitorJSONChangeMedia(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -945,6 +955,7 @@ static int qemuMonitorJSONSaveMemory(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -991,6 +1002,7 @@ int qemuMonitorJSONSetMigrationSpeed(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1084,6 +1096,7 @@ int qemuMonitorJSONGetMigrationStatus(qemuMonitorPtr mon,
         ret = -1;

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1109,6 +1122,7 @@ static int qemuMonitorJSONMigrate(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1206,6 +1220,7 @@ int qemuMonitorJSONMigrateCancel(qemuMonitorPtr mon)
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1229,6 +1244,7 @@ static int qemuMonitorJSONAddUSB(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1369,6 +1385,7 @@ int qemuMonitorJSONAddPCIHostDevice(qemuMonitorPtr mon,
         ret = -1;

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1410,6 +1427,7 @@ int qemuMonitorJSONAddPCIDisk(qemuMonitorPtr mon,
         ret = -1;

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1442,6 +1460,7 @@ int qemuMonitorJSONAddPCINetwork(qemuMonitorPtr mon,
         ret = -1;

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1475,6 +1494,7 @@ int qemuMonitorJSONRemovePCIDevice(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1498,6 +1518,7 @@ int qemuMonitorJSONSendFileHandle(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1520,6 +1541,7 @@ int qemuMonitorJSONCloseFileHandle(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1542,6 +1564,7 @@ int qemuMonitorJSONAddHostNetwork(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1566,6 +1589,7 @@ int qemuMonitorJSONRemoveHostNetwork(qemuMonitorPtr mon,
         ret = qemuMonitorJSONCheckError(cmd, reply);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1665,6 +1689,7 @@ int qemuMonitorJSONGetPtyPaths(qemuMonitorPtr mon,
         ret = qemuMonitorJSONExtractPtyPaths(reply, paths);

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1705,6 +1730,7 @@ int qemuMonitorJSONAttachPCIDiskController(qemuMonitorPtr mon,
         ret = -1;

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
@@ -1773,6 +1799,7 @@ int qemuMonitorJSONAttachDrive(qemuMonitorPtr mon,
         ret = -1;

     virJSONValueFree(cmd);
+    VIR_FREE(cmd);
     virJSONValueFree(reply);
     return ret;
 }
--
1.7.0.rc0.140.gfbe7




More information about the libvir-list mailing list