rusage: properly free the queue memory
authorGuillaume Desmottes <guillaume.desmottes@collabora.co.uk>
Wed, 13 Apr 2016 09:38:05 +0000 (12:38 +0300)
committerSebastian Dröge <sebastian@centricular.com>
Wed, 13 Apr 2016 09:40:41 +0000 (12:40 +0300)
The queue is allocated as part of the tracer struct so we should not
use g_queue_free() to free it.

https://bugzilla.gnome.org/show_bug.cgi?id=764985

plugins/tracers/gstrusage.c

index c4110ec..9df6b69 100644 (file)
@@ -85,7 +85,8 @@ make_trace_values (GstClockTime window)
 static void
 free_trace_values (GstTraceValues * self)
 {
-  g_queue_free_full (&self->values, free_trace_value);
+  g_queue_foreach (&self->values, (GFunc) free_trace_value, NULL);
+  g_queue_clear (&self->values);
   g_slice_free (GstTraceValues, self);
 }