[patch] Please drop the dangerous libssl and libcrypto deps - Mailing list pgadmin-hackers

From Raphaël Enrici
Subject [patch] Please drop the dangerous libssl and libcrypto deps
Date
Msg-id 43B0757E.2020509@club-internet.fr
Whole thread Raw
Responses Re: [patch] Please drop the dangerous libssl and libcrypto deps
List pgadmin-hackers
Dear friends,

Loic Minier(CCed) provided a patch to prevent pgadmin3 1.2.2 from being
linked to a different libssl version than libpq when dynamically built
with an already ssl enabled libpq. The full bug report and original
patch by Loic can be found at
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=341117

Attached is an svn diff for 1.4.1 release that I'm using for the package
I'm about to upload to Debian (patch_libpqssl_1.4.1).
You'll also find a fully untested patch for trunk (patch_libpqssl_current).

Please recheck and apply if eveything's ok with them.

Cheers,
Raph
Index: acinclude.m4
===================================================================
--- acinclude.m4    (revision 4858)
+++ acinclude.m4    (working copy)
@@ -227,7 +227,10 @@
     else
         if test "$pgsql_ssl_libpq" = "yes"
         then
-            LIBS="$LIBS -lssl -lcrypto -lpq"
+        # no idea why -lssl and -lcrypto were included here, as this
+            # support is provided via libpq
+            #LIBS="$LIBS -lssl -lcrypto -lpq
+            LIBS="$LIBS -lpq"
         else
             LIBS="$LIBS -lcrypto -lpq"
         fi
Index: acinclude.m4
===================================================================
--- acinclude.m4    (revision 4858)
+++ acinclude.m4    (working copy)
@@ -233,7 +233,7 @@
         else
             if test "$PG_SSL" = "yes"
             then
-                LIBS="$LIBS -L${PG_LIB} -lssl -lcrypto -lpq"
+                LIBS="$LIBS -L${PG_LIB} -lpq"
             else
                 LIBS="$LIBS -L${PG_LIB} -lcrypto -lpq"
             fi

pgadmin-hackers by date:

Previous
From: Raphaël Enrici
Date:
Subject: pgAgent manpage
Next
From: Andreas Pflug
Date:
Subject: Re: [patch] Please drop the dangerous libssl and libcrypto deps