interesting
/
riscv-openocd
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

rtos: Do not dereference null pointers 

detected by clang.

Change-Id: Id395f9d33bc2903b29a158fc3540ae51857e6aa0
Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-on: http://openocd.zylin.com/1555
Tested-by: jenkins
This commit is contained in:
Spencer Oliver 2013-08-19 14:42:53 +01:00
parent e475896eb1
commit e1d2abc9ca
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/rtos/linux.c
View File

@ -533,7 +533,7 @@ int get_current(struct target *target, int create)
LOG_ERROR LOG_ERROR
("error in linux current thread update"); ("error in linux current thread update");
if (create) { if (create && ct) {
struct threads *t; struct threads *t;
t = calloc(1, sizeof(struct threads)); t = calloc(1, sizeof(struct threads));
t->base_addr = ct->TS; t->base_addr = ct->TS;
@ -1140,13 +1140,12 @@ int linux_gdb_thread_packet(struct target *target,
char *tmp_str = out_str; char *tmp_str = out_str;
tmp_str += sprintf(tmp_str, "m"); tmp_str += sprintf(tmp_str, "m");
struct threads *temp = linux_os->thread_list; struct threads *temp = linux_os->thread_list;
tmp_str += sprintf(tmp_str, "%016" PRIx64, temp->threadid);
temp = temp->next;
while (temp != NULL) { while (temp != NULL) {
tmp_str += sprintf(tmp_str, ",");
tmp_str += sprintf(tmp_str, "%016" PRIx64, temp->threadid); tmp_str += sprintf(tmp_str, "%016" PRIx64, temp->threadid);
temp = temp->next; temp = temp->next;
if (temp)
tmp_str += sprintf(tmp_str, ",");
} }
gdb_put_packet(connection, out_str, strlen(out_str)); gdb_put_packet(connection, out_str, strlen(out_str));