This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB 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] validate binary before use


On Tue, 02 Apr 2013 18:24:18 +0200, Aleksandar Ristovski wrote:
> >>+      const CORE_ADDR sect_lma = l_addr + bfd_section_vma (so->abfd, asect);
> >
> >"lma" is irrelevant here, it needs to be VMA as you do runtime
> >target_read_memory for that address.
> 
> [AR] I read "lma" as load memory address, i.e. address where it is
> actually loaded. This is to differentiate load address from virtual
> address as set by the link-editor.

It is no longer in the code but LMA was confusing, LMA is an established term
in bfd scope with a different meaning:
	$ info '(ld)Output Section LMA'


This is not yet a full review.


Thanks,
Jan


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