From: Phil Muldoon <pmuldoon@redhat.com>
Date: Fri, 27 Feb 2009 19:33:06 +0000 (+0000)
Subject: 2009-02-27  Phil Muldoon  <pmuldoon@redhat.com>
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=b5096abe11166944e2de1bb8a27a1c36d02a63ef;p=binutils-gdb.git

2009-02-27  Phil Muldoon  <pmuldoon@redhat.com>

	* valprint.c (read_string): Rework clean-up logic. Use
	free_current_contents to clean-up buffer.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 64c0b53cb63..15c98f911c8 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2009-02-27  Phil Muldoon  <pmuldoon@redhat.com>
+
+	* valprint.c (read_string): Rework clean-up logic. Use
+	free_current_contents to clean-up buffer.
+
 2009-02-27  Andreas Schwab  <schwab@linux-m68k.org>
 
 	* MAINTAINERS: Update e-mail address.
diff --git a/gdb/valprint.c b/gdb/valprint.c
index d13dc4cd20c..b02e9dfd174 100644
--- a/gdb/valprint.c
+++ b/gdb/valprint.c
@@ -1226,13 +1226,14 @@ read_string (CORE_ADDR addr, int len, int width, unsigned int fetchlimit,
      some error, such as bumping into the end of the address space.  */
 
   found_nul = 0;
-  old_chain = make_cleanup (null_cleanup, 0);
+  *buffer = NULL;
+
+  old_chain = make_cleanup (free_current_contents, buffer);
 
   if (len > 0)
     {
       *buffer = (gdb_byte *) xmalloc (len * width);
       bufptr = *buffer;
-      old_chain = make_cleanup (xfree, *buffer);
 
       nfetch = partial_memory_read (addr, bufptr, len * width, &errcode)
 	/ width;
@@ -1243,8 +1244,6 @@ read_string (CORE_ADDR addr, int len, int width, unsigned int fetchlimit,
     {
       unsigned long bufsize = 0;
 
-      *buffer = NULL;
-
       do
 	{
 	  QUIT;
@@ -1253,13 +1252,9 @@ read_string (CORE_ADDR addr, int len, int width, unsigned int fetchlimit,
 	  if (*buffer == NULL)
 	    *buffer = (gdb_byte *) xmalloc (nfetch * width);
 	  else
-	    {
-	      discard_cleanups (old_chain);
-	      *buffer = (gdb_byte *) xrealloc (*buffer,
-					       (nfetch + bufsize) * width);
-	    }
+	    *buffer = (gdb_byte *) xrealloc (*buffer,
+					     (nfetch + bufsize) * width);
 
-	  old_chain = make_cleanup (xfree, *buffer);
 	  bufptr = *buffer + bufsize * width;
 	  bufsize += nfetch;