diff options
| author | Antonio Borneo <borneo.antonio@gmail.com> | 2021-09-04 23:01:09 +0200 |
|---|---|---|
| committer | Antonio Borneo <borneo.antonio@gmail.com> | 2021-09-18 15:22:30 +0000 |
| commit | 79800db98a985bcd601e8a892aed76d96548a51b (patch) | |
| tree | 2250fd09d5be8834fe9c8e0ec513134758ccd971 /src/server | |
| parent | ea562985b5eff536feea022b074122b21c3610ea (diff) | |
| download | riscv-openocd-79800db98a985bcd601e8a892aed76d96548a51b.zip riscv-openocd-79800db98a985bcd601e8a892aed76d96548a51b.tar.gz riscv-openocd-79800db98a985bcd601e8a892aed76d96548a51b.tar.bz2 | |
openocd: remove last NULL comparisons
The NULL pointers preceded by cast where not detected by the
scripting tools looking for NULL pointer comparison.
Remove them and, while there, further simplify the code and apply
the other coding style rules.
Change-Id: Ia7406122e07ef56ef311579ab0ee7ddb22c8e4b5
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/6539
Tested-by: jenkins
Reviewed-by: Tarek BOCHKATI <tarek.bouchkati@gmail.com>
Diffstat (limited to 'src/server')
| -rw-r--r-- | src/server/gdb_server.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c index 015baa1..a16b4cc 100644 --- a/src/server/gdb_server.c +++ b/src/server/gdb_server.c @@ -3006,8 +3006,10 @@ static bool gdb_handle_vrun_packet(struct connection *connection, const char *pa free(next_hex_encoded_field(&parse, ';')); char *cmdline = next_hex_encoded_field(&parse, ';'); - char *arg; - while (cmdline && (arg = next_hex_encoded_field(&parse, ';')) != NULL) { + while (cmdline) { + char *arg = next_hex_encoded_field(&parse, ';'); + if (!arg) + break; char *new_cmdline = alloc_printf("%s %s", cmdline, arg); free(cmdline); free(arg); @@ -3549,7 +3551,7 @@ static int gdb_target_start(struct target *target, const char *port) struct target_list *head; struct target *curr; head = target->head; - while (head != (struct target_list *)NULL) { + while (head) { curr = head->target; if (curr != target) curr->gdb_service = gdb_service; |