summaryrefslogtreecommitdiff
path: root/src/ipa
diff options
context:
space:
mode:
authorKieran Bingham <kieran.bingham@ideasonboard.com>2019-07-18 05:59:22 +0100
committerKieran Bingham <kieran.bingham@ideasonboard.com>2019-08-01 16:20:27 +0100
commit9ecc60e10c96d1abd7d68899831e3ab143a16520 (patch)
treecf3c4314a1bdbc992b2108beee097ce5865a1f09 /src/ipa
parent1a6b80e4a0fa7b2be8fa6c4a25a33f92ce66f563 (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 'src/ipa')
0 files changed, 0 insertions, 0 deletions