From: msnyder <msnyder>
Date: Wed, 9 Mar 2011 03:58:52 +0000 (+0000)
Subject: 2011-02-27  Michael Snyder  <msnyder@vmware.com>
X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=369f7f8894ab17786b9f6b29c936bb3b3a5ae650;p=pf3gnuchains%2Fsourceware.git

2011-02-27  Michael Snyder  <msnyder@vmware.com>

	* target.c (read_whatever_is_readable): Fix memory leak.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 2a1f02a887..08bd2105c1 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,7 @@
 2011-03-08  Michael Snyder  <msnyder@vmware.com>
 
+	* target.c (read_whatever_is_readable): Fix memory leak.
+
 	* i386-tdep.c (i386_process_record): Document fall through.
 
 2011-03-08  Jan Kratochvil  <jan.kratochvil@redhat.com>
diff --git a/gdb/target.c b/gdb/target.c
index 0792707018..aa5992048d 100644
--- a/gdb/target.c
+++ b/gdb/target.c
@@ -1968,7 +1968,10 @@ read_whatever_is_readable (struct target_ops *ops,
 
   /* If we previously failed to read 1 byte, nothing can be done here.  */
   if (end - begin <= 1)
-    return;
+    {
+      xfree (buf);
+      return;
+    }
 
   /* Check that either first or the last byte is readable, and give up
      if not.  This heuristic is meant to permit reading accessible memory
@@ -1987,6 +1990,7 @@ read_whatever_is_readable (struct target_ops *ops,
     }
   else
     {
+      xfree (buf);
       return;
     }