summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLukas Nykryn <lnykryn@redhat.com>2011-10-25 09:37:44 +0200
committerGreg Kroah-Hartman <gregkh@suse.de>2012-01-11 15:34:39 -0800
commitc04e55ccef320ecac31baaa0f646888fdd7c8f25 (patch)
treebf1c9aef3cdd57b2e753ca01bbd8c78f5a095a68
parente8a312a0e8fda39ba8488e88807cdc3bae3ec20c (diff)
downloadusbutils-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.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/lsusb-t.c b/lsusb-t.c
index 7d9b0a8..a07c902 100644
--- a/lsusb-t.c
+++ b/lsusb-t.c
@@ -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);