diff options
| author | Christian Rauch <Rauch.Christian@gmx.de> | 2022-09-03 23:33:29 +0200 |
|---|---|---|
| committer | Paul Elder <paul.elder@ideasonboard.com> | 2022-09-12 18:28:59 +0900 |
| commit | cbc2be34ed9e47f5b17d0955bf3496d735359795 (patch) | |
| tree | c21bdfa1c40dc8cbcfdff6506df6c90234099ae1 /test/serialization/serialization_test.cpp | |
| parent | 560ceb1ea854b1cb0c7d91efffa0e7aabbed2328 (diff) | |
libcamera: control_serializer: store/load all ControlValue types
The min/max/def ControlValue of a ControlInfo can take arbitrary types that
are different from each other and different from the ControlId type.
The serialiser serialises these ControlValue separately by their type but
does not store the type. The deserialiser assumes that ControlValue types
match the ControlId type. If this is not the case, deserialisation will try
to deserialise values of the wrong type.
Fix this by serialising each of the min/max/def ControlValue's ControlType
and storing it just before the serialised ControlValue.
Fixes: https://bugs.libcamera.org/show_bug.cgi?id=137
Signed-off-by: Christian Rauch <Rauch.Christian@gmx.de>
Tested-by: Paul Elder <paul.elder@ideasonboard.com>
Reviewed-by: Paul Elder <paul.elder@ideasonboard.com>
Reviewed-by: Umang Jain <umang.jain@ideasonboard.com>
Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
Diffstat (limited to 'test/serialization/serialization_test.cpp')
0 files changed, 0 insertions, 0 deletions