diff options
author | Artur Wojcik <artur.wojcik@intel.com> | 2009-12-10 12:03:40 -0700 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2009-12-10 12:03:40 -0700 |
commit | e207da2f1b16470429d621ee47938e3215844caa (patch) | |
tree | 4050a4f437775ba093b8f52525e4826cc3db2495 /super-intel.c | |
parent | 4e5e717d72453bed761f561f24393d95d1817352 (diff) | |
download | mdadm-e207da2f1b16470429d621ee47938e3215844caa.tar.gz mdadm-e207da2f1b16470429d621ee47938e3215844caa.tar.xz mdadm-e207da2f1b16470429d621ee47938e3215844caa.zip |
Fix for memory leak defect.
Dynamic memory stored in 'devnum2devname(st->container_dev)' allocated
through function 'devnum2devname' at line 1274 is lost at line 1278.
Signed-off-by: Artur Wojcik <artur.wojcik@intel.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'super-intel.c')
-rw-r--r-- | super-intel.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/super-intel.c b/super-intel.c index 72fa68f..fbde7e6 100644 --- a/super-intel.c +++ b/super-intel.c @@ -1244,6 +1244,7 @@ static void getinfo_super_imsm_volume(struct supertype *st, struct mdinfo *info) struct imsm_dev *dev = get_imsm_dev(super, super->current_vol); struct imsm_map *map = get_imsm_map(dev, 0); struct dl *dl; + char *devname; for (dl = super->disks; dl; dl = dl->next) if (dl->raiddisk == info->disk.raid_disk) @@ -1285,9 +1286,11 @@ static void getinfo_super_imsm_volume(struct supertype *st, struct mdinfo *info) info->array.major_version = -1; info->array.minor_version = -2; - sprintf(info->text_version, "/%s/%d", - devnum2devname(st->container_dev), - info->container_member); + devname = devnum2devname(st->container_dev); + *info->text_version = '\0'; + if (devname) + sprintf(info->text_version, "/%s/%d", devname, info->container_member); + free(devname); info->safe_mode_delay = 4000; /* 4 secs like the Matrix driver */ uuid_from_super_imsm(st, info->uuid); } |