This is an automated email from Gerrit.

Andreas Fritiofson ([email protected]) just uploaded a new patch set 
to Gerrit, which you can find at http://openocd.zylin.com/1665

-- gerrit

commit 7227dbe59ab0aad7fc1bd1d20f4f4265114e7c80
Author: Andreas Fritiofson <[email protected]>
Date:   Sat Sep 28 16:07:14 2013 +0200

    jtag/tcl: Fix double free in jtag newtap argument parsing
    
    Found when grepping for void* casts. I'm pretty sure there are about
    *picking a number out of the blue* 0.31 bugs per pointer cast in OpenOCD.
    
    Verified by valgrind, i.e.:
    valgrind openocd -f interface/jtagkey.cfg -c "jtag newtap dummy cpu -irlen 
foo"
    
    Change-Id: Idc12e8f18036cc908fdb572828fa46563be14242
    Signed-off-by: Andreas Fritiofson <[email protected]>

diff --git a/src/jtag/tcl.c b/src/jtag/tcl.c
index d0020cd..227222e 100644
--- a/src/jtag/tcl.c
+++ b/src/jtag/tcl.c
@@ -467,7 +467,6 @@ static int jim_newtap_ir_param(Jim_Nvp *n, Jim_GetOptInfo 
*goi,
        if (e != JIM_OK) {
                Jim_SetResultFormatted(goi->interp,
                        "option: %s bad parameter", n->name);
-               free((void *)pTap->dotted_name);
                return e;
        }
        switch (n->value) {

-- 

------------------------------------------------------------------------------
October Webinars: Code for Performance
Free Intel webinars can help you accelerate application performance.
Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from 
the latest Intel processors and coprocessors. See abstracts and register >
http://pubads.g.doubleclick.net/gampad/clk?id=60133471&iu=/4140/ostg.clktrk
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to