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 /include | |
| 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 'include')
0 files changed, 0 insertions, 0 deletions