A couple of thoughts. <br />1. I would suggest that ASSERT should not be the only protection for an invalid operation as ASSERT is usually disabled on release builds.  <br />2. We do have a library to make this more explicit and common.  <a href="https://github.com/tianocore/edk2/blob/master/MdePkg/Include/Library/SafeIntLib.h#L548">https://github.com/tianocore/edk2/blob/master/MdePkg/Include/Library/SafeIntLib.h#L548</a><br /><br />Thanks

<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/56771">View/Reply Online (#56771)</a> |


  


|


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

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



<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>