diff options
| author | Antonio Borneo <borneo.antonio@gmail.com> | 2020-05-25 21:37:12 +0200 |
|---|---|---|
| committer | Antonio Borneo <borneo.antonio@gmail.com> | 2020-06-06 18:04:57 +0100 |
| commit | f6b4079ce5793f86b2c26c6bfcc8c717e2c8a3c3 (patch) | |
| tree | d590f047727a2903a714592beff3f24f47a5cac7 /src | |
| parent | 37330f89d789e5b0a74aa14f7a7bfcfb4260abff (diff) | |
| download | riscv-openocd-f6b4079ce5793f86b2c26c6bfcc8c717e2c8a3c3.zip riscv-openocd-f6b4079ce5793f86b2c26c6bfcc8c717e2c8a3c3.tar.gz riscv-openocd-f6b4079ce5793f86b2c26c6bfcc8c717e2c8a3c3.tar.bz2 | |
libusb_helper: fix memory leak when no adapter is found
When jtag_libusb_open() fails to find the adapter, it returns
error but left libusb initialized causing memory leak of libusb
internal data.
Issue found with valgrind when no adapter or board is connected to
the host, e.g.
valgrind openocd -f board/st_nucleo_f4.cfg
Close the libusb operations if jtag_libusb_open() has to return
error.
Change-Id: Ieb2f110be15705dafe80c099e7d83c07056c2a41
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: http://openocd.zylin.com/5701
Tested-by: jenkins
Diffstat (limited to 'src')
| -rw-r--r-- | src/jtag/drivers/libusb_helper.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/jtag/drivers/libusb_helper.c b/src/jtag/drivers/libusb_helper.c index fbbfb41..184882a 100644 --- a/src/jtag/drivers/libusb_helper.c +++ b/src/jtag/drivers/libusb_helper.c @@ -208,6 +208,9 @@ int jtag_libusb_open(const uint16_t vids[], const uint16_t pids[], if (serial_mismatch) LOG_INFO("No device matches the serial string"); + if (retval != ERROR_OK) + libusb_exit(jtag_libusb_context); + return retval; } |