Comment 27 for bug 10271

Message-Id: <1102027306.25600.183.camel@duncow>
Date: Thu, 02 Dec 2004 22:41:46 +0000
From: Richard Mortimer <email address hidden>
To: <email address hidden>
Cc: "Jurzitza, Dieter" <email address hidden>,
 <email address hidden>, <email address hidden>
Subject: Re: X11 crashing on 2.4.28

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 address hidden>