[libcoro-perl] 06/07: add a patch reverting the disabling of FORTIFY_SOURCE in the hope that current libc is no longer problematic

Damyan Ivanov dmn at moszumanska.debian.org
Sat Oct 28 21:06:24 UTC 2017


This is an automated email from the git hooks/post-receive script.

dmn pushed a commit to branch master
in repository libcoro-perl.

commit 12021c765b80441461d91c2f1072aaa630e2bb9c
Author: Damyan Ivanov <dmn at debian.org>
Date:   Sat Oct 28 21:05:34 2017 +0000

    add a patch reverting the disabling of FORTIFY_SOURCE in the hope that current libc is no longer problematic
    
    this is unfinished. needs to be tested on architectures
    using setjump/longjump implementation (amd64 is not one of those)
---
 debian/patches/do-not-disable-fortify-source.patch | 30 ++++++++++++++++++++++
 debian/patches/series                              |  1 +
 2 files changed, 31 insertions(+)

diff --git a/debian/patches/do-not-disable-fortify-source.patch b/debian/patches/do-not-disable-fortify-source.patch
new file mode 100644
index 0000000..07aec40
--- /dev/null
+++ b/debian/patches/do-not-disable-fortify-source.patch
@@ -0,0 +1,30 @@
+Description: Experimental patch enabling FORTIFY_SOURCE
+ FIXME: this needs to be tested on architectures that use the
+ setjump/longjump implementation
+Author: Damyan Ivanov <dmn at debian.org>
+
+--- a/Coro/Makefile.PL
++++ b/Coro/Makefile.PL
+@@ -219,7 +219,6 @@ if ($iface eq "u") {
+    print "\nUsing ucontext implementation\n\n";
+    conftest ("TEST_makecontext");
+ } elsif ($iface eq "s") {
+-   $DEFINE .= " -D_FORTIFY_SOURCE=0";
+    $DEFINE .= " -DCORO_SJLJ";
+    print "\nUsing setjmp/longjmp/sigaltstack implementation\n\n";
+    conftest ("TEST_sigaltstack");
+--- a/Coro/libcoro/coro.h
++++ b/Coro/libcoro/coro.h
+@@ -342,12 +342,6 @@ struct coro_context
+ #  define _GNU_SOURCE /* for glibc */
+ # endif
+ 
+-/* try to disable well-meant but buggy checks in some libcs */
+-# ifdef _FORTIFY_SOURCE
+-#  undef _FORTIFY_SOURCE
+-#  undef __USE_FORTIFY_LEVEL /* helps some more when too much has been included already */
+-# endif
+-
+ # if !CORO_LOSER
+ #  include <unistd.h>
+ # endif
diff --git a/debian/patches/series b/debian/patches/series
index d63b0d0..aa7db98 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,3 +1,4 @@
 fix-wrong-path-for-interpreter.patch
 whatis.patch
 spelling.patch
+do-not-disable-fortify-source.patch

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-perl/packages/libcoro-perl.git



More information about the Pkg-perl-cvs-commits mailing list