This is an automated email from Gerrit.

Tomas Vanek (van...@fbl.cz) just uploaded a new patch set to Gerrit, which you 
can find at http://openocd.zylin.com/5519

-- gerrit

commit 287a6d4b78ada828ffd12b1080925f51ca707a71
Author: Tomas Vanek <van...@fbl.cz>
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 <van...@fbl.cz>

diff --git a/src/jtag/drivers/ulink.c b/src/jtag/drivers/ulink.c
index 77fbe61..9235eb8 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;
                }
        }

-- 


_______________________________________________
OpenOCD-devel mailing list
OpenOCD-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to