* compress.c (bfd_get_full_section_contents): Don't cache
authorTom Tromey <tromey@redhat.com>
Fri, 19 Oct 2012 15:51:53 +0000 (15:51 +0000)
committerTom Tromey <tromey@redhat.com>
Fri, 19 Oct 2012 15:51:53 +0000 (15:51 +0000)
decompressed contents.

bfd/ChangeLog
bfd/compress.c

index 5c7e869d2721ab00473846439e366908d76397c9..ab216bd4204e07bc70bbf172613fdc856cdba33b 100644 (file)
@@ -1,3 +1,8 @@
+2012-10-19  Tom Tromey  <tromey@redhat.com>
+
+       * compress.c (bfd_get_full_section_contents): Don't cache
+       decompressed contents.
+
 2012-10-18  Kai Tietz  <ktietz@redhat.com>
 
        PR binutils/14067
index 17b5928a026b0b7c080c5f23f71c340f94e7194c..34faa088fd411657ce077927bd9d176d590db6cf 100644 (file)
@@ -165,7 +165,6 @@ bfd_get_full_section_contents (bfd *abfd, sec_ptr sec, bfd_byte **ptr)
   bfd_size_type uncompressed_size;
   bfd_size_type rawsize;
   bfd_byte *compressed_buffer;
-  bfd_byte *uncompressed_buffer;
 #endif
 
   if (abfd->direction != write_direction && sec->rawsize != 0)
@@ -220,24 +219,24 @@ bfd_get_full_section_contents (bfd *abfd, sec_ptr sec, bfd_byte **ptr)
       if (!ret)
        goto fail_compressed;
 
-      uncompressed_buffer = (bfd_byte *) bfd_malloc (uncompressed_size);
-      if (uncompressed_buffer == NULL)
+      if (p == NULL)
+       p = (bfd_byte *) bfd_malloc (uncompressed_size);
+      if (p == NULL)
        goto fail_compressed;
 
       if (!decompress_contents (compressed_buffer, compressed_size,
-                               uncompressed_buffer, uncompressed_size))
+                               p, uncompressed_size))
        {
          bfd_set_error (bfd_error_bad_value);
-         free (uncompressed_buffer);
+         free (p);
        fail_compressed:
          free (compressed_buffer);
          return FALSE;
        }
 
       free (compressed_buffer);
-      sec->contents = uncompressed_buffer;
-      sec->compress_status = COMPRESS_SECTION_DONE;
-      /* Fall thru */
+      *ptr = p;
+      return TRUE;
 #endif
 
     case COMPRESS_SECTION_DONE: