[v2] ipa: simple: Initialize ccmEnabled to false
diff mbox series

Message ID 20250401141455.492231-1-stanislaw.gruszka@linux.intel.com
State Superseded
Headers show
Series
  • [v2] ipa: simple: Initialize ccmEnabled to false
Related show

Commit Message

Stanislaw Gruszka April 1, 2025, 2:14 p.m. UTC
ccmEnabled variable is not initialized by default, which results in
usage of CCM when the algorithm itself is not enabled and configured.

The bug manifests itself as seldom reproducible corrupted video stream.
Fix by initialize the variable in IPAContext class initialization list.

Reviewed-by: Milan Zamazal <mzamazal@redhat.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Fixes: ac3068655643 ("libcamera: software_isp: Track whether CCM is enabled")
Signed-off-by: Stanislaw Gruszka <stanislaw.gruszka@linux.intel.com>
---
v1 -> v2:
 - initialize on declaration 
 - fix spelling

 src/ipa/simple/ipa_context.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Stanislaw Gruszka April 1, 2025, 2:18 p.m. UTC | #1
On Tue, Apr 01, 2025 at 04:14:55PM +0200, Stanislaw Gruszka wrote:
> ccmEnabled variable is not initialized by default, which results in
> usage of CCM when the algorithm itself is not enabled and configured.
> 
> The bug manifests itself as seldom reproducible corrupted video stream.
> Fix by initialize the variable in IPAContext class initialization list.

Forgot to fix "initialization list" here,  will post v3

> Reviewed-by: Milan Zamazal <mzamazal@redhat.com>
> Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> Fixes: ac3068655643 ("libcamera: software_isp: Track whether CCM is enabled")
> Signed-off-by: Stanislaw Gruszka <stanislaw.gruszka@linux.intel.com>
> ---
> v1 -> v2:
>  - initialize on declaration 
>  - fix spelling
> 
>  src/ipa/simple/ipa_context.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/ipa/simple/ipa_context.h b/src/ipa/simple/ipa_context.h
> index 7dc2cd7ae828..51522720ab98 100644
> --- a/src/ipa/simple/ipa_context.h
> +++ b/src/ipa/simple/ipa_context.h
> @@ -91,7 +91,7 @@ struct IPAContext {
>  	IPAActiveState activeState;
>  	FCQueue<IPAFrameContext> frameContexts;
>  	ControlInfoMap::Map ctrlMap;
> -	bool ccmEnabled;
> +	bool ccmEnabled = false;
>  };
>  
>  } /* namespace ipa::soft */
> -- 
> 2.43.0
>

Patch
diff mbox series

diff --git a/src/ipa/simple/ipa_context.h b/src/ipa/simple/ipa_context.h
index 7dc2cd7ae828..51522720ab98 100644
--- a/src/ipa/simple/ipa_context.h
+++ b/src/ipa/simple/ipa_context.h
@@ -91,7 +91,7 @@  struct IPAContext {
 	IPAActiveState activeState;
 	FCQueue<IPAFrameContext> frameContexts;
 	ControlInfoMap::Map ctrlMap;
-	bool ccmEnabled;
+	bool ccmEnabled = false;
 };
 
 } /* namespace ipa::soft */