[libcamera-devel] Fwd: New Defects reported by Coverity Scan for libcamera

[Umang Jain]

Hi Kieran

On 8/19/21 2:05 PM, Kieran Bingham wrote:
> Hi Umang,
>
> Coverity just sent me this warning from the daily build.
yes, it sent me too,
>
> Could you check it please?
>
> It looks like we set allocated_ in init() so I don't think this is
> actually a bug.


I have marked as false positive, commenting that we actually set the 
allocated_ to true in init().


>
> --
> Kieran
>
>
> -------- Forwarded Message --------
> Subject: New Defects reported by Coverity Scan for libcamera
> Date: Thu, 19 Aug 2021 08:21:19 +0000 (UTC)
> From: scan-admin at coverity.com
> To: kieran.bingham at ideasonboard.com
>
> Hi,
>
> Please find the latest report on new defect(s) introduced to libcamera
> found with Coverity Scan.
>
> 1 new defect(s) introduced to libcamera found with Coverity Scan.
>
>
> New defect(s) Reported-by: Coverity Scan
> Showing 1 of 1 defect(s)
>
>
> ** CID 355558:  Uninitialized members  (UNINIT_CTOR)
> /home/kbingham/iob/libcamera/ci/libcamera-ci/src/libcamera/test/camera/camera_reconfigure.cpp:
> 35 in <unnamed>::CameraReconfigure::CameraReconfigure()()
>
>
> ________________________________________________________________________________________________________
> *** CID 355558:  Uninitialized members  (UNINIT_CTOR)
> /home/kbingham/iob/libcamera/ci/libcamera-ci/src/libcamera/test/camera/camera_reconfigure.cpp:
> 35 in <unnamed>::CameraReconfigure::CameraReconfigure()()
> 29     {
> 30     public:
> 31     	/* Initialize CameraTest with isolated IPA */
> 32     	CameraReconfigure()
> 33     		: CameraTest(kCamId_, true)
> 34     	{
>>>>      CID 355558:  Uninitialized members  (UNINIT_CTOR)
>>>>      Non-static class member "allocated_" is not initialized in this constructor nor in any functions that it calls.
> 35     	}
> 36     37     private:
> 38     	static constexpr const char *kCamId_ = "platform/vimc.0 Sensor B";
> 39     	static constexpr const char *kIpaProxyName_ = "vimc_ipa_proxy";
> 40     	static constexpr unsigned int kNumOfReconfigures_ = 10;
>
>
> ________________________________________________________________________________________________________
> To view the defects in Coverity Scan visit,
> https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P0qcxCbhZ31OYv50yrU0jHZztxHSfchEdEv-2BUERKUFDFlzchxJzxDB2S7OB7lr-2Fth-2F9ZTRF1ITl33bQQto-3D5Nee_qXXIY1fr7HINeQlTFVXABgVX8iKf8Pc2W6xJCOtsiCw64f2YdDCRdgu4-2FkWCuqEXuxOh9eXXcEsug0LJs94H99fa61WirxJ5dwX551C0c-2FX1YqaKrTXHG-2Faur-2BSR5aE8BZLFLGquo5g-2FfLfHfXLFVkli9ajmCFQV792XcHPIB3ZZmDd7oQ4NOMRZ-2BkN8YJkEeu9ZUlRlF9D2nPvQH6PcR2hee3TIzpF0nQef-2F6vke4Q-3D
>