From 0f2e8cac8dd43544adad50bf42b0fda496ddb055 Mon Sep 17 00:00:00 2001 From: Jacopo Mondi Date: Mon, 16 Sep 2019 11:54:13 +0200 Subject: test: buffer_import: Initialize video_ to nullptr 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_ field, which, if not correctly initialized to nullptr contains random values and accessing it causes 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 Signed-off-by: Jacopo Mondi --- test/camera/buffer_import.cpp | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'test/camera') diff --git a/test/camera/buffer_import.cpp b/test/camera/buffer_import.cpp index 9364e3d1..9cac19d8 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; -- cgit v1.2.1