summaryrefslogtreecommitdiff
path: root/ShellPkg
diff options
context:
space:
mode:
authorSami Mujawar <sami.mujawar@arm.com>2019-11-18 16:37:30 +0000
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2020-03-31 21:22:39 +0000
commitdd7523b5b123de6f0730f2f2abb207f2a5c1ccd4 (patch)
tree3d210c0667d5c2affd8cc085ba504131328e8852 /ShellPkg
parent9e7a063b07cb7c4ae803f0145d90b0227b85afd7 (diff)
downloadedk2-dd7523b5b123de6f0730f2f2abb207f2a5c1ccd4.zip
edk2-dd7523b5b123de6f0730f2f2abb207f2a5c1ccd4.tar.gz
edk2-dd7523b5b123de6f0730f2f2abb207f2a5c1ccd4.tar.bz2
DynamicTablesPkg: SRAT: Fix uninitialized memory usage
On enabling the /analyse option the VS2017 compiler reports: warning C6001: Using uninitialized memory. This warning is reported for the Status variable in AddGenericInitiatorAffinity() as it is not initialised to a default value. This condition is only valid if GenInitAffCount is equal to 0. Since GenInitAffCount is already checked in BuildSratTable() this condition can never happen. The value of the Status variable is returned in failure cases from appropriate locations in AddGenericInitiatorAffinity(). The only case where Status value is being used un-initialised is the return statement at the end of AddGenericInitiatorAffinity(). Therefore, to fix this issue EFI_SUCCESS can be safely returned instead of returning the Status variable at the end of the function. Signed-off-by: Sami Mujawar <sami.mujawar@arm.com> Reviewed-by: Alexei Fedorov <Alexei.Fedorov@arm.com>
Diffstat (limited to 'ShellPkg')
0 files changed, 0 insertions, 0 deletions