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: loadbase alignment - ld.so/prelink/kernel or bfd_elf_bfd_from_remote_memory() bug?


On Wed, Aug 15, 2007 at 04:40:55PM +0200, Jan Kratochvil wrote:
> On Wed, 15 Aug 2007 16:03:02 +0200, H.J. Lu wrote:
> > On Wed, Aug 15, 2007 at 03:56:10PM +0200, Jan Kratochvil wrote:
> ...
> > > --- glibc-20070810T2152-orig/elf/dl-load.c	2007-08-03 17:50:24.000000000 +0200
> > > +++ glibc-20070810T2152/elf/dl-load.c	2007-08-15 00:41:03.000000000 +0200
> > > @@ -1012,6 +1012,7 @@ _dl_map_object_from_fd (const char *name
> > >  	int prot;
> > >        } loadcmds[l->l_phnum], *c;
> > >      size_t nloadcmds = 0;
> > > +    ElfW(Addr) alignmax = GLRO(dl_pagesize);
> > 
> > Why not replace all GLRO(dl_pagesize) with alignmax?
> 
> Sorry if I did not understand your suggestion.
> 

You introduced alignmax for GLRO(dl_pagesize). But you still use
GLRO(dl_pagesize) in some places instead of alignmax.


H.J.


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