[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]
[libvir] [PATCH] Double free in openvzGetVPSInfo
- From: "Anton Protopopov" <aspsk2 gmail com>
- To: libvir-list redhat com
- Subject: [libvir] [PATCH] Double free in openvzGetVPSInfo
- Date: Thu, 21 Feb 2008 18:07:48 +0300
Hi,
without this openvzGetVPSInfo() will cause "double free" when parse bad (i.e., really bad or empty) output from vzlist.
diff --git a/src/openvz_conf.c b/src/openvz_conf.c
index 482de82..0bc7084 100644
--- a/src/openvz_conf.c
+++ b/src/openvz_conf.c
@@ -536,7 +536,6 @@ openvzGetVPSInfo(virConnectPtr conn) {
if (fscanf(fp, "%d %s\n", &veid, status) != 2) {
error(conn, VIR_ERR_INTERNAL_ERROR,
"Failed to parse vzlist output");
- free(*pnext);
goto error;
}
if(strcmp(status, "stopped")) {
@@ -557,7 +556,6 @@ openvzGetVPSInfo(virConnectPtr conn) {
vmdef = calloc(1, sizeof(*vmdef));
if(!vmdef) {
error(conn, VIR_ERR_INTERNAL_ERROR, "calloc failed");
- free(*pnext);
goto error;
}
@@ -568,7 +566,6 @@ openvzGetVPSInfo(virConnectPtr conn) {
if(ret == -1) {
error(conn, VIR_ERR_INTERNAL_ERROR,
"UUID in config file malformed");
- free(*pnext);
free(vmdef);
goto error;
}
[Date Prev][Date Next] [Thread Prev][Thread Next]
[Thread Index]
[Date Index]
[Author Index]