Skip to content

Conversation

@Javagedes
Copy link
Contributor

Description

The local variable 'WillReturn' was being used without prior initialization in some code paths.
This patch ensures that 'WillReturn' is properly initialized to prevent undefined behavior.

Cherry-pick from EDKII tianocore/edk2@30b6d08
Cherry-pick from dev/202405 36f763d

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?

How This Was Tested

Cherry-pick from EDKII tianocore/edk2@30b6d08

Integration Instructions

Cherry-pick from EDKII tianocore/edk2@30b6d08

@github-actions github-actions bot added the impact:non-functional Does not have a functional impact label Sep 23, 2024
The local variable 'WillReturn' was being used without prior
initialization in some code paths.
This patch ensures that 'WillReturn' is properly initialized
to prevent undefined behavior.

Signed-off-by: Zhiguang Liu <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Sep 24, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 1.59%. Comparing base (85f4938) to head (07944e7).

Files with missing lines Patch % Lines
MdeModulePkg/Core/PiSmmCore/Smi.c 0.00% 1 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                 @@
##           release/202405    #1159      +/-   ##
==================================================
- Coverage            1.59%    1.59%   -0.01%     
==================================================
  Files                1448     1448              
  Lines              362500   362501       +1     
  Branches             5632     5632              
==================================================
  Hits                 5776     5776              
- Misses             356617   356618       +1     
  Partials              107      107              
Flag Coverage Δ
MdeModulePkg 0.68% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Javagedes Javagedes enabled auto-merge (rebase) September 24, 2024 00:33
@Javagedes Javagedes merged commit f254700 into microsoft:release/202405 Sep 24, 2024
31 checks passed
@kuqin12 kuqin12 added type:bug Something isn't working and removed impact:non-functional Does not have a functional impact labels Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants