diff options
| author | Simon Marchi <simon.marchi@efficios.com> | 2020-01-13 18:12:08 -0500 |
|---|---|---|
| committer | Simon Marchi <simon.marchi@efficios.com> | 2020-01-13 18:12:08 -0500 |
| commit | 0cac9354bfb07a5cf53d70dbefaa35c6afa7da0a (patch) | |
| tree | 92db38d9e027bd2eea944586d3f1aed2aa10472d /gdb/corelow.c | |
| parent | 9a6d629ccf328e3f041c3fcb7e91f49a5d72d0fb (diff) | |
| download | binutils-0cac9354bfb07a5cf53d70dbefaa35c6afa7da0a.zip binutils-0cac9354bfb07a5cf53d70dbefaa35c6afa7da0a.tar.gz binutils-0cac9354bfb07a5cf53d70dbefaa35c6afa7da0a.tar.bz2 | |
gdb: use gdb::byte_vector instead of std::vector<char> in core_target::get_core_register_section
Since the data held by the `contents` variable is arbitrary binary data,
it should have gdb_byte elements, not char elements. Also, using
gdb::byte_vector is preferable, since it doesn't unnecessarily
zero-initialize the values.
Instead of adding a cast in the call to m_core_vec->core_read_registers,
I have changed core_read_registers' argument to be a gdb_byte* instead
of a char*.
gdb/ChangeLog:
* gdbcore.h (struct core_fns) <core_read_registers>: Change
core_reg_sect type to gdb_byte *.
* arm-nbsd-nat.c (fetch_elfcore_registers): Likewise.
* cris-tdep.c (fetch_core_registers): Likewise.
* corelow.c (core_target::get_core_register_section): Change
type of `contents` to gdb::byte_vector.
Diffstat (limited to 'gdb/corelow.c')
| -rw-r--r-- | gdb/corelow.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gdb/corelow.c b/gdb/corelow.c index 0418ec2..5cd058d 100644 --- a/gdb/corelow.c +++ b/gdb/corelow.c @@ -621,7 +621,7 @@ core_target::get_core_register_section (struct regcache *regcache, section_name.c_str ()); } - std::vector<char> contents (size); + gdb::byte_vector contents (size); if (!bfd_get_section_contents (core_bfd, section, contents.data (), (file_ptr) 0, size)) { |