From e15c3098c6106d30f59615b5a2acd3becac94c9a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=C3=89rico=20Rolim?= Date: Mon, 14 Dec 2020 10:47:59 -0300 Subject: [PATCH] libmilter: set explicit stack size. Patch taken from alpine. libmilter stack overflows with musl's default thread stack size. --- .../libmilter/patches/thread-stacksize.patch | 42 +++++++++++++++++++ srcpkgs/libmilter/template | 2 +- 2 files changed, 43 insertions(+), 1 deletion(-) create mode 100644 srcpkgs/libmilter/patches/thread-stacksize.patch diff --git a/srcpkgs/libmilter/patches/thread-stacksize.patch b/srcpkgs/libmilter/patches/thread-stacksize.patch new file mode 100644 index 00000000000..0881abdfd94 --- /dev/null +++ b/srcpkgs/libmilter/patches/thread-stacksize.patch @@ -0,0 +1,42 @@ +Set default pthread stack size to 256 KB + +This patch tries to fix various crashes for applications depending on libmilter +by setting the stack size for pthreads to 256 KB. The default stack size for +musl libc is set to 80 KB whereas glibc has it set to 8 MB. This causes problems +when a large amount of memory is allocated on the stack. + +For example, opendkim allocates blocks of 64 KB multiple times, which causes +libmilter (and therefore opendkim) to crash. For now, a stack size of 256 KB +looks sufficient and makes opendkim stop crashing. + +Fixes https://bugs.alpinelinux.org/issues/6360 + +--- libmilter/libmilter.h ++++ libmilter/libmilter.h +@@ -127,10 +127,10 @@ + # define MI_SOCK_READ(s, b, l) read(s, b, l) + # define MI_SOCK_READ_FAIL(x) ((x) < 0) + # define MI_SOCK_WRITE(s, b, l) write(s, b, l) +- +-# define thread_create(ptid,wr,arg) pthread_create(ptid, NULL, wr, arg) + # define sthread_get_id() pthread_self() + ++extern int thread_create(pthread_t *ptid, void *(*wr) (void *), void *arg); ++ + typedef pthread_mutex_t smutex_t; + # define smutex_init(mp) (pthread_mutex_init(mp, NULL) == 0) + # define smutex_destroy(mp) (pthread_mutex_destroy(mp) == 0) +--- libmilter/main.c ++++ libmilter/main.c +@@ -16,6 +16,12 @@ + #include + #include + ++int thread_create(pthread_t *ptid, void *(*wr) (void *), void *arg) { ++ pthread_attr_t attr; ++ pthread_attr_init(&attr); ++ pthread_attr_setstacksize(&attr,256*1024); ++ return pthread_create(ptid, &attr, wr, arg); ++} + + static smfiDesc_ptr smfi = NULL; diff --git a/srcpkgs/libmilter/template b/srcpkgs/libmilter/template index b801ee3e5b9..0c4a3a84a54 100644 --- a/srcpkgs/libmilter/template +++ b/srcpkgs/libmilter/template @@ -1,7 +1,7 @@ # Template file for 'libmilter' pkgname=libmilter version=1.0.2 -revision=3 +revision=4 _pkgname=sendmail _version=8.15.2 hostmakedepends="m4"