2007-08-18 Michael Snyder <msnyder@access-company.com>

* event-loop.c (delete_async_signal_handler): Move pointer null test to before pointer dereference.
19 years ago · 32107cd542
2 changed files with 4 additions and 1 deletions
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@ -1,5 +1,8 @@
 2007-08-18  Michael Snyder  <msnyder@svkmacdonelllnx>

+	* event-loop.c (delete_async_signal_handler): Move pointer null
+	test to before pointer dereference.
+
 	* ui-out.c (append_header_to_list): Possible cut and paste error.

 	* MAINTAINERS: white space tweak.
--- a/gdb/event-loop.c
+++ b/gdb/event-loop.c
@ -950,7 +950,7 @@ delete_async_signal_handler (async_signal_handler ** async_handler_ptr)
  else
    {
      prev_ptr = sighandler_list.first_handler;
-      while (prev_ptr->next_handler != (*async_handler_ptr) && prev_ptr)
+      while (prev_ptr && prev_ptr->next_handler != (*async_handler_ptr))
 	prev_ptr = prev_ptr->next_handler;
      prev_ptr->next_handler = (*async_handler_ptr)->next_handler;
      if (sighandler_list.last_handler == (*async_handler_ptr))