From 5eae7aeaf754549d3645a0c590c5ef9f0134a32d Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Wed, 25 Oct 2017 15:52:23 -0600 Subject: [PATCH] Use unique_xmalloc_ptr in find_separate_debug_file_by_debuglink This changes find_separate_debug_file_by_debuglink to use unique_xmalloc_ptr, removing some cleanups. gdb/ChangeLog 2017-11-04 Tom Tromey * symfile.c (find_separate_debug_file_by_debuglink): Use unique_xmalloc_ptr. --- gdb/ChangeLog | 5 +++++ gdb/symfile.c | 36 ++++++++++++++---------------------- 2 files changed, 19 insertions(+), 22 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index a9eea8256bf..f707be415b4 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2017-11-04 Tom Tromey + + * symfile.c (find_separate_debug_file_by_debuglink): Use + unique_xmalloc_ptr. + 2017-11-04 Tom Tromey * compile/compile-loc2c.c (compute_stack_depth_worker): Change diff --git a/gdb/symfile.c b/gdb/symfile.c index 69358e44ce0..dae750ef051 100644 --- a/gdb/symfile.c +++ b/gdb/symfile.c @@ -1545,13 +1545,11 @@ terminate_after_last_dir_separator (char *path) char * find_separate_debug_file_by_debuglink (struct objfile *objfile) { - char *debuglink; - char *dir, *canon_dir; char *debugfile; unsigned long crc32; - struct cleanup *cleanups; - debuglink = bfd_get_debug_link_info (objfile->obfd, &crc32); + gdb::unique_xmalloc_ptr debuglink + (bfd_get_debug_link_info (objfile->obfd, &crc32)); if (debuglink == NULL) { @@ -1560,15 +1558,12 @@ find_separate_debug_file_by_debuglink (struct objfile *objfile) return NULL; } - cleanups = make_cleanup (xfree, debuglink); - dir = xstrdup (objfile_name (objfile)); - make_cleanup (xfree, dir); - terminate_after_last_dir_separator (dir); - canon_dir = lrealpath (dir); + std::string dir = objfile_name (objfile); + terminate_after_last_dir_separator (&dir[0]); + gdb::unique_xmalloc_ptr canon_dir (lrealpath (dir.c_str ())); - debugfile = find_separate_debug_file (dir, canon_dir, debuglink, - crc32, objfile); - xfree (canon_dir); + debugfile = find_separate_debug_file (dir.c_str (), canon_dir.get (), + debuglink.get (), crc32, objfile); if (debugfile == NULL) { @@ -1580,19 +1575,17 @@ find_separate_debug_file_by_debuglink (struct objfile *objfile) if (lstat (objfile_name (objfile), &st_buf) == 0 && S_ISLNK (st_buf.st_mode)) { - char *symlink_dir; - - symlink_dir = lrealpath (objfile_name (objfile)); + gdb::unique_xmalloc_ptr symlink_dir + (lrealpath (objfile_name (objfile))); if (symlink_dir != NULL) { - make_cleanup (xfree, symlink_dir); - terminate_after_last_dir_separator (symlink_dir); - if (strcmp (dir, symlink_dir) != 0) + terminate_after_last_dir_separator (symlink_dir.get ()); + if (dir != symlink_dir.get ()) { /* Different directory, so try using it. */ - debugfile = find_separate_debug_file (symlink_dir, - symlink_dir, - debuglink, + debugfile = find_separate_debug_file (symlink_dir.get (), + symlink_dir.get (), + debuglink.get (), crc32, objfile); } @@ -1600,7 +1593,6 @@ find_separate_debug_file_by_debuglink (struct objfile *objfile) } } - do_cleanups (cleanups); return debugfile; } -- 2.30.2