Fix gdb segv when objfile can't be opened
authorMike Gulick <mike.gulick@mathworks.com>
Mon, 30 Oct 2017 22:13:44 +0000 (18:13 -0400)
committerSimon Marchi <simon.marchi@ericsson.com>
Wed, 17 Jan 2018 17:54:59 +0000 (12:54 -0500)
commit416675305692976aca45860e24b963982a2e682a
tree5999c0ba0fccc5fdffa709272d61b791747b3abd
parent4d9b86e17505063c96a01d40cdf5b4fc2080a798
Fix gdb segv when objfile can't be opened

This fixes PR 16577.

This patch changes gdb_bfd_map_section to issue a warning rather than an error
if it is unable to read the object file, and sets the size of the section/frame
that it attempted to read to 0 on error.

The description of gdb_bfd_map_section states that it will try to read or map
the contents of the section SECT, and if successful, the section data is
returned and *SIZE is set to the size of the section data.  This function was
throwing an error and leaving *SIZE as-is.  Setting the section size to 0
indicates to dwarf2_build_frame_info that there is no data to read, otherwise
it will try to read from an invalid frame pointer.

Changing the error to a warning allows this to be handled gracefully.
Additionally, the error was clobbering the breakpoint output indicating the
current frame (function name, arguments, source file, and line number).  E.g.

Thread 3 "foo" hit Breakpoint 1, BFD: reopening /tmp/jna-1013829440/jna2973250704389291330.tmp: No such file or directory
BFD: reopening /tmp/jna-1013829440/jna2973250704389291330.tmp: No such file or directory
(gdb)

While the "BFD: reopening ..." messages will still appear interspersed in the
breakpoint output, the current frame info is now displayed:

Thread 3 "foo" hit Breakpoint 1, BFD: reopening /tmp/jna-1013829440/jna1875755897659885075.tmp: No such file or directory
BFD: reopening /tmp/jna-1013829440/jna1875755897659885075.tmp: No such file or directory
warning: Can't read data for section '.eh_frame' in file '/tmp/jna-1013829440/jna1875755897659885075.tmp'
do_something () at file.cpp:80
80 {
(gdb)
gdb/ChangeLog
gdb/gdb_bfd.c
gdb/gdb_bfd.h
gdb/testsuite/ChangeLog
gdb/testsuite/gdb.base/solib-vanish-lib1.c [new file with mode: 0644]
gdb/testsuite/gdb.base/solib-vanish-lib2.c [new file with mode: 0644]
gdb/testsuite/gdb.base/solib-vanish-main.c [new file with mode: 0644]
gdb/testsuite/gdb.base/solib-vanish.exp [new file with mode: 0644]