diff options
author | Lukas Nykryn <lnykryn@redhat.com> | 2011-10-25 09:37:44 +0200 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2012-01-11 15:34:39 -0800 |
commit | c04e55ccef320ecac31baaa0f646888fdd7c8f25 (patch) | |
tree | bf1c9aef3cdd57b2e753ca01bbd8c78f5a095a68 | |
parent | e8a312a0e8fda39ba8488e88807cdc3bae3ec20c (diff) | |
download | usbutils-c04e55ccef320ecac31baaa0f646888fdd7c8f25.tar.gz |
Possible resource leak
If (!pn || p == pn) is true function returns
and memory allocated few lines earlier will
be lost.
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r-- | lsusb-t.c | 3 |
1 files changed, 3 insertions, 0 deletions
@@ -319,7 +319,10 @@ static void add_usb_interface(const char *d_name) p = pn + 1; i = strtoul(p, &pn, 10); if (!pn || p == pn) + { + free(e); return; + } e->ifnum = i; if (snprintf(e->name, MY_SYSFS_FILENAME_LEN, "%s", d_name) >= MY_SYSFS_FILENAME_LEN) printf("warning: '%s' truncated to '%s'\n", e->name, d_name); |