Use unique_xmalloc_ptr in solib-aix.c
authorTom Tromey <tom@tromey.com>
Sun, 27 May 2018 15:33:24 +0000 (09:33 -0600)
committerTom Tromey <tom@tromey.com>
Mon, 18 Jun 2018 19:26:33 +0000 (13:26 -0600)
This removes a cleanup from solib-aix.c via unique_xmalloc_ptr.

gdb/ChangeLog
2018-06-18  Tom Tromey  <tom@tromey.com>

* solib-aix.c (solib_aix_get_section_offsets): Return
unique_xmalloc_ptr.
(solib_aix_solib_create_inferior_hook): Update.

gdb/ChangeLog
gdb/solib-aix.c

index c6112d2..4fc6cf5 100644 (file)
@@ -1,5 +1,11 @@
 2018-06-18  Tom Tromey  <tom@tromey.com>
 
+       * solib-aix.c (solib_aix_get_section_offsets): Return
+       unique_xmalloc_ptr.
+       (solib_aix_solib_create_inferior_hook): Update.
+
+2018-06-18  Tom Tromey  <tom@tromey.com>
+
        * solib-darwin.c (darwin_current_sos): Use unique_xmalloc_ptr.
 
 2018-06-18  Tom Tromey  <tom@tromey.com>
index 6e5b8b1..8e8901e 100644 (file)
@@ -439,14 +439,14 @@ solib_aix_clear_solib (void)
    The resulting array is computed on the heap and must be
    deallocated after use.  */
 
-static struct section_offsets *
+static gdb::unique_xmalloc_ptr<struct section_offsets>
 solib_aix_get_section_offsets (struct objfile *objfile,
                               lm_info_aix *info)
 {
-  struct section_offsets *offsets;
   bfd *abfd = objfile->obfd;
 
-  offsets = XCNEWVEC (struct section_offsets, objfile->num_sections);
+  gdb::unique_xmalloc_ptr<struct section_offsets> offsets
+    (XCNEWVEC (struct section_offsets, objfile->num_sections));
 
   /* .text */
 
@@ -515,12 +515,10 @@ solib_aix_solib_create_inferior_hook (int from_tty)
 
   if (symfile_objfile != NULL)
     {
-      struct section_offsets *offsets
+      gdb::unique_xmalloc_ptr<struct section_offsets> offsets
        = solib_aix_get_section_offsets (symfile_objfile, exec_info);
-      struct cleanup *cleanup = make_cleanup (xfree, offsets);
 
-      objfile_relocate (symfile_objfile, offsets);
-      do_cleanups (cleanup);
+      objfile_relocate (symfile_objfile, offsets.get ());
     }
 }