From dfbcfc3a426573bb04aa12dacb6bf137012a1443 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marisol=20Zeled=C3=B3n?= Date: Tue, 12 Oct 2021 18:12:16 -0600 Subject: [PATCH 1/3] Free memory in interpipesrc finalize function Co-authored-by: Michael Gruner --- gst/interpipe/gstinterpipesrc.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/gst/interpipe/gstinterpipesrc.c b/gst/interpipe/gstinterpipesrc.c index e5c7a46..0378d98 100644 --- a/gst/interpipe/gstinterpipesrc.c +++ b/gst/interpipe/gstinterpipesrc.c @@ -368,6 +368,11 @@ gst_inter_pipe_src_finalize (GObject * object) /* Chain up to the parent class */ G_OBJECT_CLASS (gst_inter_pipe_src_parent_class)->finalize (object); + + if (src->listen_to) { + g_free (src->listen_to); + src->listen_to = NULL; + } } /* GstBaseSrc Implementation*/ From 10b72c274b5735ea011bee6a9cf743309a2ee0cf Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marisol=20Zeled=C3=B3n?= Date: Wed, 13 Oct 2021 12:04:51 -0600 Subject: [PATCH 2/3] Bump project version --- configure.ac | 2 +- meson.build | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index ea6a54f..31e21d8 100644 --- a/configure.ac +++ b/configure.ac @@ -6,7 +6,7 @@ dnl please read gstreamer/docs/random/autotools before changing this file dnl initialize autoconf dnl releases only do -Wall, git and prerelease does -Werror too dnl use a three digit version number for releases, and four for git/pre -AC_INIT([GstInterpipe],[1.1.6], +AC_INIT([GstInterpipe],[1.1.7], [http://www.github.com/RidgeRun/gst-interpipe-1.0], [gst-interpipe], [http://developer.ridgerun.com/wiki/index.php?title=GstInterpipe]) diff --git a/meson.build b/meson.build index 46bbdb5..8de7fe1 100644 --- a/meson.build +++ b/meson.build @@ -1,5 +1,5 @@ project('gst-interpipes', 'c', - version : '1.1.6', + version : '1.1.7', meson_version : '>= 0.50',) gst_interpipes_version = meson.project_version() From d332b5d87c50a65536202ddff8644cf6ee154b8c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marisol=20Zeled=C3=B3n?= Date: Thu, 21 Oct 2021 16:11:46 -0600 Subject: [PATCH 3/3] Move the chain up to parent class call to be the last destructor call Co-authored-by: Michael Gruner --- gst/interpipe/gstinterpipesrc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/gst/interpipe/gstinterpipesrc.c b/gst/interpipe/gstinterpipesrc.c index 0378d98..9799d8c 100644 --- a/gst/interpipe/gstinterpipesrc.c +++ b/gst/interpipe/gstinterpipesrc.c @@ -366,13 +366,13 @@ gst_inter_pipe_src_finalize (GObject * object) g_queue_free_full (src->pending_serial_events, (GDestroyNotify) gst_event_unref); - /* Chain up to the parent class */ - G_OBJECT_CLASS (gst_inter_pipe_src_parent_class)->finalize (object); - if (src->listen_to) { g_free (src->listen_to); src->listen_to = NULL; } + + /* Chain up to the parent class */ + G_OBJECT_CLASS (gst_inter_pipe_src_parent_class)->finalize (object); } /* GstBaseSrc Implementation*/