From: Pavel Hrdina <phrdina@redhat.com> In case driver is null we would leak memory allocated in event. Fixes: b56458d443ba1d890d336535b5840252cbb9df4b Signed-off-by: Pavel Hrdina <phrdina@redhat.com> --- src/node_device/node_device_udev.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/node_device/node_device_udev.c b/src/node_device/node_device_udev.c index 89aca2d4d6..104433fb88 100644 --- a/src/node_device/node_device_udev.c +++ b/src/node_device/node_device_udev.c @@ -183,7 +183,7 @@ G_DEFINE_AUTOPTR_CLEANUP_FUNC(nodeDeviceEvent, nodeDeviceEventFree); */ static int nodeDeviceEventSubmit(nodeDeviceEventType eventType, void *data, virFreeCallback dataFreeFunc) { - nodeDeviceEvent *event = g_new0(nodeDeviceEvent, 1); + nodeDeviceEvent *event = NULL; udevEventData *priv = NULL; if (!driver) @@ -191,6 +191,7 @@ static int nodeDeviceEventSubmit(nodeDeviceEventType eventType, void *data, virF priv = driver->privateData; + event = g_new0(nodeDeviceEvent, 1); event->eventType = eventType; event->data = data; event->dataFreeFunc = dataFreeFunc; -- 2.53.0