This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


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


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]