|
 |
405ea9 |
From FEDORA_PATCHES Mon Sep 17 00:00:00 2001
|
|
|
405ea9 |
From: Kevin Buettner <kevinb@redhat.com>
|
|
|
405ea9 |
Date: Wed, 17 Feb 2021 17:58:54 -0700
|
|
|
405ea9 |
Subject: gdb-rhbz1912985-libstdc++-assert.patch
|
|
|
405ea9 |
|
|
|
405ea9 |
;; Backport fix for libstdc++ assert when performing tab completion
|
|
|
405ea9 |
;; (RH BZ 1912985).
|
|
|
405ea9 |
|
|
|
405ea9 |
Fix completion related libstdc++ assert when using -D_GLIBCXX_DEBUG
|
|
|
405ea9 |
|
|
|
405ea9 |
This commit fixes a libstdc++ assertion failure encountered when
|
|
|
405ea9 |
running gdb.base/completion.exp. In order to see this problem,
|
|
|
405ea9 |
GDB must be built with the follow CFLAGS and CXXFLAGS as part
|
|
|
405ea9 |
of the configure line:
|
|
|
405ea9 |
|
|
|
405ea9 |
CFLAGS='-D_GLIBCXX_DEBUG' CXXFLAGS='-D_GLIBCXX_DEBUG'
|
|
|
405ea9 |
|
|
|
405ea9 |
(Also, this problem was encountered using Fedora rawhide. It might
|
|
|
405ea9 |
not be reproducible in Fedora versions prior to Fedora 34.)
|
|
|
405ea9 |
|
|
|
405ea9 |
Using the gdb.base/completion.exp test program, the problem can be
|
|
|
405ea9 |
observed as follows:
|
|
|
405ea9 |
|
|
|
405ea9 |
[kev@rawhide-1 gdb]$ ./gdb -q testsuite/outputs/gdb.base/completion/completion
|
|
|
405ea9 |
Reading symbols from testsuite/outputs/gdb.base/completion/completion...
|
|
|
405ea9 |
(gdb) start
|
|
|
405ea9 |
Temporary breakpoint 1 at 0x401179: file ../../worktree-master/gdb/testsuite/gdb.base/break.c, line 43.
|
|
|
405ea9 |
Starting program: testsuite/outputs/gdb.base/completion/completion
|
|
|
405ea9 |
|
|
|
405ea9 |
Temporary breakpoint 1, main (argc=1, argv=0x7fffffffd718, envp=0x7fffffffd728) at ../../worktree-master/gdb/testsuite/gdb.base/break.c:43
|
|
|
405ea9 |
43 if (argc == 12345) { /* an unlikely value < 2^16, in case uninited */ /* set breakpoint 6 here */
|
|
|
405ea9 |
(gdb) p <TAB>/usr/include/c++/11/string_view:211: constexpr const value_type& std::basic_string_view<_CharT, _Traits>::operator[](std::basic_string_view<_CharT, _Traits>::size_type) const [with _CharT = char; _Traits = std::char_traits<char>; std::basic_string_view<_CharT, _Traits>::const_reference = const char&; std::basic_string_view<_CharT, _Traits>::size_type = long unsigned int]: Assertion '__pos < this->_M_len' failed.
|
|
|
405ea9 |
Aborted (core dumped)
|
|
|
405ea9 |
|
|
|
405ea9 |
(Note that I added "<TAB>" to make it clear where the tab key was
|
|
|
405ea9 |
pressed.)
|
|
|
405ea9 |
|
|
|
405ea9 |
gdb/ChangeLog:
|
|
|
405ea9 |
|
|
|
405ea9 |
* ada-lang.c (ada_fold_name): Check for non-empty string prior
|
|
|
405ea9 |
to accessing it.
|
|
|
405ea9 |
(ada_lookup_name_info): Likewise.
|
|
|
405ea9 |
|
|
|
405ea9 |
diff --git a/gdb/ada-lang.c b/gdb/ada-lang.c
|
|
|
405ea9 |
--- a/gdb/ada-lang.c
|
|
|
405ea9 |
+++ b/gdb/ada-lang.c
|
|
|
405ea9 |
@@ -997,7 +997,7 @@ ada_fold_name (gdb::string_view name)
|
|
|
405ea9 |
int len = name.size ();
|
|
|
405ea9 |
GROW_VECT (fold_buffer, fold_buffer_size, len + 1);
|
|
|
405ea9 |
|
|
|
405ea9 |
- if (name[0] == '\'')
|
|
|
405ea9 |
+ if (!name.empty () && name[0] == '\'')
|
|
|
405ea9 |
{
|
|
|
405ea9 |
strncpy (fold_buffer, name.data () + 1, len - 2);
|
|
|
405ea9 |
fold_buffer[len - 2] = '\000';
|
|
|
405ea9 |
@@ -13597,7 +13597,7 @@ ada_lookup_name_info::ada_lookup_name_info (const lookup_name_info &lookup_name)
|
|
|
405ea9 |
{
|
|
|
405ea9 |
gdb::string_view user_name = lookup_name.name ();
|
|
|
405ea9 |
|
|
|
405ea9 |
- if (user_name[0] == '<')
|
|
|
405ea9 |
+ if (!user_name.empty () && user_name[0] == '<')
|
|
|
405ea9 |
{
|
|
|
405ea9 |
if (user_name.back () == '>')
|
|
|
405ea9 |
m_encoded_name
|