Avoid warnings in tortls.h includes

diff --git a/src/common/tortls.h b/src/common/tortls.h
index 6db0e8b527572c991aa16e487696475c728c4cdc..2f467a2f557756d59836cae285829c9be76ebe8e 100644 (file)
--- a/src/common/tortls.h
+++ b/src/common/tortls.h
@@ -11,9 +11,6 @@
  * \brief Headers for tortls.c
  **/
 
-#include <openssl/ssl.h>
-#include <openssl/ssl3.h>
-
 #include "crypto.h"
 #include "compat.h"
 #include "testsupport.h"

diff --git a/src/test/test_tortls.c b/src/test/test_tortls.c
index de900bed96920a18578722b3f1ef4087297402fa..67b645c1caccaddfa0c7acb5d896ebac86feeb10 100644 (file)
--- a/src/test/test_tortls.c
+++ b/src/test/test_tortls.c
@@ -5,19 +5,41 @@
 #define LOG_PRIVATE
 #include "orconfig.h"
 
-#include "or.h"
-#include "torlog.h"
-#include "config.h"
-#include "tortls.h"
+#ifdef __GNUC__
+#define GCC_VERSION (__GNUC__ * 100 + __GNUC_MINOR__)
+#endif
 
-#include "test.h"
-#include "log_test_helpers.h"
+#if __GNUC__ && GCC_VERSION >= 402
+#if GCC_VERSION >= 406
+#pragma GCC diagnostic push
+#endif
+/* Some versions of OpenSSL declare SSL_get_selected_srtp_profile twice in
+ * srtp.h. Suppress the GCC warning so we can build with -Wredundant-decl. */
+#pragma GCC diagnostic ignored "-Wredundant-decls"
+#endif
+
+#include <openssl/opensslv.h>
 
 #include <openssl/ssl.h>
 #include <openssl/ssl3.h>
 #include <openssl/err.h>
 #include <openssl/asn1t.h>
 
+#if __GNUC__ && GCC_VERSION >= 402
+#if GCC_VERSION >= 406
+#pragma GCC diagnostic pop
+#else
+#pragma GCC diagnostic warning "-Wredundant-decls"
+#endif
+#endif
+
+#include "or.h"
+#include "torlog.h"
+#include "config.h"
+#include "tortls.h"
+
+#include "test.h"
+#include "log_test_helpers.h"
 #define NS_MODULE tortls
 
 extern tor_tls_context_t *server_tls_context;