openocd: avoid checking for non NULL pointer to free it
authorAntonio Borneo <borneo.antonio@gmail.com>
Sun, 16 Aug 2020 19:35:10 +0000 (21:35 +0200)
committerAntonio Borneo <borneo.antonio@gmail.com>
Sat, 5 Sep 2020 16:11:50 +0000 (17:11 +0100)
commit4e98d44fd1dc67f763f06eeecc0453d65b1290dc
treef70d92827b2c6855213a37e9f0b17b5b1fe09109
parent62329444abc89ad3b37fbb4ebc2edfd1dee23351
openocd: avoid checking for non NULL pointer to free it

The function free() can be called with a NULL pointer as argument,
no need to check the argument before. If the pointer is NULL, no
operation is performed by free().

Remove the occurrences of pattern:
if (ptr)
free(ptr);

While there replace a sequence malloc(size)+memset(,0,size) with a
calloc(1,size).
Replace a pointer assignment to '0' with an assignment to NULL.
In server/*, an error is logged if the ptr was already NULL. This
cannot happen since the pointer was already referenced few lines
before and openocd would have been already SIGSEGV in that case,
so remove the log.

Change-Id: I10822029fe8390b59edff4070575bf7f754e44ac
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: http://openocd.zylin.com/5808
Reviewed-by: Adrian M Negreanu <adrian.negreanu@nxp.com>
Tested-by: jenkins
src/helper/ioutil.c
src/rtos/chibios.c
src/rtos/linux.c
src/rtos/rtos.c
src/server/gdb_server.c
src/server/server.c
src/server/telnet_server.c
src/svf/svf.c
src/xsvf/xsvf.c