[libcamera-devel] [PATCH 3/4] cam: fix use of uninitialized field

Niklas Söderlund niklas.soderlund at ragnatech.se
Wed Oct 7 18:05:09 CEST 2020


Hi Tomi,

Thanks for your work.

On 2020-10-07 12:22:38 +0300, Tomi Valkeinen wrote:
> Initialize last_ to 0 to avoid uninitialized use.
> 
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen at iki.fi>

Reviewed-by: Niklas Söderlund <niklas.soderlund at ragnatech.se>

> ---
>  src/cam/capture.cpp | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/cam/capture.cpp b/src/cam/capture.cpp
> index 5510c00..995614a 100644
> --- a/src/cam/capture.cpp
> +++ b/src/cam/capture.cpp
> @@ -17,7 +17,7 @@ using namespace libcamera;
>  
>  Capture::Capture(std::shared_ptr<Camera> camera, CameraConfiguration *config,
>  		 EventLoop *loop)
> -	: camera_(camera), config_(config), writer_(nullptr), loop_(loop),
> +	: camera_(camera), config_(config), writer_(nullptr), last_(0), loop_(loop),
>  	  captureCount_(0), captureLimit_(0)
>  {
>  }
> -- 
> Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
> Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki
> 
> _______________________________________________
> libcamera-devel mailing list
> libcamera-devel at lists.libcamera.org
> https://lists.libcamera.org/listinfo/libcamera-devel

-- 
Regards,
Niklas Söderlund


More information about the libcamera-devel mailing list