[libcamera-devel] [PATCH] test: buffer_import: Initialize media_ and video_
Jacopo Mondi
jacopo at jmondi.org
Mon Sep 16 22:18:59 CEST 2019
When the vivid module used to test buffer importing is not loaded, the
test correctly bails out, but during the clean up procedure tries to
access the video_ fields, which, if not correctly initialized to
nullptr, contains random values and accessing it cause a segfault.
Fix this by initializing video_ field to nullptr to make sure it is
properly handled in cleanup().
Fixes: e1a5873701a9 ("test: camera: Add buffer import and mapping test")
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
Signed-off-by: Jacopo Mondi <jacopo at jmondi.org>
---
test/camera/buffer_import.cpp | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/test/camera/buffer_import.cpp b/test/camera/buffer_import.cpp
index 9364e3d1bc44..9cac19d8ce81 100644
--- a/test/camera/buffer_import.cpp
+++ b/test/camera/buffer_import.cpp
@@ -28,6 +28,11 @@ static constexpr unsigned int CAMERA_BUFFER_COUNT = 4;
class FrameSink
{
public:
+ FrameSink()
+ : video_(nullptr)
+ {
+ }
+
int init()
{
int ret;
--
2.23.0
More information about the libcamera-devel
mailing list