FreeChainXenon/binutils-gdb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix self-test failure in addrmap

Browse source 
Mark pointed out that my recent addrmap C++-ficiation changes caused a
regression in the self-tests.  This patch fixes the problem by
updating this test not to allocate the mutable addrmap on an obstack.
This commit is contained in:
Tom Tromey 2022-06-12 15:53:40 -06:00
parent 32681b482a
commit aa09537375
1 changed files with 3 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

10
gdb/addrmap.c
View file

@ -429,9 +429,8 @@ test_addrmap ()
void *val2 = &array[2];
/* Create mutable addrmap. */
struct obstack temp_obstack;
obstack_init (&temp_obstack);
struct addrmap_mutable *map = new (&temp_obstack) addrmap_mutable;
auto_obstack temp_obstack;
std::unique_ptr<struct addrmap_mutable> map (new addrmap_mutable);
SELF_CHECK (map != nullptr);
/* Check initial state. */
@ -445,7 +444,7 @@ test_addrmap ()
/* Create corresponding fixed addrmap. */
struct addrmap *map2
= new (&temp_obstack) addrmap_fixed (&temp_obstack, map);
= new (&temp_obstack) addrmap_fixed (&temp_obstack, map.get ());
SELF_CHECK (map2 != nullptr);
CHECK_ADDRMAP_FIND (map2, array, 0, 9, nullptr);
CHECK_ADDRMAP_FIND (map2, array, 10, 12, val1);
@ -479,9 +478,6 @@ test_addrmap ()
CHECK_ADDRMAP_FIND (map, array, 10, 12, val1);
CHECK_ADDRMAP_FIND (map, array, 13, 13, val2);
CHECK_ADDRMAP_FIND (map, array, 14, 19, nullptr);
/* Cleanup. */
obstack_free (&temp_obstack, NULL);
}
} // namespace selftests