Re: reread mimedefang after sa-update

On 09/14/2016 11:48 PM, Richard Laager wrote:
> On 09/14/2016 11:12 AM, Marcus Schopen wrote:
>> Am Mittwoch, den 14.09.2016, 11:51 -0400 schrieb Dianne Skoll:
>>> On Wed, 14 Sep 2016 17:46:07 +0200
>>> Marcus Schopen <[email protected]> wrote:
>>>
>>>> Sep 14 17:39:55 scansrv mimedefang-multiplexor[24029]: Cannot destroy
>>>> and recreate a Perl interpreter safely on this platform.  Filter rules
>>>> will NOT be reread.
>>>
>>> Huh!  That is very weird.  I don't get anything like that on Debian Jessie.
>>
>> Hmmm, I get this error on Ubuntu 12.04 LTS too, mimedefang 2.71, perl
>> 5.14.2.
>>
>>> If you get that message, then you'll need to do restart instead of reread.
>>
>> Okay, but this might harm a mail which is in process. Any ideas how to
>> fix above error?
> 
> I'm digging into it right now. The problem seems to be that Perl outputs
> -lpthread as a required LDFLAG, but gcc needs -pthread (with no "l"), or
> else it fails linking to the symbol pthread_setspecific.

So configure.in has a check for this pthread flag, but does not use
$PTHREAD_FLAG as part of the $CFLAGS for the SAFE_EMBED_PERL test. Patch
attached, which results in SAFE_EMBED_PERL set and reread working for me
on Ubuntu 16.04.

The patch is for configure.in. You'll need to either regenerate
configure, or apply the same change to configure.

Thanks for catching this. I just upgraded my mail servers a couple days
ago and I hadn't actually tested the reread operation.

-- 
Richard

_______________________________________________
NOTE: If there is a disclaimer or other legal boilerplate in the above
message, it is NULL AND VOID.  You may ignore it.

Visit http://www.mimedefang.org and http://www.roaringpenguin.com
MIMEDefang mailing list [email protected]
http://lists.roaringpenguin.com/mailman/listinfo/mimedefang

--- mimedefang-2.78/configure.in	2015-04-23 09:37:03.000000000 -0500
+++ mimedefang-2.78-new/configure.in	2016-09-14 23:58:36.713587787 -0500
@@ -343,7 +343,7 @@
    AC_MSG_CHECKING([if it is safe to destroy and recreate a Perl interpreter])
    LIBS="-lperl $LIBS"
    LDFLAGS="$EMBPERLLDFLAGS $LDFLAGS"
-   CFLAGS="$EMBPERLCFLAGS $CFLAGS"
+   CFLAGS="$EMBPERLCFLAGS $CFLAGS $PTHREAD_FLAG"
    AC_TRY_RUN([
 #include <EXTERN.h>
 #include <perl.h>