Source: exim4
Severity: wishlist
Tags: patch

I think this should be a pretty good idea:

1. Add -fvisibility=hidden to CFLAGS, e.g. right after where LFLAGS is set 
to -rdynamic.

LFLAGS=-rdynamic
CFLAGS += -fvisibility=hidden

1a. Change LFLAGS to LDFLAGS above, i.e.

LDFLAGS += -rdynamic
CFLAGS += -fvisibility=hidden

$(LDFLAGS) is only applied to the main exim binary, not the utilities.

2. Add the following hunk:

--- exim4-4.66/build-tree/src/local_scan.h      2007-03-05 19:46:40.853429411 
+0100
+++ exim4-4.66/build-tree/src/local_scan.h      2007-03-05 19:46:42.673669439 
+0100
@@ -18,6 +18,7 @@
 settings, and the store functions. */
 
 #include <sys/types.h>
+#pragma GCC visibility push(default)
 #include "config.h"
 #include "mytypes.h"
 #include "store.h"
@@ -188,4 +189,6 @@
 extern uschar *string_copyn(uschar *, int);
 extern uschar *string_sprintf(char *, ...);
 
+#pragma GCC visibility pop
+
 /* End of local_scan.h */


The result should be that only those symbols declared in local_scan.h and its 
included files are included in the dynamic symbol table.

-- 
Magnus Holmgren        [EMAIL PROTECTED]
                       (No Cc of list mail needed, thanks)

  "Exim is better at being younger, whereas sendmail is better for 
   Scrabble (50 point bonus for clearing your rack)" -- Dave Evans

Attachment: pgpXDtKPF0eN6.pgp
Description: PGP signature

Reply via email to