jtag/tcl: fix memory leak in command 'irscan'
authorAntonio Borneo <borneo.antonio@gmail.com>
Thu, 21 May 2020 14:03:17 +0000 (16:03 +0200)
committerAntonio Borneo <borneo.antonio@gmail.com>
Sat, 6 Jun 2020 17:05:46 +0000 (18:05 +0100)
commitbd425de3fbb9ba73d4e24573e2b2262ba1b8a3f5
tree9081243ed77313fb18cb808c0f2077a70598b676
parent6f88aa0fb3bb7a91b5327b75e8fb772ed6d3be2d
jtag/tcl: fix memory leak in command 'irscan'

If the function parse_u64() fails, we jump to return, thus leaking
the memory just allocated in 'v'.
Issue identified by clang.

Move earlier the call to parse_u64() and the associated test,
before memory allocation.

While there, fix a possible NULL pointer dereferencing in case the
calloc() fails, by testing for allocation failure.

Change-Id: I6a77ee17aceb282bbdfefe7cdafeba2e0e7012f1
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: http://openocd.zylin.com/5692
Tested-by: jenkins
Reviewed-by: Tarek BOCHKATI <tarek.bouchkati@gmail.com>
src/jtag/tcl.c