[PATCH v2 4/6] Remove some dead code from handle_search_memory

Tom Tromey tromey@adacore.com
Fri Sep 25 19:49:11 GMT 2020


handle_search_memory had some code after a call to error.  This code
is dead, and this patch removes it.

gdbserver/ChangeLog
2020-09-25  Tom Tromey  <tromey@adacore.com>

	* server.cc (handle_search_memory): Remove dead code.
---
 gdbserver/ChangeLog | 4 ++++
 gdbserver/server.cc | 9 ++-------
 2 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/gdbserver/server.cc b/gdbserver/server.cc
index 1c21660027e..674462c9ae6 100644
--- a/gdbserver/server.cc
+++ b/gdbserver/server.cc
@@ -1054,11 +1054,8 @@ handle_search_memory (char *own_buf, int packet_len)
 
   pattern = (gdb_byte *) malloc (packet_len);
   if (pattern == NULL)
-    {
-      error ("Unable to allocate memory to perform the search");
-      strcpy (own_buf, "E00");
-      return;
-    }
+    error ("Unable to allocate memory to perform the search");
+
   if (decode_search_memory_packet (own_buf + cmd_name_len,
 				   packet_len - cmd_name_len,
 				   &start_addr, &search_space_len,
@@ -1066,8 +1063,6 @@ handle_search_memory (char *own_buf, int packet_len)
     {
       free (pattern);
       error ("Error in parsing qSearch:memory packet");
-      strcpy (own_buf, "E00");
-      return;
     }
 
   auto read_memory = [] (CORE_ADDR addr, gdb_byte *result, size_t len)
-- 
2.26.2



More information about the Gdb-patches mailing list