]> git.proxmox.com Git - mirror_edk2.git/commit
CryptoPkg/BaseCryptLib: Fix potential integer overflow issue.
authorLong Qin <qin.long@intel.com>
Wed, 24 Oct 2018 13:16:42 +0000 (21:16 +0800)
committerLong Qin <qin.long@intel.com>
Wed, 31 Oct 2018 03:07:53 +0000 (11:07 +0800)
commit269f3b51803685eb3f4f4cd4415dc833d375efba
tree887217b1784fdccad65d02cd3173bd2cb96126cf
parentbeabfd5800515e0a51fc2461671514adc0cef584
CryptoPkg/BaseCryptLib: Fix potential integer overflow issue.

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1275

The LookupFreeMemRegion() in RuntimeMemAllocate.c is used to look-up
free memory region for runtime resource allocation, which was designed
to support runtime authenticated variable service.
The ReqPages in this function is the required pages to be allocated,
which depends on the malloc() call in internal OpenSSL routines. The
direct offset subtractions on ReqPages may bring possible integer
overflow issue.

This patch is to add the extra parameter checks to remove this possible
overflow risk.

Cc: Ye Ting <ting.ye@intel.com>
Cc: Laszlo Ersek <lersek@redhat.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Long Qin <qin.long@intel.com>
Reviewed-by: Ye Ting <ting.ye@intel.com>
CryptoPkg/Library/BaseCryptLib/SysCall/RuntimeMemAllocation.c