External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-12-22 13:45:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

virObjectEventNew: Use virObjectUnref() to free virObjectEvent

Browse Source 
While no leak was observed yet, there might be one if
virObjectEventClass is ever derived from another class. Because
in that case plain VIR_FREE() will not call dispose() from parent
classes possibly leaking some memory.

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Michal Privoznik 2016-08-04 08:33:17 +02:00
parent b3df357da6
commit b660238eb1
1 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/conf/object_event.c
View File

@ -649,13 +649,9 @@ virObjectEventNew(virClassPtr klass,
event->eventID = eventID;
event->remoteID = -1;
if (VIR_STRDUP(event->meta.name, name) < 0) {
VIR_FREE(event);
return NULL;
}
if (VIR_STRDUP(event->meta.key, key) < 0) {
VIR_FREE(event->meta.name);
VIR_FREE(event);
if (VIR_STRDUP(event->meta.name, name) < 0 ||
VIR_STRDUP(event->meta.key, key) < 0) {
virObjectUnref(event);
return NULL;
}
event->meta.id = id;