[libcamera-devel] [PATCH v2 1/3] test: gstreamer_single_stream_test: Fix memory leak

[Vedant Paranjape]

The test hold a valid reference to convert0_ and sink0_ but
not released. This results in a memory leak and can be checked
via valgrind. Drop the references with test cleanup() virtual
function.

Valgrind log (glib and gst suppression files were used):
==345380== LEAK SUMMARY:
==345380==    definitely lost: 1,688 bytes in 2 blocks
==345380==    indirectly lost: 7,069 bytes in 42 blocks

The patch fixes the leaks reported by valgrind above to:
==348870== LEAK SUMMARY:
==348870==    definitely lost: 0 bytes in 0 blocks
==348870==    indirectly lost: 0 bytes in 0 blocks

Signed-off-by: Vedant Paranjape <vedantparanjape160201 at gmail.com>
Signed-off-by: Umang Jain <umang.jain at ideasonboard.com>
Reviewed-by: Nicolas Dufresne <nicolas.dufresne at collabora.com>
---
 test/gstreamer/gstreamer_single_stream_test.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/test/gstreamer/gstreamer_single_stream_test.cpp b/test/gstreamer/gstreamer_single_stream_test.cpp
index 7292f3280617..2b561d6bf9f2 100644
--- a/test/gstreamer/gstreamer_single_stream_test.cpp
+++ b/test/gstreamer/gstreamer_single_stream_test.cpp
@@ -72,6 +72,12 @@ protected:
 		return TestPass;
 	}
 
+	void cleanup() override
+	{
+		g_clear_object(&convert0_);
+		g_clear_object(&sink0_);
+	}
+
 private:
 	GstElement *convert0_;
 	GstElement *sink0_;
-- 
2.25.1