[libcamera-devel] [PATCH] libcamera: media_device: Zero media graph arrays when querying topology

Laurent Pinchart laurent.pinchart at ideasonboard.com
Wed Jan 2 12:31:22 CET 2019


This silences valgrind that otherwise warns about usage of uninitialized
values. While not strictly required as the kernel should fill the whole
arrays in MEDIA_IOC_G_TOPOLOGY, the extra cost, in a non-critical path,
is negligible compared to the ability to run without valgrind warnings.

Signed-off-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
---
 src/libcamera/media_device.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/libcamera/media_device.cpp b/src/libcamera/media_device.cpp
index 4ff9ffe1041d..cc307dac1c33 100644
--- a/src/libcamera/media_device.cpp
+++ b/src/libcamera/media_device.cpp
@@ -240,10 +240,10 @@ int MediaDevice::populate()
 		delete[] pads;
 		delete[] interfaces;
 
-		ents = new media_v2_entity[topology.num_entities];
-		links = new media_v2_link[topology.num_links];
-		pads = new media_v2_pad[topology.num_pads];
-		interfaces = new media_v2_interface[topology.num_interfaces];
+		ents = new media_v2_entity[topology.num_entities]();
+		links = new media_v2_link[topology.num_links]();
+		pads = new media_v2_pad[topology.num_pads]();
+		interfaces = new media_v2_interface[topology.num_interfaces]();
 
 		version = topology.topology_version;
 	}
-- 
Regards,

Laurent Pinchart



More information about the libcamera-devel mailing list