diff options
| author | Daniel Jacobowitz <dan@debian.org> | 2005-06-17 04:01:05 +0000 |
|---|---|---|
| committer | Daniel Jacobowitz <dan@debian.org> | 2005-06-17 04:01:05 +0000 |
| commit | a4d97b18be5c18c9a3a09aba8b3e9b448e0683ae (patch) | |
| tree | 918113a37ac5f7a175310bffc2409e6df8169523 /gdb/gdbserver | |
| parent | 3e11dd4d9ec6d346adaf43d8362c5871c0b51b84 (diff) | |
| download | gdb-a4d97b18be5c18c9a3a09aba8b3e9b448e0683ae.tar.gz | |
* linux-low.c (linux_wait, linux_send_signal): Don't test
an unsigned long variable for > 0 if it could be MAX_ULONG.
* server.c (myresume): Likewise.
* target.c (set_desired_inferior): Likewise.
Diffstat (limited to 'gdb/gdbserver')
| -rw-r--r-- | gdb/gdbserver/ChangeLog | 7 | ||||
| -rw-r--r-- | gdb/gdbserver/linux-low.c | 4 | ||||
| -rw-r--r-- | gdb/gdbserver/server.c | 4 | ||||
| -rw-r--r-- | gdb/gdbserver/target.c | 3 |
4 files changed, 13 insertions, 5 deletions
diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog index d5882214e1d..4c3e0fef6ff 100644 --- a/gdb/gdbserver/ChangeLog +++ b/gdb/gdbserver/ChangeLog @@ -1,3 +1,10 @@ +2005-06-16 Daniel Jacobowitz <dan@codesourcery.com> + + * linux-low.c (linux_wait, linux_send_signal): Don't test + an unsigned long variable for > 0 if it could be MAX_ULONG. + * server.c (myresume): Likewise. + * target.c (set_desired_inferior): Likewise. + 2005-06-13 Mark Kettenis <kettenis@gnu.org> * configure.ac: Simplify and improve check for socklen_t. diff --git a/gdb/gdbserver/linux-low.c b/gdb/gdbserver/linux-low.c index 9e94addd8ce..fbc3033f672 100644 --- a/gdb/gdbserver/linux-low.c +++ b/gdb/gdbserver/linux-low.c @@ -667,7 +667,7 @@ retry: then we need to make sure we restart the other threads. We could pick a thread at random or restart all; restarting all is less arbitrary. */ - if (cont_thread > 0) + if (cont_thread != 0 && cont_thread != -1) { child = (struct thread_info *) find_inferior_id (&all_threads, cont_thread); @@ -1435,7 +1435,7 @@ linux_send_signal (int signum) { extern unsigned long signal_pid; - if (cont_thread > 0) + if (cont_thread != 0 && cont_thread != -1) { struct process_info *process; diff --git a/gdb/gdbserver/server.c b/gdb/gdbserver/server.c index 55f7c2164c6..b5295bbabf2 100644 --- a/gdb/gdbserver/server.c +++ b/gdb/gdbserver/server.c @@ -281,7 +281,7 @@ myresume (int step, int sig) struct thread_resume resume_info[2]; int n = 0; - if (step || sig || cont_thread > 0) + if (step || sig || (cont_thread != 0 && cont_thread != -1)) { resume_info[0].thread = ((struct inferior_list_entry *) current_inferior)->id; @@ -293,7 +293,7 @@ myresume (int step, int sig) resume_info[n].thread = -1; resume_info[n].step = 0; resume_info[n].sig = 0; - resume_info[n].leave_stopped = (cont_thread > 0); + resume_info[n].leave_stopped = (cont_thread != 0 && cont_thread != -1); (*the_target->resume) (resume_info); } diff --git a/gdb/gdbserver/target.c b/gdb/gdbserver/target.c index 0de69e80249..b9222e2a3b5 100644 --- a/gdb/gdbserver/target.c +++ b/gdb/gdbserver/target.c @@ -42,7 +42,8 @@ set_desired_inferior (int use_general) /* If we are continuing any (all) thread(s), use step_thread to decide which thread to step and/or send the specified signal to. */ - if (step_thread > 0 && (cont_thread == 0 || cont_thread == -1)) + if ((step_thread != 0 && step_thread != -1) + && (cont_thread == 0 || cont_thread == -1)) found = (struct thread_info *) find_inferior_id (&all_threads, step_thread); |