mirrors/pipewire
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pipewire/pipewire.git synced 2025-11-01 22:58:50 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

libcamera: Handle missing control info default values

Browse source 
Libcamera does not always provide default values for all control infos, see
https://git.libcamera.org/libcamera/libcamera.git/tree/src/ipa/rkisp1/rkisp1.cpp?h=v0.0.2#n98

Sanitize those values, avoiding crashes.
This commit is contained in:
Robert Mader 2022-11-25 01:46:14 +01:00 • committed by Wim Taymans
parent 0da1a3ba82
commit b952d52b59
1 changed files with 35 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

50
spa/plugins/libcamera/libcamera-utils.cpp
View file

@ -500,28 +500,48 @@ next:
0);
switch (ctrl_id->type()) {
case ControlTypeBool:
case ControlTypeBool: {
bool def;
if (ctrl_info.def().isNone())
def = ctrl_info.min().get<bool>();
else
def = ctrl_info.def().get<bool>();
spa_pod_builder_add(&b,
SPA_PROP_INFO_type, SPA_POD_CHOICE_Bool(
(bool)ctrl_info.def().get<bool>()),
0);
break;
case ControlTypeFloat:
SPA_PROP_INFO_type, SPA_POD_CHOICE_Bool(
def),
0);
} break;
case ControlTypeFloat: {
float min = ctrl_info.min().get<float>();
float max = ctrl_info.max().get<float>();
float def;
if (ctrl_info.def().isNone())
def = (min + max) / 2;
else
def = ctrl_info.def().get<float>();
spa_pod_builder_add(&b,
SPA_PROP_INFO_type, SPA_POD_CHOICE_RANGE_Float(
(float)ctrl_info.def().get<float>(),
(float)ctrl_info.min().get<float>(),
(float)ctrl_info.max().get<float>()),
def, min, max),
0);
break;
case ControlTypeInteger32:
} break;
case ControlTypeInteger32: {
int32_t min = ctrl_info.min().get<int32_t>();
int32_t max = ctrl_info.max().get<int32_t>();
int32_t def;
if (ctrl_info.def().isNone())
def = (min + max) / 2;
else
def = ctrl_info.def().get<int32_t>();
spa_pod_builder_add(&b,
SPA_PROP_INFO_type, SPA_POD_CHOICE_RANGE_Int(
(int32_t)ctrl_info.def().get<int32_t>(),
(int32_t)ctrl_info.min().get<int32_t>(),
(int32_t)ctrl_info.max().get<int32_t>()),
def, min, max),
0);
break;
} break;
default:
goto next;
}