External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-01-11 23:37:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

remote: Add missing virCondDestroy calls

Browse Source 
The virCond of the remote_thread_call struct was leaked in some
places. This results in leaking the underlying mutex. Which in turn
leaks a handle on Windows.

Reported by Aliaksandr Chabatar and Ihar Smertsin.
This commit is contained in:
Matthias Bolte 2011-03-17 17:51:33 +01:00
parent 80e6200f32
commit 55fb386671
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/remote/remote_driver.c
View File

@ -8609,6 +8609,7 @@ remoteStreamPacket(virStreamPtr st,
xdr_destroy (&xdr);
ret = remoteIO(st->conn, priv, 0, thiscall);
ignore_value(virCondDestroy(&thiscall->cond));
VIR_FREE(thiscall);
if (ret < 0)
return -1;
@ -8617,6 +8618,7 @@ remoteStreamPacket(virStreamPtr st,
error:
xdr_destroy (&xdr);
ignore_value(virCondDestroy(&thiscall->cond));
VIR_FREE(thiscall);
return -1;
}
@ -8746,6 +8748,7 @@ remoteStreamRecv(virStreamPtr st,
}
ret = remoteIO(st->conn, priv, 0, thiscall);
ignore_value(virCondDestroy(&thiscall->cond));
VIR_FREE(thiscall);
if (ret < 0)
goto cleanup;
@ -9719,6 +9722,7 @@ prepareCall(struct private_data *priv,
error:
xdr_destroy (&xdr);
ignore_value(virCondDestroy(&rv->cond));
VIR_FREE(rv);
return NULL;
}
@ -10818,6 +10822,7 @@ call (virConnectPtr conn, struct private_data *priv,
}
rv = remoteIO(conn, priv, flags, thiscall);
ignore_value(virCondDestroy(&thiscall->cond));
VIR_FREE(thiscall);
return rv;
}