[libcamera-devel] [PATCH v2 2/4] simple-cam: Early return if no cameras are found on the system.

Laurent Pinchart laurent.pinchart at ideasonboard.com
Tue Dec 1 19:19:04 CET 2020


Hi Umang,

Thank you for the patch.

s/\.$// on the subject line.

On Tue, Dec 01, 2020 at 11:13:12PM +0530, Umang Jain wrote:
> Failing to do so, the codepath will segfault while trying to acquire
> a non-existent camera.
> 
> Signed-off-by: Umang Jain <email at uajain.com>
> Reviewed-by: Kieran Bingham <kieran.bingham at ideasonboard.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

> ---
>  simple-cam.cpp | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/simple-cam.cpp b/simple-cam.cpp
> index e88fb85..f844ff4 100644
> --- a/simple-cam.cpp
> +++ b/simple-cam.cpp
> @@ -307,6 +307,13 @@ int main()
>  	for (std::unique_ptr<Request> &request : requests)
>  		camera->queueRequest(request.get());
>  
> +	if(!cm->cameras().size()) {

Missing space after if. Doesn't checkstyle.py warn you ? Ah, no, we have
no checkstyle.py for simple-cam :-)

> +		std::cout << "No cameras were identified on the system."
> +			  << std::endl;
> +		cm->stop();
> +		return EXIT_FAILURE;
> +	}
> +
>  	/*
>  	 * --------------------------------------------------------------------
>  	 * Run an EventLoop

-- 
Regards,

Laurent Pinchart


More information about the libcamera-devel mailing list