]> git.pld-linux.org Git - packages/gdb.git/blobdiff - gdb-bz528668-symfile-cleanup.patch
- updated to 7.2 with fedora patchset w/o rhel compat fixes.
[packages/gdb.git] / gdb-bz528668-symfile-cleanup.patch
diff --git a/gdb-bz528668-symfile-cleanup.patch b/gdb-bz528668-symfile-cleanup.patch
deleted file mode 100644 (file)
index 2a843fa..0000000
+++ /dev/null
@@ -1,137 +0,0 @@
-http://sourceware.org/ml/gdb-patches/2009-10/msg00509.html
-Subject: [patch 2/3] find_separate_debug_file cleanup
-
-Hi,
-
-current code was:
-* difficult to maintain as a new variable required xfree on many places
-* was causing memory corruptions due to silently misapplied 3rd party patches
-  as the close code fragments unfortunately match patch context
-
-
-Thanks,
-Jan
-
-
-gdb/
-2009-10-21  Jan Kratochvil  <jan.kratochvil@redhat.com>
-
-       * symfile.c (find_separate_debug_file): Initialize dir, debugfile and
-       canon_name to NULL.  Change alloca to xmalloc, newly call xfree for it.
-       New label cleanup_return_debugfile, jump to it from the failure paths.
-
---- a/gdb/symfile.c
-+++ b/gdb/symfile.c
-@@ -1333,11 +1333,10 @@ static char *
- find_separate_debug_file (struct objfile *objfile)
- {
-   asection *sect;
--  char *basename;
--  char *dir;
--  char *debugfile;
--  char *name_copy;
--  char *canon_name;
-+  char *basename, *name_copy;
-+  char *dir = NULL;
-+  char *debugfile = NULL;
-+  char *canon_name = NULL;
-   bfd_size_type debuglink_size;
-   unsigned long crc32;
-   int i;
-@@ -1366,7 +1365,7 @@ find_separate_debug_file (struct objfile *objfile)
-   if (basename == NULL)
-     /* There's no separate debug info, hence there's no way we could
-        load it => no warning.  */
--    return NULL;
-+    goto cleanup_return_debugfile;
-   dir = xstrdup (objfile->name);
-@@ -1388,24 +1387,19 @@ find_separate_debug_file (struct objfile *objfile)
-   if (canon_name && strlen (canon_name) > i)
-     i = strlen (canon_name);
--  debugfile = alloca (strlen (debug_file_directory) + 1
--                      + i
--                      + strlen (DEBUG_SUBDIRECTORY)
--                      + strlen ("/")
--                      + strlen (basename)
--                      + 1);
-+  debugfile = xmalloc (strlen (debug_file_directory) + 1
-+                     + i
-+                     + strlen (DEBUG_SUBDIRECTORY)
-+                     + strlen ("/")
-+                     + strlen (basename)
-+                     + 1);
-   /* First try in the same directory as the original file.  */
-   strcpy (debugfile, dir);
-   strcat (debugfile, basename);
-   if (separate_debug_file_exists (debugfile, crc32, objfile->name))
--    {
--      xfree (basename);
--      xfree (dir);
--      xfree (canon_name);
--      return xstrdup (debugfile);
--    }
-+    goto cleanup_return_debugfile;
-   /* Then try in the subdirectory named DEBUG_SUBDIRECTORY.  */
-   strcpy (debugfile, dir);
-@@ -1414,12 +1408,7 @@ find_separate_debug_file (struct objfile *objfile)
-   strcat (debugfile, basename);
-   if (separate_debug_file_exists (debugfile, crc32, objfile->name))
--    {
--      xfree (basename);
--      xfree (dir);
--      xfree (canon_name);
--      return xstrdup (debugfile);
--    }
-+    goto cleanup_return_debugfile;
-   /* Then try in the global debugfile directory.  */
-   strcpy (debugfile, debug_file_directory);
-@@ -1428,12 +1417,7 @@ find_separate_debug_file (struct objfile *objfile)
-   strcat (debugfile, basename);
-   if (separate_debug_file_exists (debugfile, crc32, objfile->name))
--    {
--      xfree (basename);
--      xfree (dir);
--      xfree (canon_name);
--      return xstrdup (debugfile);
--    }
-+    goto cleanup_return_debugfile;
-   /* If the file is in the sysroot, try using its base path in the
-      global debugfile directory.  */
-@@ -1447,20 +1431,17 @@ find_separate_debug_file (struct objfile *objfile)
-       strcat (debugfile, basename);
-       if (separate_debug_file_exists (debugfile, crc32, objfile->name))
--      {
--        xfree (canon_name);
--        xfree (basename);
--        xfree (dir);
--        return xstrdup (debugfile);
--      }
-+      goto cleanup_return_debugfile;
-     }
-   
--  if (canon_name)
--    xfree (canon_name);
-+  xfree (debugfile);
-+  debugfile = NULL;
-+cleanup_return_debugfile:
-+  xfree (canon_name);
-   xfree (basename);
-   xfree (dir);
--  return NULL;
-+  return debugfile;
- }
-
This page took 0.033716 seconds and 4 git commands to generate.