This is an automated email from Gerrit.

"zapb <[email protected]>" just uploaded a new patch set to Gerrit, which you can 
find at https://review.openocd.org/c/openocd/+/9735

-- gerrit

commit 8093eec19a49f9a5dc375cef1470764e415efb79
Author: Marc Schink <[email protected]>
Date:   Wed Jun 10 09:17:21 2026 +0200

    adapter/usbprog: Fix memory leak
    
    Add missing free() call to avoid a memory leak when jtag_read_buffer()
    fails.
    
    Change-Id: I8f025b2aaac4d71941a65b1a7ec37cfc56677017
    Signed-off-by: Marc Schink <[email protected]>

diff --git a/src/jtag/drivers/usbprog.c b/src/jtag/drivers/usbprog.c
index 4783e7569a..0762b006cc 100644
--- a/src/jtag/drivers/usbprog.c
+++ b/src/jtag/drivers/usbprog.c
@@ -124,8 +124,10 @@ static int usbprog_execute_queue(struct jtag_command 
*cmd_queue)
                        scan_size = jtag_build_buffer(cmd->cmd.scan, &buffer);
                        type = jtag_scan_type(cmd->cmd.scan);
                        usbprog_scan(cmd->cmd.scan->ir_scan, type, buffer, 
scan_size);
-                       if (jtag_read_buffer(buffer, cmd->cmd.scan) != ERROR_OK)
+                       if (jtag_read_buffer(buffer, cmd->cmd.scan) != 
ERROR_OK) {
+                               free(buffer);
                                return ERROR_JTAG_QUEUE_FAILED;
+                       }
                        free(buffer);
                        break;
                case JTAG_SLEEP:

-- 

Reply via email to