Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dm: fix possible buffer overflow in 'acrn_load_elf()' #1613

Merged
merged 1 commit into from Oct 27, 2018

Conversation

yonghuah
Copy link
Contributor

Will attempt to access element 2048..2049 of
array "ptr32" if below conditions are both true:

  1. ptr32[i] == MULTIBOOT_HEAD_MAGIC
  2. (i == (ELF_BUF_LEN/4) - 1)

Tracked-On: #1252
Signed-off-by: Yonghua Huang yonghua.huang@intel.com
Acked-by: Fengwei Yin fengwei.yin@intel.com

@ghost
Copy link

ghost commented Oct 26, 2018

1613
The PR does not touch any hypervisor source file. Skip coding guideline checks.

@ClaudZhang1995
Copy link

OK to Verify

@ghost
Copy link

ghost commented Oct 26, 2018

1613
The PR does not touch any hypervisor source file. Skip coding guideline checks.

@ClaudZhang1995
Copy link

Ready to merge

  Will attempt to access element 2048..2049 of
  array "ptr32" if below conditions are both true:
  1) ptr32[i] == MULTIBOOT_HEAD_MAGIC
  2) (i == (ELF_BUF_LEN/4) - 1)

Tracked-On: projectacrn#1252
Signed-off-by: Yonghua Huang <yonghua.huang@intel.com>
Acked-by: Fengwei Yin <fengwei.yin@intel.com>
@NanlinXie NanlinXie merged commit 8860af3 into projectacrn:master Oct 27, 2018
@yonghuah yonghuah deleted the dm_fix_BO_sw_load_elf branch June 20, 2019 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants