From d6b9b80f9483b6c1a3a018c0fcaf813ca098d8af Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Sat, 9 Sep 2017 09:56:27 -0600 Subject: [PATCH] Remove cleanups from findcmd.c This removes cleanups from findcmd.c, replacing manual buffer management with a gdb::byte_vector. ChangeLog 2017-09-11 Tom Tromey * findcmd.c (put_bits): Take a gdb::byte_vector. (parse_find_args): Return gdb::byte_vector. "args" now const. Remove "pattern_bufp" and "pattern_lenp" parameters. Remove cleanups. (find_command): Update. --- gdb/ChangeLog | 8 ++++++ gdb/findcmd.c | 91 ++++++++++++++++++----------------------------------------- 2 files changed, 35 insertions(+), 64 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index a0dfb78..805204c 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,13 @@ 2017-09-11 Tom Tromey + * findcmd.c (put_bits): Take a gdb::byte_vector. + (parse_find_args): Return gdb::byte_vector. "args" now const. + Remove "pattern_bufp" and "pattern_lenp" parameters. Remove + cleanups. + (find_command): Update. + +2017-09-11 Tom Tromey + * cli/cli-script.c (class scoped_restore_hook_in): New. (clear_hook_in_cleanup): Remove. (execute_cmd_pre_hook, execute_cmd_post_hook): Use diff --git a/gdb/findcmd.c b/gdb/findcmd.c index f905e70..5eba2dc 100644 --- a/gdb/findcmd.c +++ b/gdb/findcmd.c @@ -25,11 +25,12 @@ #include "target.h" #include "cli/cli-utils.h" #include +#include "common/byte-vector.h" /* Copied from bfd_put_bits. */ static void -put_bits (bfd_uint64_t data, gdb_byte *buf, int bits, bfd_boolean big_p) +put_bits (bfd_uint64_t data, gdb::byte_vector &buf, int bits, bfd_boolean big_p) { int i; int bytes; @@ -37,11 +38,13 @@ put_bits (bfd_uint64_t data, gdb_byte *buf, int bits, bfd_boolean big_p) gdb_assert (bits % 8 == 0); bytes = bits / 8; + size_t last = buf.size (); + buf.resize (last + bytes); for (i = 0; i < bytes; i++) { int index = big_p ? bytes - i - 1 : i; - buf[index] = data & 0xff; + buf[last + index] = data & 0xff; data >>= 8; } } @@ -49,9 +52,8 @@ put_bits (bfd_uint64_t data, gdb_byte *buf, int bits, bfd_boolean big_p) /* Subroutine of find_command to simplify it. Parse the arguments of the "find" command. */ -static void -parse_find_args (char *args, ULONGEST *max_countp, - gdb_byte **pattern_bufp, ULONGEST *pattern_lenp, +static gdb::byte_vector +parse_find_args (const char *args, ULONGEST *max_countp, CORE_ADDR *start_addrp, ULONGEST *search_space_lenp, bfd_boolean big_p) { @@ -59,27 +61,15 @@ parse_find_args (char *args, ULONGEST *max_countp, char size = '\0'; ULONGEST max_count = ~(ULONGEST) 0; /* Buffer to hold the search pattern. */ - gdb_byte *pattern_buf; - /* Current size of search pattern buffer. - We realloc space as needed. */ -#define INITIAL_PATTERN_BUF_SIZE 100 - ULONGEST pattern_buf_size = INITIAL_PATTERN_BUF_SIZE; - /* Pointer to one past the last in-use part of pattern_buf. */ - gdb_byte *pattern_buf_end; - ULONGEST pattern_len; + gdb::byte_vector pattern_buf; CORE_ADDR start_addr; ULONGEST search_space_len; const char *s = args; - struct cleanup *old_cleanups; struct value *v; if (args == NULL) error (_("Missing search parameters.")); - pattern_buf = (gdb_byte *) xmalloc (pattern_buf_size); - pattern_buf_end = pattern_buf; - old_cleanups = make_cleanup (free_current_contents, &pattern_buf); - /* Get search granularity and/or max count if specified. They may be specified in either order, together or separately. */ @@ -131,9 +121,8 @@ parse_find_args (char *args, ULONGEST *max_countp, len = value_as_long (v); if (len == 0) { - do_cleanups (old_cleanups); printf_filtered (_("Empty search range.\n")); - return; + return pattern_buf; } if (len < 0) error (_("Invalid length.")); @@ -169,53 +158,36 @@ parse_find_args (char *args, ULONGEST *max_countp, { LONGEST x; struct type *t; - ULONGEST pattern_buf_size_need; s = skip_spaces (s); v = parse_to_comma_and_eval (&s); t = value_type (v); - /* Keep it simple and assume size == 'g' when watching for when we - need to grow the pattern buf. */ - pattern_buf_size_need = (pattern_buf_end - pattern_buf - + std::max (TYPE_LENGTH (t), - (unsigned) sizeof (int64_t))); - if (pattern_buf_size_need > pattern_buf_size) - { - size_t current_offset = pattern_buf_end - pattern_buf; - - pattern_buf_size = pattern_buf_size_need * 2; - pattern_buf = (gdb_byte *) xrealloc (pattern_buf, pattern_buf_size); - pattern_buf_end = pattern_buf + current_offset; - } - if (size != '\0') { x = value_as_long (v); switch (size) { case 'b': - *pattern_buf_end++ = x; + pattern_buf.push_back (x); break; case 'h': - put_bits (x, pattern_buf_end, 16, big_p); - pattern_buf_end += sizeof (int16_t); + put_bits (x, pattern_buf, 16, big_p); break; case 'w': - put_bits (x, pattern_buf_end, 32, big_p); - pattern_buf_end += sizeof (int32_t); + put_bits (x, pattern_buf, 32, big_p); break; case 'g': - put_bits (x, pattern_buf_end, 64, big_p); - pattern_buf_end += sizeof (int64_t); + put_bits (x, pattern_buf, 64, big_p); break; } } else { - memcpy (pattern_buf_end, value_contents (v), TYPE_LENGTH (t)); - pattern_buf_end += TYPE_LENGTH (t); + const gdb_byte *contents = value_contents (v); + pattern_buf.insert (pattern_buf.end (), contents, + contents + TYPE_LENGTH (t)); } if (*s == ',') @@ -223,23 +195,17 @@ parse_find_args (char *args, ULONGEST *max_countp, s = skip_spaces (s); } - if (pattern_buf_end == pattern_buf) + if (pattern_buf.empty ()) error (_("Missing search pattern.")); - pattern_len = pattern_buf_end - pattern_buf; - - if (search_space_len < pattern_len) + if (search_space_len < pattern_buf.size ()) error (_("Search space too small to contain pattern.")); *max_countp = max_count; - *pattern_bufp = pattern_buf; - *pattern_lenp = pattern_len; *start_addrp = start_addr; *search_space_lenp = search_space_len; - /* We successfully parsed the arguments, leave the freeing of PATTERN_BUF - to the caller now. */ - discard_cleanups (old_cleanups); + return pattern_buf; } static void @@ -250,33 +216,32 @@ find_command (char *args, int from_tty) /* Command line parameters. These are initialized to avoid uninitialized warnings from -Wall. */ ULONGEST max_count = 0; - gdb_byte *pattern_buf = 0; - ULONGEST pattern_len = 0; CORE_ADDR start_addr = 0; ULONGEST search_space_len = 0; /* End of command line parameters. */ unsigned int found_count; CORE_ADDR last_found_addr; - struct cleanup *old_cleanups; - parse_find_args (args, &max_count, &pattern_buf, &pattern_len, - &start_addr, &search_space_len, big_p); - - old_cleanups = make_cleanup (free_current_contents, &pattern_buf); + gdb::byte_vector pattern_buf = parse_find_args (args, &max_count, + &start_addr, + &search_space_len, + big_p); /* Perform the search. */ found_count = 0; last_found_addr = 0; - while (search_space_len >= pattern_len + while (search_space_len >= pattern_buf.size () && found_count < max_count) { /* Offset from start of this iteration to the next iteration. */ ULONGEST next_iter_incr; CORE_ADDR found_addr; int found = target_search_memory (start_addr, search_space_len, - pattern_buf, pattern_len, &found_addr); + pattern_buf.data (), + pattern_buf.size (), + &found_addr); if (found <= 0) break; @@ -313,8 +278,6 @@ find_command (char *args, int from_tty) else printf_filtered ("%d pattern%s found.\n", found_count, found_count > 1 ? "s" : ""); - - do_cleanups (old_cleanups); } void -- 2.7.4