Memory leak reading frame register during inferior event handling
authorJoel Brobecker <brobecker@adacore.com>
Fri, 13 Feb 2015 10:57:29 +0000 (11:57 +0100)
committerJoel Brobecker <brobecker@adacore.com>
Wed, 20 May 2015 07:37:55 +0000 (09:37 +0200)
When using a conditional breakpoint where the condition evaluated
to false a large number of times before the program stopped,
a user reported that GDB's memory consumption was growing very
quickly until it ran out of memory.

The problem was tracked down to temporary struct values being created
each time the program stops and handles an inferior event.  Because
the breakpoint condition usually evaluates to false, there can be
a fairly large number of such events to be handled before we eventually
return the prompt to the user (which is when we would normally purge
such values).

This patch fixes the issue by making sure that handle_inferior_event
releases all new values created during its execution.

gdb/ChangeLog:

        * infrun.c (handle_inferior_event_1): Renames handle_inferior_event.
        (handle_inferior_event): New function.

gdb/ChangeLog
gdb/infrun.c

index 291702295aa914b9e99307fad8489bdadf362cb0..28d6b7188b25da9c35685256bba9fe2554dcd689 100644 (file)
@@ -1,3 +1,8 @@
+2015-05-20  Joel Brobecker  <brobecker@adacore.com>
+
+       * infrun.c (handle_inferior_event_1): Renames handle_inferior_event.
+       (handle_inferior_event): New function.
+
 2015-05-20  Joel Brobecker  <brobecker@adacore.com>
 
        * ada-lang.c (to_fixed_array_type): Rename local variable
index 71cf208e41fe63a809c1cfbcd620eaf362385488..2f6bc41d0b70b6479bdd1c7669c8321c230e45f4 100644 (file)
@@ -3680,7 +3680,7 @@ get_inferior_stop_soon (ptid_t ptid)
    once).  */
 
 static void
-handle_inferior_event (struct execution_control_state *ecs)
+handle_inferior_event_1 (struct execution_control_state *ecs)
 {
   enum stop_kind stop_soon;
 
@@ -4202,6 +4202,22 @@ Cannot fill $_exitsignal with the correct signal number.\n"));
     }
 }
 
+/* A wrapper around handle_inferior_event_1, which also makes sure
+   that all temporary struct value objects that were created during
+   the handling of the event get deleted at the end.  */
+
+static void
+handle_inferior_event (struct execution_control_state *ecs)
+{
+  struct value *mark = value_mark ();
+
+  handle_inferior_event_1 (ecs);
+  /* Purge all temporary values created during the event handling,
+     as it could be a long time before we return to the command level
+     where such values would otherwise be purged.  */
+  value_free_to_mark (mark);
+}
+
 /* Come here when the program has stopped with a signal.  */
 
 static void