[PATCH ARC 8/8] OPCODE: new ARC port

Claudiu Zissulescu claziss@gmail.com
Fri Sep 4 16:20:00 GMT 2015


On 04/09/15 17:38, Nick Clifton wrote:
> Hi Claudiu,
> 
>    This worries me:
> 
> +  /* Always read second word in case of limm we ignore the result
> +     since last insn may not have a limm.  */
> +  status = (*info->read_memory_func) (memaddr + insnLen, buffer, 4, info);
> 
>    Does this mean that it is possible to read past the end of 
> instruction buffer ?  If so that would trigger a memory access failure, 
> especially if sanitization has been enabled.

You are right, I missed this aspect when I did the port. I will change the reading of the long immediate to be conditionally.

Thank you,
Claudiu



More information about the Binutils mailing list