From: Simon Marchi <simon.marchi@ericsson.com>
Date: Fri, 2 Jun 2017 21:24:21 +0000 (+0200)
Subject: Use delete instead of xfree for gdb_timer
X-Git-Tag: binutils-2_29~394
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0e05cf3a61e1996829359e1c32eedb6820930331;p=external%2Fbinutils.git

Use delete instead of xfree for gdb_timer

gdb_timer objects are new'ed in create_timer, but xfree'd in
poll_timers.  Use delete instead.

gdb/ChangeLog:

	* event-loop.c (poll_timers): Unallocate timer using delete
	instead of xfree.
---

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 0338387..ed063fc 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,8 @@
+2017-06-02  Simon Marchi  <simon.marchi@ericsson.com>
+
+	* event-loop.c (poll_timers): Unallocate timer using delete
+	instead of xfree.
+
 2017-06-02  Simon Marchi  <simon.marchi@polymtl.ca>
 
 	* breakpoint.h (struct breakpoint_ops) <dtor>: Remove.
diff --git a/gdb/event-loop.c b/gdb/event-loop.c
index 4b3ddb8..7304ca7 100644
--- a/gdb/event-loop.c
+++ b/gdb/event-loop.c
@@ -1270,7 +1270,7 @@ poll_timers (void)
       /* Delete the timer before calling the callback, not after, in
 	 case the callback itself decides to try deleting the timer
 	 too.  */
-      xfree (timer_ptr);
+      delete timer_ptr;
 
       /* Call the procedure associated with that timer.  */
       (proc) (client_data);