From 24ca846a27d3b5f3e371dcd039df6ee74dd113c1 Mon Sep 17 00:00:00 2001
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Date: Mon, 5 Jul 2021 07:19:00 +0300
Subject: cam: camera_session: Use std::unique_ptr<> to manage class members

Store the BufferWriter and FrameBufferAllocator pointers in
std::unique_ptr<> instances to simplify memory management and avoid
leaks.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
---
 src/cam/camera_session.h | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

(limited to 'src/cam/camera_session.h')

diff --git a/src/cam/camera_session.h b/src/cam/camera_session.h
index 5131cfd4..31e8d6db 100644
--- a/src/cam/camera_session.h
+++ b/src/cam/camera_session.h
@@ -44,7 +44,7 @@ private:
 	libcamera::CameraConfiguration *config_;
 
 	std::map<const libcamera::Stream *, std::string> streamName_;
-	BufferWriter *writer_;
+	std::unique_ptr<BufferWriter> writer_;
 	uint64_t last_;
 
 	unsigned int queueCount_;
@@ -52,7 +52,7 @@ private:
 	unsigned int captureLimit_;
 	bool printMetadata_;
 
-	libcamera::FrameBufferAllocator *allocator_;
+	std::unique_ptr<libcamera::FrameBufferAllocator> allocator_;
 	std::vector<std::unique_ptr<libcamera::Request>> requests_;
 };
 
-- 
cgit v1.2.1