Re: [patches] [PATCH] [18/19] x86_64: Overlapping program headers in physical addr space fix

>@@ -17,6 +17,7 @@ PHDRS {
> 	text PT_LOAD FLAGS(5);	/* R_E */
> 	data PT_LOAD FLAGS(7);	/* RWE */
> 	user PT_LOAD FLAGS(7);	/* RWE */
>+	data.init PT_LOAD FLAGS(7);	/* RWE */
> 	note PT_NOTE FLAGS(4);	/* R__ */
> }
> SECTIONS

Even though it's only cosmetic, I think it would have been
more than appropriate to remove the ill 'E' permission on data
with that change. (Btw., why does 'note' need 'R'?) Also, I
consider the naming of the new segment misleading - just 'init'
would have been more correct.

Jan

-
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/

---

• Follow-Ups:
  • Re: [patches] [PATCH] [18/19] x86_64: Overlapping program headers in physical addr space fix
    • From: Vivek Goyal <[email protected]>

• References:
  • [PATCH] [18/19] x86_64: Overlapping program headers in physical addr space fix
    • From: Andi Kleen <[email protected]>

• Prev by Date: Re: incorrect taint of ndiswrapper
• Next by Date: [RFD][PATCH 1/2] sysctl: Allow a zero ctl_name in the middle of a sysctl table
• Previous by thread: [PATCH] [18/19] x86_64: Overlapping program headers in physical addr space fix
• Next by thread: Re: [patches] [PATCH] [18/19] x86_64: Overlapping program headers in physical addr space fix
• Index(es):
  • Date
  • Thread

[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]