From e82f45982cf1660d3287332d3d70bf5b01de1fd1 Mon Sep 17 00:00:00 2001 From: Matt Caswell Date: Thu, 29 Oct 2020 12:46:28 +0000 Subject: [PATCH] Fix some missed usage of DEFINE_LHASH_OF() PR#12860 fixed issues with the Lhash code. It replaced usage of DEFINE_LHASH_OF() in the public headers. Unfortunately it missed a couple of instances. Reviewed-by: Paul Dale Reviewed-by: Richard Levitte (Merged from https://github.com/openssl/openssl/pull/13274) --- .gitignore | 1 + build.info | 2 ++ include/openssl/{lhash.h => lhash.h.in} | 23 ++++++++--------------- 3 files changed, 11 insertions(+), 15 deletions(-) rename include/openssl/{lhash.h => lhash.h.in} (97%) diff --git a/.gitignore b/.gitignore index 8795a2c5dd..7a07450106 100644 --- a/.gitignore +++ b/.gitignore @@ -35,6 +35,7 @@ /include/openssl/err.h /include/openssl/ess.h /include/openssl/fipskey.h +/include/openssl/lhash.h /include/openssl/ocsp.h /include/openssl/opensslv.h /include/openssl/pkcs12.h diff --git a/build.info b/build.info index dc239a706d..44ecee35cb 100644 --- a/build.info +++ b/build.info @@ -26,6 +26,7 @@ DEPEND[]=include/openssl/asn1.h \ include/openssl/err.h \ include/openssl/ess.h \ include/openssl/fipskey.h \ + include/openssl/lhash.h \ include/openssl/opensslv.h \ include/openssl/ocsp.h \ include/openssl/pkcs12.h \ @@ -53,6 +54,7 @@ GENERATE[include/openssl/ct.h]=include/openssl/ct.h.in GENERATE[include/openssl/err.h]=include/openssl/err.h.in GENERATE[include/openssl/ess.h]=include/openssl/ess.h.in GENERATE[include/openssl/fipskey.h]=include/openssl/fipskey.h.in +GENERATE[include/openssl/lhash.h]=include/openssl/lhash.h.in GENERATE[include/openssl/ocsp.h]=include/openssl/ocsp.h.in GENERATE[include/openssl/opensslv.h]=include/openssl/opensslv.h.in GENERATE[include/openssl/pkcs12.h]=include/openssl/pkcs12.h.in diff --git a/include/openssl/lhash.h b/include/openssl/lhash.h.in similarity index 97% rename from include/openssl/lhash.h rename to include/openssl/lhash.h.in index ccdd3a60ee..571dce43ea 100644 --- a/include/openssl/lhash.h +++ b/include/openssl/lhash.h.in @@ -7,6 +7,10 @@ * https://www.openssl.org/source/license.html */ +{- +use OpenSSL::stackhash qw(generate_lhash_macros); +-} + /* * Header for dynamic hash table routines Author - Eric Young */ @@ -240,21 +244,10 @@ void OPENSSL_LH_node_usage_stats_bio(const OPENSSL_LHASH *lh, BIO *out); } \ LHASH_OF(type) -DEFINE_LHASH_OF(OPENSSL_STRING); -# ifdef _MSC_VER -/* - * push and pop this warning: - * warning C4090: 'function': different 'const' qualifiers - */ -# pragma warning (push) -# pragma warning (disable: 4090) -# endif - -DEFINE_LHASH_OF(OPENSSL_CSTRING); - -# ifdef _MSC_VER -# pragma warning (pop) -# endif +{- + generate_lhash_macros("OPENSSL_STRING") + .generate_lhash_macros("OPENSSL_CSTRING"); +-} #ifdef __cplusplus }