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

Thanks

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#56771): https://edk2.groups.io/g/devel/message/56771
Mute This Topic: https://groups.io/mt/72673992/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to