libusb: fix a memory leak in sunos_new_string_list func

In sunos_new_string_list func, if alloc list->string fails,
we will return NULL without free list, which has alread been
allocated successfully.

Fixes: 17348731b4 ('Solaris backend depends on Solaris private symbols')
Closes #756

Signed-off-by: Zhiqiang Liu <lzhq28@mail.ustc.edu.cn>
Signed-off-by: Chris Dickens <christopher.a.dickens@gmail.com>

diff --git a/libusb/os/sunos_usb.c b/libusb/os/sunos_usb.c
index 3e2fb4fdbaa9f07d2ccdfa74ca4db6b40086c6d1..73b96dafb63eb1c9251b2e54045268eb2a5db0ef 100644 (file)
--- a/libusb/os/sunos_usb.c
+++ b/libusb/os/sunos_usb.c
@@ -248,8 +248,10 @@ sunos_new_string_list(void)
        if (list == NULL)
                return (NULL);
        list->string = calloc(DEFAULT_LISTSIZE, sizeof(char *));
-       if (list->string == NULL)
+       if (list->string == NULL) {
+               free(list);
                return (NULL);
+       }
        list->nargs = 0;
        list->listsize = DEFAULT_LISTSIZE;
 

diff --git a/libusb/version_nano.h b/libusb/version_nano.h
index ea0c9713344e1239bccbfede0a522212c5d9a621..baac006cb82bbfbe5514782b4ac9ba3b30fc6173 100644 (file)
--- a/libusb/version_nano.h
+++ b/libusb/version_nano.h
@@ -1 +1 @@
-#define LIBUSB_NANO 11525
+#define LIBUSB_NANO 11526