From: OpenOCD-Gerrit <ope...@us...> - 2020-04-20 17:26:53
|
This is an automated email from the git hooks/post-receive script. It was generated because a ref change was pushed to the repository containing the project "Main OpenOCD repository". The branch, master has been updated via c0644401622d1d5bf42e522452c7c9f83293cfd8 (commit) from 46f077aa003449f95781170fc2d0cf674272ad3e (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email; so we list those revisions in full, below. - Log ----------------------------------------------------------------- commit c0644401622d1d5bf42e522452c7c9f83293cfd8 Author: Tomas Vanek <va...@fb...> Date: Fri Dec 20 23:34:19 2019 +0100 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 <va...@fb...> Reviewed-on: http://openocd.zylin.com/5519 Tested-by: jenkins Reviewed-by: Antonio Borneo <bor...@gm...> Reviewed-by: Oleksij Rempel <li...@re...> diff --git a/src/jtag/drivers/ulink.c b/src/jtag/drivers/ulink.c index 77fbe6193..9235eb893 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; } } ----------------------------------------------------------------------- Summary of changes: src/jtag/drivers/ulink.c | 1 + 1 file changed, 1 insertion(+) hooks/post-receive -- Main OpenOCD repository |