From 0b97ce28d67c0548819b28d1e42b5b06b8da138c Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Wed, 31 Aug 2011 08:23:05 +0000 Subject: [PATCH] Fix resource leak when strdup fails Static analyzer noticed, strdup failing path leaks dmt structure. --- daemons/dmeventd/dmeventd.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/daemons/dmeventd/dmeventd.c b/daemons/dmeventd/dmeventd.c index ec514ee98..791c47dd9 100644 --- a/daemons/dmeventd/dmeventd.c +++ b/daemons/dmeventd/dmeventd.c @@ -751,8 +751,10 @@ static struct dm_task *_get_device_status(struct thread_status *ts) if (!dmt) return NULL; - if (!dm_task_set_uuid(dmt, ts->device.uuid)) - return NULL; + if (!dm_task_set_uuid(dmt, ts->device.uuid)) { + dm_task_destroy(dmt); + return NULL; + } if (!dm_task_run(dmt)) { dm_task_destroy(dmt); -- 2.43.5