summaryrefslogtreecommitdiff
path: root/BaseTools/Source/Python/CommonDataClass/CommonClass.py
diff options
context:
space:
mode:
authorSami Mujawar <sami.mujawar@arm.com>2019-11-18 19:37:30 +0300
committermergify[bot] <37929162+mergify[bot]@users.noreply.github.com>2020-04-01 00:22:39 +0300
commitdd7523b5b123de6f0730f2f2abb207f2a5c1ccd4 (patch)
tree3d210c0667d5c2affd8cc085ba504131328e8852 /BaseTools/Source/Python/CommonDataClass/CommonClass.py
parent9e7a063b07cb7c4ae803f0145d90b0227b85afd7 (diff)
downloadedk2-dd7523b5b123de6f0730f2f2abb207f2a5c1ccd4.tar.xz
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 'BaseTools/Source/Python/CommonDataClass/CommonClass.py')
0 files changed, 0 insertions, 0 deletions