Re: [PATCH] binfmt_elf: randomize PIE binaries (2nd try)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Oct 20, 2006 at 02:20:31PM +0200, Jesper Juhl wrote:
> On 20/10/06, Marcus Meissner <[email protected]> wrote:
> >Randomizes -pie compiled binaries from PAGE_SIZE up to
> >ELF_ET_DYN_BASE.
> >
> >0 -> PAGE_SIZE is excluded to allow NULL ptr accesses
> >to fail.
> >
> >Signed-off-by: Marcus Meissner <[email protected]>
> >
> >----
> > binfmt_elf.c |    8 +++++++-
> > 1 file changed, 7 insertions(+), 1 deletion(-)
> >
> >--- linux-2.6.18/fs/binfmt_elf.c.xx     2006-10-20 10:42:03.000000000 +0200
> >+++ linux-2.6.18/fs/binfmt_elf.c        2006-10-20 10:51:27.000000000 +0200
> >@@ -856,7 +856,13 @@
> >                         * default mmap base, as well as whatever program 
> >                         they
> >                         * might try to exec.  This is because the brk will
> >                         * follow the loader, and is not movable.  */
> >-                       load_bias = ELF_PAGESTART(ELF_ET_DYN_BASE - vaddr);
> >+                       if (current->flags & PF_RANDOMIZE)
> >+                               load_bias = randomize_range(PAGE_SIZE,
> >+                                                           
> >ELF_ET_DYN_BASE,
> >+                                                           0);
> 
> How about putting the two lines above on one line?  ^^^^^

My editor said it would end at column 80. Starting from 0 it would col 79
actually, but I did not want to risk it.

Ciao, Marcus
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux