diff options
| author | Kieran Bingham <kieran.bingham@ideasonboard.com> | 2019-07-18 05:59:22 +0100 |
|---|---|---|
| committer | Kieran Bingham <kieran.bingham@ideasonboard.com> | 2019-08-01 16:20:27 +0100 |
| commit | 9ecc60e10c96d1abd7d68899831e3ab143a16520 (patch) | |
| tree | cf3c4314a1bdbc992b2108beee097ce5865a1f09 /test/media_device/media_device_test.cpp | |
| parent | 1a6b80e4a0fa7b2be8fa6c4a25a33f92ce66f563 (diff) | |
libcamera: ipa_module: prevent uninitialised access
The IPAModule::loadIPAModuleInfo() function includes a *data pointer
which is used as a null-pointer comparison in the error path with a
conditional statement of "if (ret || !data)".
The data variable is not initialised, and a single error path evaluates
this as "if (true || uninitialised)".
Whilst this error path does not incorrectly utilise the uninitialised
data, as the ret evaluates to true already, it does leave a statement
which includes an uninitialised variable.
Help the static analysers by initialising the data variable when it is
defined.
Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Diffstat (limited to 'test/media_device/media_device_test.cpp')
0 files changed, 0 insertions, 0 deletions