This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [patch] Fix for PR gdb/9538 (loading of separate debuginfo and symlinks).
On Thu, 12 Jan 2012 23:19:17 +0100, Paul Pluzhnikov wrote:
> On Thu, Jan 12, 2012 at 1:29 PM, Jan Kratochvil
> <jan.kratochvil@redhat.com> wrote:
> >> + Âdir1 = xstrdup (objfile->name);
> >> + Âterminate_after_last_dir_separator (dir1);
> >> + Âcanon_dir = lrealpath (dir1);
> >
> > lrealpath can return NULL. ÂGDB did not crash before. ÂIt will now.
>
> Where? canon_dir is passed into the utility function, which checks for NULL
> (as it did before).
I misread some comment as a code before, sorry. I agree.
> 2012-01-11 Paul Pluzhnikov <ppluzhnikov@google.com>
>
> PR gdb/9538
> * symfile.c (find_separate_debug_file): New function.
> (terminate_after_last_dir_separator): Likewise.
> (find_separate_debug_file_by_debuglink): Also try realpath.
>
>
> testsuite/ChangeLog:
>
> PR gdb/9538
> * gdb.base/sepdebug.exp: New test.
I do not see the new patch but the described changes look OK.
Thanks,
Jan