<div dir="ltr"><div dir="ltr"><div>Hi Benjamin,</div><div><br></div><div>Thank you for this fix!</div><div><br></div></div><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Fri, 23 May 2025 at 10:57, Benjamin Mugnier <<a href="mailto:benjamin.mugnier@foss.st.com">benjamin.mugnier@foss.st.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Even without AGC definition in the tuning file, the application would<br>
still dereference agc unconditionally, leading to a segmentation fault<br>
if AGC is absent.<br>
This is relevant for sensors already providing AGC/AEC by themselves.<br>
Check if AGC is present prior to setting maximum exposure time.<br>
<br>
Signed-off-by: Benjamin Mugnier <<a href="mailto:benjamin.mugnier@foss.st.com" target="_blank">benjamin.mugnier@foss.st.com</a>><br></blockquote><div><br></div><div>Reviewed-by: Naushir Patuck <<a href="mailto:naush@raspberrypi.com">naush@raspberrypi.com</a>></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
 src/ipa/rpi/common/ipa_base.cpp | 3 ++-<br>
 1 file changed, 2 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/ipa/rpi/common/ipa_base.cpp b/src/ipa/rpi/common/ipa_base.cpp<br>
index e0a93daa9db229bb2829803fffc6b2f6c8f11061..e0f8b7e782f4340a069a2d7b9cf9dee3141c72df 100644<br>
--- a/src/ipa/rpi/common/ipa_base.cpp<br>
+++ b/src/ipa/rpi/common/ipa_base.cpp<br>
@@ -1563,7 +1563,8 @@ void IpaBase::applyFrameDurations(Duration minFrameDuration, Duration maxFrameDu<br>
<br>
        RPiController::AgcAlgorithm *agc = dynamic_cast<RPiController::AgcAlgorithm *>(<br>
                controller_.getAlgorithm("agc"));<br>
-       agc->setMaxExposureTime(maxExposureTime);<br>
+       if (agc)<br>
+               agc->setMaxExposureTime(maxExposureTime);<br>
 }<br>
<br>
 void IpaBase::applyAGC(const struct AgcStatus *agcStatus, ControlList &ctrls)<br>
<br>
---<br>
base-commit: ad5326c926831fe7a943d10fd800de43e596f171<br>
change-id: 20250523-fix_segfault_agc-5f2735edec5f<br>
<br>
Best regards,<br>
-- <br>
Benjamin Mugnier <<a href="mailto:benjamin.mugnier@foss.st.com" target="_blank">benjamin.mugnier@foss.st.com</a>><br>
<br>
</blockquote></div></div>