Return unique_xmalloc_ptr from some solib.c functions
authorTom Tromey <tom@tromey.com>
Fri, 10 Nov 2017 20:07:46 +0000 (13:07 -0700)
committerTom Tromey <tom@tromey.com>
Wed, 14 Feb 2018 15:09:52 +0000 (08:09 -0700)
commit797bc1cb25b9dbdbc663cf711aecb0acc2450276
tree8a38a8625056ea60dcaea4458a6dead3b9ba86c1
parent12ef3f5a7c5a6b89964842fd3da047b8d07dec91
Return unique_xmalloc_ptr from some solib.c functions

This changes a couple of solib.c functions -- exec_file_find and
solib_find -- to return a unique_xmalloc_ptr, and then fixes up the
users.  This allows the removal of some cleanups.

This also changes solib_bfd_open to not take ownership of its
argument.  I think this change is somewhat cleaner.

gdb/ChangeLog
2018-02-14  Tom Tromey  <tom@tromey.com>

* solist.h (exec_file_find, solib_find): Return
unique_xmalloc_ptr.
(solib_bfd_fopen): Take a const char *.
* solib.c (solib_find_1): Return unique_xmalloc_ptr.
(exec_file_find, solib_find): Likewise.
(solib_bfd_fopen): Do not take ownership of "pathname".
(solib_bfd_open): Use unique_xmalloc_ptr.
* solib-darwin.c (darwin_bfd_open): Use unique_xmalloc_ptr.
* solib-aix.c (solib_aix_bfd_open): Use unique_xmalloc_ptr.
* infrun.c (follow_exec): Use unique_xmalloc_ptr.
* exec.c (exec_file_locate_attach): Use unique_xmalloc_ptr.
gdb/ChangeLog
gdb/exec.c
gdb/infrun.c
gdb/solib-aix.c
gdb/solib-darwin.c
gdb/solib.c
gdb/solist.h