Don't needlessly clear xmemdup allocated memory.
authorAlan Modra <amodra@gmail.com>
Tue, 31 May 2016 11:04:47 +0000 (20:34 +0930)
committerAlan Modra <amodra@gmail.com>
Tue, 31 May 2016 11:04:47 +0000 (20:34 +0930)
* xmemdup.c (xmemdup): Use xmalloc rather than xcalloc.

libiberty/ChangeLog
libiberty/xmemdup.c

index 358a6b0f58632f0705929130d15abcc4740b5417..c55af153a9b0a2c009e60d447ca5abeb446bedd0 100644 (file)
@@ -1,3 +1,7 @@
+2016-05-31  Alan Modra  <amodra@gmail.com>
+
+       * xmemdup.c (xmemdup): Use xmalloc rather than xcalloc.
+
 2016-01-27  Iain Buclaw  <ibuclaw@gdcproject.org>
 
        * d-demangle.c (dlang_call_convention): Handle extern Objective-C
index aa56f0bf572bb71a52967878ddd3774cd7c9c1d2..4602afd7d9f6acb5e1bc57c2bd5ac3da4e25d5aa 100644 (file)
@@ -1,4 +1,4 @@
-/* xmemdup.c -- Duplicate a memory buffer, using xcalloc.
+/* xmemdup.c -- Duplicate a memory buffer, using xmalloc.
    This trivial function is in the public domain.
    Jeff Garzik, September 1999.  */
 
@@ -34,6 +34,8 @@ allocated, the remaining memory is zeroed.
 PTR
 xmemdup (const PTR input, size_t copy_size, size_t alloc_size)
 {
-  PTR output = xcalloc (1, alloc_size);
+  PTR output = xmalloc (alloc_size);
+  if (alloc_size > copy_size)
+    memset ((char *) output + copy_size, 0, alloc_size - copy_size);
   return (PTR) memcpy (output, input, copy_size);
 }