From d592fa2f7f5a88b52b22c29bb5dd0ce9c49d998d Mon Sep 17 00:00:00 2001 From: Daniel Jacobowitz Date: Fri, 17 Jun 2005 04:01:05 +0000 Subject: [PATCH] * 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. --- gdb/gdbserver/ChangeLog | 7 +++++++ gdb/gdbserver/linux-low.c | 4 ++-- gdb/gdbserver/server.c | 4 ++-- 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 + + * 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 * 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); -- 2.30.2