Blame SOURCES/gdb-6.5-bz185337-resolve-tls-without-debuginfo-v2.patch

4416f5
From FEDORA_PATCHES Mon Sep 17 00:00:00 2001
4416f5
From: Jan Kratochvil <jan.kratochvil@redhat.com>
4416f5
Date: Fri, 27 Oct 2017 21:07:50 +0200
4416f5
Subject: gdb-6.5-bz185337-resolve-tls-without-debuginfo-v2.patch
4416f5
4416f5
;; Support TLS symbols (+`errno' suggestion if no pthread is found) (BZ 185337).
4416f5
;;=push+jan: It should be replaced by Infinity project.
4416f5
4416f5
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=185337
4416f5
4416f5
2008-02-24  Jan Kratochvil  <jan.kratochvil@redhat.com>
4416f5
4416f5
	Port to GDB-6.8pre.
4416f5
4416f5
currently for trivial nonthreaded helloworld with no debug info up to -ggdb2 you
4416f5
will get:
4416f5
        (gdb) p errno
4416f5
        [some error]
4416f5
4416f5
* with -ggdb2 and less "errno" in fact does not exist anywhere as it was
4416f5
  compiled to "(*__errno_location ())" and the macro definition is not present.
4416f5
  Unfortunately gdb will find the TLS symbol and it will try to access it but
4416f5
  as the program has been compiled without -lpthread the TLS base register
4416f5
  (%gs on i386) is not setup and it will result in:
4416f5
        Cannot access memory at address 0x8
4416f5
4416f5
Attached suggestion patch how to deal with the most common "errno" symbol
4416f5
for the most common under-ggdb3 compiled programs.
4416f5
4416f5
Original patch hooked into target_translate_tls_address.  But its inferior
4416f5
call invalidates `struct frame *' in the callers - RH BZ 690908.
4416f5
4416f5
https://bugzilla.redhat.com/show_bug.cgi?id=1166549
4416f5
4416f5
2007-11-03  Jan Kratochvil  <jan.kratochvil@redhat.com>
4416f5
4416f5
	* ./gdb/dwarf2read.c (read_partial_die, dwarf2_linkage_name): Prefer
4416f5
	DW_AT_MIPS_linkage_name over DW_AT_name now only for non-C.
4416f5
4416f5
glibc-debuginfo-2.7-2.x86_64: /usr/lib/debug/lib64/libc.so.6.debug:
4416f5
  <81a2>     DW_AT_name        : (indirect string, offset: 0x280e): __errno_location
4416f5
  <81a8>     DW_AT_MIPS_linkage_name: (indirect string, offset: 0x2808): *__GI___errno_location
4416f5
4416f5
diff --git a/gdb/printcmd.c b/gdb/printcmd.c
4416f5
--- a/gdb/printcmd.c
4416f5
+++ b/gdb/printcmd.c
4416f5
@@ -1210,6 +1210,10 @@ print_command_1 (const char *args, int voidprint)
4416f5
 
4416f5
   if (exp != nullptr && *exp)
4416f5
     {
4416f5
+      /* '*((int *(*) (void)) __errno_location) ()' is incompatible with
4416f5
+	 function descriptors.  */
4416f5
+      if (target_has_execution && strcmp (exp, "errno") == 0)
4416f5
+	exp = "*(*(int *(*)(void)) __errno_location) ()";
4416f5
       expression_up expr = parse_expression (exp);
4416f5
       val = evaluate_expression (expr.get ());
4416f5
     }
4416f5
diff --git a/gdb/testsuite/gdb.dwarf2/dw2-errno.c b/gdb/testsuite/gdb.dwarf2/dw2-errno.c
4416f5
new file mode 100644
4416f5
--- /dev/null
4416f5
+++ b/gdb/testsuite/gdb.dwarf2/dw2-errno.c
4416f5
@@ -0,0 +1,28 @@
4416f5
+/* This testcase is part of GDB, the GNU debugger.
4416f5
+
4416f5
+   Copyright 2005, 2007 Free Software Foundation, Inc.
4416f5
+
4416f5
+   This program is free software; you can redistribute it and/or modify
4416f5
+   it under the terms of the GNU General Public License as published by
4416f5
+   the Free Software Foundation; either version 3 of the License, or
4416f5
+   (at your option) any later version.
4416f5
+
4416f5
+   This program is distributed in the hope that it will be useful,
4416f5
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
4416f5
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
4416f5
+   GNU General Public License for more details.
4416f5
+
4416f5
+   You should have received a copy of the GNU General Public License
4416f5
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
4416f5
+
4416f5
+   Please email any bugs, comments, and/or additions to this file to:
4416f5
+   bug-gdb@prep.ai.mit.edu  */
4416f5
+
4416f5
+#include <errno.h>
4416f5
+
4416f5
+int main()
4416f5
+{
4416f5
+  errno = 42;
4416f5
+
4416f5
+  return 0;	/* breakpoint */
4416f5
+}
4416f5
diff --git a/gdb/testsuite/gdb.dwarf2/dw2-errno.exp b/gdb/testsuite/gdb.dwarf2/dw2-errno.exp
4416f5
new file mode 100644
4416f5
--- /dev/null
4416f5
+++ b/gdb/testsuite/gdb.dwarf2/dw2-errno.exp
4416f5
@@ -0,0 +1,60 @@
4416f5
+# Copyright 2007 Free Software Foundation, Inc.
4416f5
+
4416f5
+# This program is free software; you can redistribute it and/or modify
4416f5
+# it under the terms of the GNU General Public License as published by
4416f5
+# the Free Software Foundation; either version 3 of the License, or
4416f5
+# (at your option) any later version.
4416f5
+#
4416f5
+# This program is distributed in the hope that it will be useful,
4416f5
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
4416f5
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
4416f5
+# GNU General Public License for more details.
4416f5
+#
4416f5
+# You should have received a copy of the GNU General Public License
4416f5
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
4416f5
+
4416f5
+set testfile dw2-errno
4416f5
+set srcfile ${testfile}.c
4416f5
+set binfile [standard_output_file ${testfile}]
4416f5
+
4416f5
+proc prep {} {
4416f5
+    global srcdir subdir binfile
4416f5
+    gdb_exit
4416f5
+    gdb_start
4416f5
+    gdb_reinitialize_dir $srcdir/$subdir
4416f5
+    gdb_load ${binfile}
4416f5
+
4416f5
+    runto_main
4416f5
+
4416f5
+    gdb_breakpoint [gdb_get_line_number "breakpoint"]
4416f5
+    gdb_continue_to_breakpoint "breakpoint"
4416f5
+}
4416f5
+
4416f5
+if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable "additional_flags=-g2"] != "" } {
4416f5
+    untested "Couldn't compile test program"
4416f5
+    return -1
4416f5
+}
4416f5
+prep
4416f5
+gdb_test "print errno" ".* = 42" "errno with macros=N threads=N"
4416f5
+
4416f5
+if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable "additional_flags=-g3"] != "" } {
4416f5
+    untested "Couldn't compile test program"
4416f5
+    return -1
4416f5
+}
4416f5
+prep
4416f5
+gdb_test "print errno" ".* = 42" "errno with macros=Y threads=N"
4416f5
+
4416f5
+if {[gdb_compile_pthreads "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable "additional_flags=-g2"] != "" } {
4416f5
+    return -1
4416f5
+}
4416f5
+prep
4416f5
+gdb_test "print errno" ".* = 42" "errno with macros=N threads=Y"
4416f5
+
4416f5
+if {[gdb_compile_pthreads "${srcdir}/${subdir}/${srcfile}" "${binfile}" executable "additional_flags=-g3"] != "" } {
4416f5
+    return -1
4416f5
+}
4416f5
+prep
4416f5
+gdb_test "print errno" ".* = 42" "errno with macros=Y threads=Y"
4416f5
+
4416f5
+# TODO: Test the error on resolving ERRNO with only libc loaded.
4416f5
+# Just how to find the current libc filename?
4416f5
diff --git a/gdb/testsuite/gdb.dwarf2/dw2-errno2.c b/gdb/testsuite/gdb.dwarf2/dw2-errno2.c
4416f5
new file mode 100644
4416f5
--- /dev/null
4416f5
+++ b/gdb/testsuite/gdb.dwarf2/dw2-errno2.c
4416f5
@@ -0,0 +1,28 @@
4416f5
+/* This testcase is part of GDB, the GNU debugger.
4416f5
+
4416f5
+   Copyright 2005, 2007 Free Software Foundation, Inc.
4416f5
+
4416f5
+   This program is free software; you can redistribute it and/or modify
4416f5
+   it under the terms of the GNU General Public License as published by
4416f5
+   the Free Software Foundation; either version 3 of the License, or
4416f5
+   (at your option) any later version.
4416f5
+
4416f5
+   This program is distributed in the hope that it will be useful,
4416f5
+   but WITHOUT ANY WARRANTY; without even the implied warranty of
4416f5
+   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
4416f5
+   GNU General Public License for more details.
4416f5
+
4416f5
+   You should have received a copy of the GNU General Public License
4416f5
+   along with this program.  If not, see <http://www.gnu.org/licenses/>.
4416f5
+
4416f5
+   Please email any bugs, comments, and/or additions to this file to:
4416f5
+   bug-gdb@prep.ai.mit.edu  */
4416f5
+
4416f5
+#include <errno.h>
4416f5
+
4416f5
+int main()
4416f5
+{
4416f5
+  errno = 42;
4416f5
+
4416f5
+  return 0;	/* breakpoint */
4416f5
+}
4416f5
diff --git a/gdb/testsuite/gdb.dwarf2/dw2-errno2.exp b/gdb/testsuite/gdb.dwarf2/dw2-errno2.exp
4416f5
new file mode 100644
4416f5
--- /dev/null
4416f5
+++ b/gdb/testsuite/gdb.dwarf2/dw2-errno2.exp
4416f5
@@ -0,0 +1,71 @@
4416f5
+# Copyright 2007 Free Software Foundation, Inc.
4416f5
+
4416f5
+# This program is free software; you can redistribute it and/or modify
4416f5
+# it under the terms of the GNU General Public License as published by
4416f5
+# the Free Software Foundation; either version 3 of the License, or
4416f5
+# (at your option) any later version.
4416f5
+#
4416f5
+# This program is distributed in the hope that it will be useful,
4416f5
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
4416f5
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
4416f5
+# GNU General Public License for more details.
4416f5
+#
4416f5
+# You should have received a copy of the GNU General Public License
4416f5
+# along with this program.  If not, see <http://www.gnu.org/licenses/>.
4416f5
+
4416f5
+set testfile dw2-errno2
4416f5
+set srcfile ${testfile}.c
4416f5
+set binfile [standard_output_file ${testfile}]
4416f5
+
4416f5
+proc prep { message {do_xfail 0} } { with_test_prefix $message {
4416f5
+    global srcdir subdir binfile variant
4416f5
+    gdb_exit
4416f5
+    gdb_start
4416f5
+    gdb_reinitialize_dir $srcdir/$subdir
4416f5
+    gdb_load ${binfile}${variant}
4416f5
+
4416f5
+    runto_main
4416f5
+
4416f5
+    gdb_breakpoint [gdb_get_line_number "breakpoint"]
4416f5
+    gdb_continue_to_breakpoint "breakpoint"
4416f5
+
4416f5
+    gdb_test "gcore ${binfile}${variant}.core" "\r\nSaved corefile .*" "gcore $variant"
4416f5
+
4416f5
+    gdb_test "print errno" ".* = 42"
4416f5
+
4416f5
+    gdb_test "kill" ".*" "kill" {Kill the program being debugged\? \(y or n\) } "y"
4416f5
+    gdb_test "core-file ${binfile}${variant}.core" "\r\nCore was generated by .*" "core-file"
4416f5
+    if $do_xfail {
4416f5
+	setup_xfail "*-*-*"
4416f5
+    }
4416f5
+    gdb_test "print (int) errno" ".* = 42" "print errno for core"
4416f5
+}}
4416f5
+
4416f5
+set variant g2thrN
4416f5
+if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}${variant}" executable "additional_flags=-g2"] != "" } {
4416f5
+    untested "Couldn't compile test program"
4416f5
+    return -1
4416f5
+}
4416f5
+prep "macros=N threads=N" 1
4416f5
+
4416f5
+set variant g3thrN
4416f5
+if  { [gdb_compile "${srcdir}/${subdir}/${srcfile}" "${binfile}${variant}" executable "additional_flags=-g3"] != "" } {
4416f5
+    untested "Couldn't compile test program"
4416f5
+    return -1
4416f5
+}
4416f5
+prep "macros=Y threads=N" 1
4416f5
+
4416f5
+set variant g2thrY
4416f5
+if {[gdb_compile_pthreads "${srcdir}/${subdir}/${srcfile}" "${binfile}${variant}" executable "additional_flags=-g2"] != "" } {
4416f5
+    return -1
4416f5
+}
4416f5
+prep "macros=N threads=Y"
4416f5
+
4416f5
+set variant g3thrY
4416f5
+if {[gdb_compile_pthreads "${srcdir}/${subdir}/${srcfile}" "${binfile}${variant}" executable "additional_flags=-g3"] != "" } {
4416f5
+    return -1
4416f5
+}
4416f5
+prep "macros=Y threads=Y" 1
4416f5
+
4416f5
+# TODO: Test the error on resolving ERRNO with only libc loaded.
4416f5
+# Just how to find the current libc filename?