From c0644401622d1d5bf42e522452c7c9f83293cfd8 Mon Sep 17 00:00:00 2001 From: Tomas Vanek Date: Fri, 20 Dec 2019 23:34:19 +0100 Subject: [PATCH] jtag/drivers/ulink: fix clang static analyzer warning scan-build-9: Description: Potential leak of memory pointed to by 'tdo_buffer' File: src/jtag/drivers/ulink.c Line: 1629 Free the buffer before error return. Change-Id: Ic47651a5ae78c7a47ae4fcbad225f329b14c45cb Signed-off-by: Tomas Vanek Reviewed-on: http://openocd.zylin.com/5519 Tested-by: jenkins Reviewed-by: Antonio Borneo Reviewed-by: Oleksij Rempel --- src/jtag/drivers/ulink.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/jtag/drivers/ulink.c b/src/jtag/drivers/ulink.c index 77fbe61933..9235eb8935 100644 --- a/src/jtag/drivers/ulink.c +++ b/src/jtag/drivers/ulink.c @@ -1627,6 +1627,7 @@ int ulink_queue_scan(struct ulink *device, struct jtag_command *cmd) if (ret != ERROR_OK) { free(tdi_buffer_start); + free(tdo_buffer_start); return ret; } } -- 2.30.2