rpms/mlton/F-7 mlton-20070826-no-execmem.patch, NONE, 1.1 mlton.spec, 1.12, 1.13

[Adam Goode (agoode)]

Author: agoode

Update of /cvs/extras/rpms/mlton/F-7
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv20826

Modified Files:
	mlton.spec 
Added Files:
	mlton-20070826-no-execmem.patch 
Log Message:
* Sat Feb  9 2008 Adam Goode <adam at spicenitz.org> - 20070826-12
- Introduce patch to not call mprotect with PROT_EXEC


mlton-20070826-no-execmem.patch:

--- NEW FILE mlton-20070826-no-execmem.patch ---
>From 544930de3b1c754fa8803169902a63bce7cc02ba Mon Sep 17 00:00:00 2001
From: Adam Goode <adam at spicenitz.org>
Date: Wed, 6 Feb 2008 20:17:51 -0500
Subject: [PATCH] Remove PROT_EXEC from mprotect

It looks like mprotect is used here as part of signal handling.
There doesn't seems to be a reason to have the area of memory
marked as executable. In fact, on Fedora 9, this causes MLton
compiled binaries (including MLton itself) to fail.
---
 runtime/platform/mmap-protect.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/runtime/platform/mmap-protect.c b/runtime/platform/mmap-protect.c
index f0dea49..df42215 100644
--- a/runtime/platform/mmap-protect.c
+++ b/runtime/platform/mmap-protect.c
@@ -7,7 +7,7 @@ void *GC_mmapAnon_safe_protect (void *start, size_t length,
         if (mprotect (low, dead_low, PROT_NONE))
                 diee ("mprotect failed");
         result = (void*)((pointer)low + dead_low);
-        if (mprotect (result, length, PROT_READ | PROT_WRITE | PROT_EXEC))
+        if (mprotect (result, length, PROT_READ | PROT_WRITE))
                 diee ("mprotect failed");
         high = (void*)((pointer)result + length);
         if (mprotect (high, dead_high, PROT_NONE))
-- 
1.5.4



Index: mlton.spec
===================================================================
RCS file: /cvs/extras/rpms/mlton/F-7/mlton.spec,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -r1.12 -r1.13
--- mlton.spec	1 Oct 2007 00:10:53 -0000	1.12
+++ mlton.spec	9 Feb 2008 08:37:01 -0000	1.13
@@ -2,7 +2,7 @@
 
 Name:		mlton
 Version:	20070826
-Release:	10%{?dist}.2
+Release:	12%{?dist}
 Summary:	Optimizing compiler for Standard ML
 
 Group:		Development/Languages
@@ -10,10 +10,10 @@
 URL:		http://mlton.org/
 Source0:	http://mlton.org/pages/Download/attachments/%{name}-%{version}-1.src.tgz
 
-#Source1:	http://mlton.org/pages/Download/attachments/mlton-20070826-1.amd64-linux.tgz
+#Source1:	mlton-20061107-1.ppc-linux.tgz
 
 Patch0:		mlton-20070826-debuginfo.patch
-
+Patch1:		mlton-20070826-no-execmem.patch
 
 BuildRoot:	%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
@@ -43,6 +43,9 @@
 # disable strip so debuginfo packages work
 %patch -P 0 -p1 -b .debuginfo
 
+# no execmem
+%patch -P 1 -p1 -b .no-execmem
+
 %if %{mlton_bootstrap}
 %setup -T -D -q -a 1
 sed -i 's/\(^lib=\).*$/\1\$MLTON_BOOTSTRAP_DIR\/lib\/mlton/g'  \
@@ -98,11 +101,11 @@
 
 
 %changelog
-* Sun Sep 30 2007 Adam Goode <adam at spicenitz.org> - 20070826-10.2
-- Bootstrap complete
+* Sat Feb  9 2008 Adam Goode <adam at spicenitz.org> - 20070826-12
+- Introduce patch to not call mprotect with PROT_EXEC
 
-* Sun Sep 30 2007 Adam Goode <adam at spicenitz.org> - 20070826-10.1
-- Enable bootstrap for x86_64 for Fedora 7
+* Fri Jan 18 2008 Adam Goode <adam at spicenitz.org> - 20070826-11
+- Rebuild for new GCC
 
 * Thu Sep 27 2007 Adam Goode <adam at spicenitz.org> - 20070826-10
 - Disable bootstrap