Don't use libcrypto private headers with mkdef.pl
authorRichard Levitte <levitte@openssl.org>
Sun, 14 Feb 2016 18:37:10 +0000 (19:37 +0100)
committerRichard Levitte <levitte@openssl.org>
Sun, 14 Feb 2016 18:37:10 +0000 (19:37 +0100)
Three header files from crypto/include/internal were used by
util/mkdef.pl.  This should never be needed.  Some test program used
these, which made it a valid reason at the time to make the some
internal symbols public in the shared libraries, but that's not the
case any more.

However, to be able to link libssl.so, some symbols found in
include/internal headers still need to be made public.

Reviewed-by: Andy Polyakov <appro@openssl.org>
util/mkdef.pl

index b5ebc18b8e2edab54c9e49bb45ccec91b3181091..2de27c078aefb922eac3329bd4a67b2158adc12f 100755 (executable)
@@ -234,10 +234,9 @@ my $ssl="include/openssl/ssl.h";
 $ssl.=" include/openssl/tls1.h";
 $ssl.=" include/openssl/srtp.h";
 
+# We use headers found in include/openssl and include/internal only.
+# The latter is needed so libssl.so/.dll/.exe can link properly.
 my $crypto ="include/openssl/crypto.h";
-$crypto.=" crypto/include/internal/cryptlib.h";
-$crypto.=" crypto/include/internal/chacha.h"; # unless $no_chacha;
-$crypto.=" crypto/include/internal/poly1305.h"; # unless $no_poly1305;
 $crypto.=" include/internal/o_dir.h";
 $crypto.=" include/internal/o_str.h";
 $crypto.=" include/openssl/des.h" ; # unless $no_des;