From 074319087452e3a8b1a0e84279a82555dd798d69 Mon Sep 17 00:00:00 2001 From: Simon Marchi Date: Tue, 14 Nov 2017 16:42:08 -0500 Subject: [PATCH] Fix mem region parsing regression and add test In my patch Get rid of VEC (mem_region) a664f67e50eff30198097d51cec0ec4690abb2a1 I introduced a regression, where the length of the memory region is assigned to the "hi" field. It should obviously be computed as "start + length". To my defense, no test had caught this :). As a penance, I wrote one. gdb/ChangeLog: * Makefile.in (SUBDIR_UNITTESTS_SRCS): Add memory-map-selftests.c. (SUBDIR_UNITTESTS_OBS): Add memory-map-selftests.o. * memory-map.c (memory_map_start_memory): Fix computation of hi address. * unittests/memory-map-selftests.c: New file. --- gdb/ChangeLog | 9 ++++ gdb/Makefile.in | 2 + gdb/memory-map.c | 2 +- gdb/unittests/memory-map-selftests.c | 81 ++++++++++++++++++++++++++++ 4 files changed, 93 insertions(+), 1 deletion(-) create mode 100644 gdb/unittests/memory-map-selftests.c diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 59324f3b76..9a22ce769f 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,12 @@ +2017-11-14 Simon Marchi + + * Makefile.in (SUBDIR_UNITTESTS_SRCS): Add + memory-map-selftests.c. + (SUBDIR_UNITTESTS_OBS): Add memory-map-selftests.o. + * memory-map.c (memory_map_start_memory): Fix computation of hi + address. + * unittests/memory-map-selftests.c: New file. + 2017-11-09 Joel Brobecker * ada-lang.c: Fix some typos in the general command documenting diff --git a/gdb/Makefile.in b/gdb/Makefile.in index 5e0181635f..a3bfbf93e1 100644 --- a/gdb/Makefile.in +++ b/gdb/Makefile.in @@ -531,6 +531,7 @@ SUBDIR_UNITTESTS_SRCS = \ unittests/environ-selftests.c \ unittests/function-view-selftests.c \ unittests/lookup_name_info-selftests.c \ + unittests/memory-map-selftests.c \ unittests/memrange-selftests.c \ unittests/offset-type-selftests.c \ unittests/optional-selftests.c \ @@ -544,6 +545,7 @@ SUBDIR_UNITTESTS_OBS = \ environ-selftests.o \ function-view-selftests.o \ lookup_name_info-selftests.o \ + memory-map-selftests.o \ memrange-selftests.o \ offset-type-selftests.o \ optional-selftests.o \ diff --git a/gdb/memory-map.c b/gdb/memory-map.c index 9582cebb60..e0982093a7 100644 --- a/gdb/memory-map.c +++ b/gdb/memory-map.c @@ -71,7 +71,7 @@ memory_map_start_memory (struct gdb_xml_parser *parser, type_p = (ULONGEST *) xml_find_attribute (attributes, "type")->value; - data->memory_map->emplace_back (*start_p, *length_p, + data->memory_map->emplace_back (*start_p, *start_p + *length_p, (enum mem_access_mode) *type_p); } diff --git a/gdb/unittests/memory-map-selftests.c b/gdb/unittests/memory-map-selftests.c new file mode 100644 index 0000000000..3b282b5554 --- /dev/null +++ b/gdb/unittests/memory-map-selftests.c @@ -0,0 +1,81 @@ +/* Self tests for memory-map for GDB, the GNU debugger. + + Copyright (C) 2017 Free Software Foundation, Inc. + + This file is part of GDB. + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 3 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program. If not, see . */ + +#include "defs.h" +#include "selftest.h" +#include "memory-map.h" + +namespace selftests { +namespace memory_map_tests { + +/* A simple valid test input for parse_memory_map. */ + +static const char *valid_mem_map = R"( + + + + + + 1024 + + +)"; + +/* Validate memory region R against some expected values (the other + parameters). */ + +static void +check_mem_region (const mem_region &r, CORE_ADDR lo, CORE_ADDR hi, + mem_access_mode mode, int blocksize) +{ + SELF_CHECK (r.lo == lo); + SELF_CHECK (r.hi == hi); + SELF_CHECK (r.enabled_p); + + SELF_CHECK (r.attrib.mode == mode); + SELF_CHECK (r.attrib.blocksize == blocksize); + +} + +/* Test the parse_memory_map function. */ + +static void +parse_memory_map_tests () +{ + std::vector regions = parse_memory_map (valid_mem_map); + + SELF_CHECK (regions.size () == 3); + + check_mem_region (regions[0], 0, 0 + 4096, MEM_RW, -1); + check_mem_region (regions[1], 65536, 65536 + 256, MEM_RO, -1); + check_mem_region (regions[2], 131072, 131072 + 65536, MEM_FLASH, 1024); +} + +} /* namespace memory_map_tests */ +} /* namespace selftests */ + +void +_initialize_memory_map_selftests () +{ + selftests::register_test + ("parse_memory_map", + selftests::memory_map_tests::parse_memory_map_tests); +} -- 2.34.1