On Thu, 2004-12-02 at 19:27, Ron Murray wrote: > At Thu, 02 Dec 2004 13:45:59 -0500, > Ron Murray wrote: > We have a minor problem. Richard's patch seems to refer to a > pristine xfree86-4.3.0 source.
Damn! There are two similar #if defined lines. I made the patch against the wrong one! I also accept that I did make the patch against pristine sources - although in this case it means that you spotted my mistake. I still stand by my analysis. Hopefully the new patch (below) will work. Note I've taken the same approach as the one that my original patch clashed with. Basically I've removed the check for ia64 because I'm assuming that the non-executable issue could in future apply to all linux versions. Richard --- xc/programs/Xserver/hw/xfree86/loader/elfloader.c.orig 2004-12-02 22:29:26.000000000 +0000 +++ xc/programs/Xserver/hw/xfree86/loader/elfloader.c 2004-12-02 22:38:37.000000000 +0000 @@ -2937,7 +2937,7 @@ ErrorF( "Unable to allocate ELF sections\n" ); return NULL; } -# if defined(linux) && defined(__ia64__) || defined(__OpenBSD__) +# if defined(linux) || defined(__OpenBSD__) { unsigned long page_size = getpagesize(); unsigned long round; -- [EMAIL PROTECTED]