<div>The function S3ResumeExecuteBootScript at the point of preparing data</div>
<div>for return back makes a call to AllocatePool and stores the return</div>
<div>value in PeiS3ResumeState. Thereafter it does a check</div>
<div> </div>
<div>    if (PeiS3ResumeState == NULL) {</div>
<div> </div>
<div>The if block further has ASSERT (FALSE); If PeiS3ResumeState is NULL,</div>
<div>then the if check passes and ASSERT hits, but this is applicable only</div>
<div>in DEBUG mode. In Release mode, the code comes out of this if block</div>
<div>and will dereference "PeiS3ResumeState" which will lead to CRASH.</div>
<div> </div>
<div>Hence, for safety do not let the flow come out of the above if block.</div>
<div> </div>
<div>REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4227</div>
<div>Signed-off-by: Ranbir Singh <Ranbir.Singh3@Dell.com></div>
<div>---</div>
<div> UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c | 5 +++++</div>
<div> 1 file changed, 5 insertions(+)</div>
<div> </div>
<div>diff --git a/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c b/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c</div>
<div>index e82f179569..b6b2e1f99c 100644</div>
<div>--- a/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c</div>
<div>+++ b/UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c</div>
<div>@@ -884,6 +884,11 @@ S3ResumeExecuteBootScript (</div>
<div>       (EFI_SOFTWARE_PEI_MODULE | EFI_SW_PEI_EC_S3_RESUME_FAILED)</div>
<div>       );</div>
<div>     ASSERT (FALSE);</div>
<div>+    //</div>
<div>+    // Never run to here</div>
<div>+    //</div>
<div>+    CpuDeadLoop ();</div>
<div>+    return;</div>
<div>   }</div>
<div> </div>
<div>   DEBUG ((DEBUG_INFO, "PeiS3ResumeState - %x\r\n", PeiS3ResumeState));</div>
<div>--</div>
<div>2.36.1.windows.1</div>


<div width="1" style="color:white;clear:both">_._,_._,_</div>
<hr>


Groups.io Links:<p>


  
    You receive all messages sent to this group.
  
  


<p>
<a target="_blank" href="https://edk2.groups.io/g/devel/message/97929">View/Reply Online (#97929)</a> |


  

|

  <a target="_blank" href="https://groups.io/mt/96047568/1813853">Mute This Topic</a>

| <a href="https://edk2.groups.io/g/devel/post">New Topic</a><br>




<a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> |
<a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> |

<a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a>

 [edk2-devel-archive@redhat.com]<br>
<div width="1" style="color:white;clear:both">_._,_._,_</div>