ArduPilot / MethodicConfigurator / 23688777357

94%

push

github

fix(params): guard against inf/nan from safe_evaluate reaching FC upload

safe_evaluate() can produce non-finite values (inf, nan) from
expressions like exp(1000), log(0), or conditional branches that
evaluate to infinity. These values bypass the existing ZeroDivisionError
and ValueError exception handlers and flow unchecked through
set_forced_or_derived_value() to the flight controller.

Add an isfinite() check on the evaluated result before creating the Par
object. Non-finite values are now rejected with a descriptive error
message, consistent with the existing validation pattern for forced vs
derived parameters.

Signed-off-by: Yash Goel <yashhzd@users.noreply.github.com>

1685 of 1752 branches covered (96.18%)

11094 of 11822 relevant lines covered (93.84%)

3.74 hits per line