sanchayanmaity/gem5
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

arm: Fix heap overflow issue in Neon64Load operation

Browse source 
This patch fixes an issue identified by ASAN where the Neon64Load
operation assumes the packet always contains 16 bytes.

Change-Id: If24a7e461d60cb80970dfbe61d923d7d56926698
Reviewed-by: Giacomo Gabrielli <giacomo.gabrielli@arm.com>
Reviewed-by: Curtis Dunham <curtis.dunham@arm.com>
This commit is contained in:
Andreas Hansson 2016-05-26 11:56:24 +01:00
parent 4a3e2156ac
commit 7dc5034ff2
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/arch/arm/isa/templates/neon64.isa
View file

@ -332,7 +332,8 @@ def template NeonLoadCompleteAcc64 {{
%(op_decl)s; %(op_decl)s;
%(op_rd)s; %(op_rd)s;
MemUnion &memUnion = *(MemUnion *)pkt->getPtr<uint8_t>(); MemUnion memUnion { { } };
memcpy(&memUnion, pkt->getPtr<uint8_t>(), pkt->getSize());
if (fault == NoFault) { if (fault == NoFault) {
%(memacc_code)s; %(memacc_code)s;