diff options
| author | Guillaume Desmottes <guillaume.desmottes@collabora.co.uk> | 2016-04-13 12:38:05 +0300 |
|---|---|---|
| committer | Tim-Philipp Müller <tim@centricular.com> | 2016-10-17 19:52:19 +0300 |
| commit | d0ee909cdd4c6a7c4f21a181661e262c504a38da (patch) | |
| tree | ede13a85217c3b644d52dad8474a7017ee7dbb6b | |
| parent | 60f088026a9367da45a1f2b46269425fc2a6a857 (diff) | |
rusage: properly free the queue memory
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
| -rw-r--r-- | plugins/tracers/gstrusage.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/plugins/tracers/gstrusage.c b/plugins/tracers/gstrusage.c index 29878f348d..7a783ee49c 100644 --- a/plugins/tracers/gstrusage.c +++ b/plugins/tracers/gstrusage.c @@ -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); } |