From 7e318dad427c7d2e6787f9412cf7c4ee761faf9f Mon Sep 17 00:00:00 2001 From: Zdenek Kabelac Date: Thu, 16 Aug 2012 20:12:38 +0200 Subject: dmsetup: fix memleak in _get_split_name Free allocated buffer in _get_split_name error path. --- WHATS_NEW_DM | 4 ++++ tools/dmsetup.c | 4 +++- 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/WHATS_NEW_DM b/WHATS_NEW_DM index 3890ea4f..186a2e15 100644 --- a/WHATS_NEW_DM +++ b/WHATS_NEW_DM @@ -1,3 +1,7 @@ +Version 1.02.77 - +================================= + Fix memory leak in dmsetup _get_split_name() error path. + Version 1.02.76 - 7th August 2012 ================================= Add dm_vasprintf to libdevmapper. diff --git a/tools/dmsetup.c b/tools/dmsetup.c index 5422f0b3..7a632c74 100644 --- a/tools/dmsetup.c +++ b/tools/dmsetup.c @@ -380,8 +380,10 @@ static struct dm_split_name *_get_split_name(const char *uuid, const char *name, return NULL; } - if (!(split_name->subsystem = _extract_uuid_prefix(uuid, separator))) + if (!(split_name->subsystem = _extract_uuid_prefix(uuid, separator))) { + dm_free(split_name); return_NULL; + } split_name->vg_name = split_name->lv_name = split_name->lv_layer = (char *) ""; -- cgit