diff options
author | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2021-09-28 00:09:37 +0300 |
---|---|---|
committer | Laurent Pinchart <laurent.pinchart@ideasonboard.com> | 2021-09-28 00:43:57 +0300 |
commit | 496e4467d298ef17fc1254f0e4e999ffc36a9b37 (patch) | |
tree | 8f57a0770008104ff2cc72c1539ba834e9aa96ca /src/android/camera_worker.cpp | |
parent | 2d5a78f664c5e0068cd3f78ff4c501cf4f331d3c (diff) |
libcamera: control_serializer: Fix usage of uninitialized variable
The idMap variable may be used uninitialized in the
ControlSerializer::deserialize<ControlList>() function as reported by
gcc 11:
../../src/libcamera/control_serializer.cpp: In member function ‘T libcamera::ControlSerializer::deserialize(libcamera::ByteStreamBuffer&) [with T = libcamera::ControlList]’:
../../src/libcamera/control_serializer.cpp:609:33: error: ‘idMap’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
609 | ControlList ctrls(*idMap);
|
This is due to a missing default case in a switch/case. Fix it by adding
the default case.
Fixes: 6b1404fc4836 ("libcamera: control_serializer: Fix usage of uninitialized variable")
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Jacopo Mondi <jacopo@jmondi.org>
Diffstat (limited to 'src/android/camera_worker.cpp')
0 files changed, 0 insertions, 0 deletions