Remove tests dependence on e_os.h
authorPauli <paul.dale@oracle.com>
Thu, 17 Aug 2017 23:50:25 +0000 (09:50 +1000)
committerPauli <paul.dale@oracle.com>
Thu, 17 Aug 2017 23:50:25 +0000 (09:50 +1000)
Apart from ssltest_old.c, the test suite relied on e_os.h for the
OSSL_NELEM macro and nothing else.

The ssltest_old.c also requires EXIT and some socket macros.

Create a new header to define the OSSL_NELEM macro and use that instead.

Reviewed-by: Rich Salz <rsalz@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/4186)

58 files changed:
e_os.h
include/internal/nelem.h [new file with mode: 0644]
test/asn1_internal_test.c
test/asn1_time_test.c
test/bad_dtls_test.c
test/bftest.c
test/bntest.c
test/casttest.c
test/cipher_overhead_test.c
test/cipherbytes_test.c
test/cipherlist_test.c
test/ciphername_test.c
test/constant_time_test.c
test/crltest.c
test/d2i_test.c
test/danetest.c
test/dhtest.c
test/drbgtest.c
test/dsatest.c
test/dtlsv1listentest.c
test/ecstresstest.c
test/ectest.c
test/evp_extra_test.c
test/exptest.c
test/handshake_helper.c
test/hmactest.c
test/ideatest.c
test/igetest.c
test/lhash_test.c
test/md2test.c
test/mdc2_internal_test.c
test/mdc2test.c
test/modes_internal_test.c
test/poly1305_internal_test.c
test/rc2test.c
test/rc4test.c
test/rc5test.c
test/rsa_test.c
test/servername_test.c
test/siphash_internal_test.c
test/ssl_cert_table_internal_test.c
test/ssl_test_ctx.c
test/ssl_test_ctx_test.c
test/sslapitest.c
test/ssltest_old.c
test/ssltestlib.c
test/stack_test.c
test/test_test.c
test/testutil/driver.c
test/testutil/format_output.c
test/testutil/main.c
test/testutil/stanza.c
test/testutil/tests.c
test/time_offset_test.c
test/v3nametest.c
test/x509_internal_test.c
test/x509_time_test.c
test/x509aux.c

diff --git a/e_os.h b/e_os.h
index 4e01f9b..87d98c2 100644 (file)
--- a/e_os.h
+++ b/e_os.h
@@ -1,5 +1,5 @@
 /*
- * Copyright 1995-2016 The OpenSSL Project Authors. All Rights Reserved.
+ * Copyright 1995-2017 The OpenSSL Project Authors. All Rights Reserved.
  *
  * Licensed under the OpenSSL license (the "License").  You may not use
  * this file except in compliance with the License.  You can obtain a copy
@@ -14,6 +14,8 @@
 
 # include <openssl/e_os2.h>
 # include <openssl/crypto.h>
+# include <internal/nelem.h>
+
 /*
  * <openssl/e_os2.h> contains what we can justify to make visible to the
  * outside; this file e_os.h is not part of the exported interface.
@@ -540,8 +542,6 @@ struct servent *getservbyname(const char *name, const char *proto);
 # endif
 /* end vxworks */
 
-#define OSSL_NELEM(x)    (sizeof(x)/sizeof((x)[0]))
-
 #ifdef FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION
 # define CRYPTO_memcmp memcmp
 #endif
diff --git a/include/internal/nelem.h b/include/internal/nelem.h
new file mode 100644 (file)
index 0000000..d65a21a
--- /dev/null
@@ -0,0 +1,14 @@
+/*
+ * Copyright 2017 The OpenSSL Project Authors. All Rights Reserved.
+ *
+ * Licensed under the OpenSSL license (the "License").  You may not use
+ * this file except in compliance with the License.  You can obtain a copy
+ * in the file LICENSE in the source distribution or at
+ * https://www.openssl.org/source/license.html
+ */
+
+#ifndef HEADER_NELEM_H
+# define HEADER_NELEM_H
+
+# define OSSL_NELEM(x)    (sizeof(x)/sizeof((x)[0]))
+#endif
index 194a073..06e324b 100644 (file)
@@ -16,7 +16,7 @@
 #include <openssl/evp.h>
 #include <openssl/objects.h>
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 /**********************************************************************
  *
index 184a18a..897436c 100644 (file)
@@ -16,7 +16,7 @@
 #include <openssl/evp.h>
 #include <openssl/objects.h>
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 struct testdata {
     char *data;             /* TIME string value */
index 2c10a1c..932cb0e 100644 (file)
@@ -39,7 +39,7 @@
 #include <openssl/kdf.h>
 
 #include "../ssl/packet_locl.h"
-#include "../e_os.h" /* for OSSL_NELEM() */
+#include <internal/nelem.h>
 
 #include "testutil.h"
 
index cc509f2..0639ec8 100644 (file)
@@ -19,7 +19,7 @@
 
 #include "testutil.h"
 
-#include "../e_os.h"
+#include <internal/nelem.h>
 
 #ifndef OPENSSL_NO_BF
 # include <openssl/blowfish.h>
index f32b308..cb292c6 100644 (file)
@@ -12,7 +12,7 @@
 #include <string.h>
 #include <ctype.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include <internal/numbers.h>
 #include <openssl/bn.h>
 #include <openssl/crypto.h>
index 0fdcb10..417a4cb 100644 (file)
@@ -12,7 +12,7 @@
 #include <stdlib.h>
 
 #include <openssl/opensslconf.h> /* To see if OPENSSL_NO_CAST is defined */
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #ifndef OPENSSL_NO_CAST
index e93bf33..a07ac7e 100644 (file)
@@ -7,7 +7,7 @@
  * https://www.openssl.org/source/license.html
  */
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #ifdef __VMS
index 6601f7b..13d6b23 100644 (file)
@@ -18,7 +18,7 @@
 #include <openssl/ssl3.h>
 #include <openssl/tls1.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 static SSL_CTX *ctx;
index 8290427..6ec9631 100644 (file)
@@ -18,7 +18,7 @@
 #include <openssl/ssl3.h>
 #include <openssl/tls1.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 typedef struct cipherlist_test_fixture {
index f010a3b..0eac902 100644 (file)
@@ -19,7 +19,7 @@
 #include <openssl/ssl3.h>
 #include <openssl/tls1.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 typedef struct cipher_id_name {
index 8575194..24cbaf3 100644 (file)
@@ -10,7 +10,7 @@
 #include <stdio.h>
 #include <stdlib.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "internal/constant_time_locl.h"
 #include "testutil.h"
 #include "internal/numbers.h"
index 720c13b..f5af4b1 100644 (file)
@@ -7,7 +7,7 @@
  * https://www.openssl.org/source/license.html
  */
 
-#include "../e_os.h"
+#include <internal/nelem.h>
 #include <string.h>
 #include <openssl/bio.h>
 #include <openssl/crypto.h>
index e5ebcfa..48edba4 100644 (file)
@@ -20,7 +20,7 @@
 #include <openssl/err.h>
 #include <openssl/x509.h>
 #include <openssl/x509v3.h>
-#include "e_os.h"
+#include <internal/nelem.h>
 
 static const ASN1_ITEM *item_type;
 static const char *test_file;
index d21f554..de51c83 100644 (file)
@@ -24,7 +24,7 @@
 #endif
 #include "testutil.h"
 
-#include "e_os.h"
+#include <internal/nelem.h>
 
 #define _UC(c) ((unsigned char)(c))
 
index 595732c..ac09c44 100644 (file)
@@ -11,7 +11,7 @@
 #include <stdlib.h>
 #include <string.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include <openssl/crypto.h>
 #include <openssl/bio.h>
 #include <openssl/bn.h>
index 62d2dbd..28d0d90 100644 (file)
@@ -8,7 +8,7 @@
  */
 
 #include <string.h>
-#include "e_os.h"
+#include <internal/nelem.h>
 #include <openssl/crypto.h>
 #include <openssl/err.h>
 #include <openssl/rand.h>
index 7c58731..d25a1d7 100644 (file)
@@ -19,7 +19,7 @@
 #include <openssl/dsa.h>
 
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 #ifndef OPENSSL_NO_DSA
 static int dsa_cb(int p, int n, BN_GENCB *arg);
index e401ccf..00120cc 100644 (file)
@@ -12,7 +12,7 @@
 #include <openssl/bio.h>
 #include <openssl/err.h>
 #include <openssl/conf.h>
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #ifndef OPENSSL_NO_SOCK
index e16ccd5..7c76f02 100644 (file)
@@ -8,7 +8,7 @@
  * or in the file LICENSE in the source distribution.
  */
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #include <stdio.h>
index 80e1001..85847e0 100644 (file)
@@ -8,7 +8,7 @@
  * https://www.openssl.org/source/license.html
  */
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #ifndef OPENSSL_NO_EC
index acccfe5..5ca95e4 100644 (file)
@@ -16,7 +16,7 @@
 #include <openssl/rsa.h>
 #include <openssl/x509.h>
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 /*
  * kExampleRSAKeyDER is an RSA private key in ASN.1, DER format. Of course, you
index b357a23..62a2377 100644 (file)
@@ -11,7 +11,7 @@
 #include <stdlib.h>
 #include <string.h>
 
-#include "../e_os.h"
+#include <internal/nelem.h>
 
 #include <openssl/bio.h>
 #include <openssl/bn.h>
index d200332..b33cdd5 100644 (file)
@@ -18,7 +18,7 @@
 
 #ifndef OPENSSL_NO_SOCK
 # define USE_SOCKETS
-# include "e_os.h"
+# include <internal/nelem.h>
 #endif
 
 #include "handshake_helper.h"
index efe21e0..abd1193 100644 (file)
@@ -11,7 +11,7 @@
 #include <string.h>
 #include <stdlib.h>
 
-#include "../e_os.h"
+#include <internal/nelem.h>
 
 # include <openssl/hmac.h>
 # include <openssl/sha.h>
index 32fea47..5b715fe 100644 (file)
@@ -9,7 +9,7 @@
 
 #include <string.h>
 
-#include "../e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #ifndef OPENSSL_NO_IDEA
index a5c890e..8a1db67 100644 (file)
@@ -12,7 +12,7 @@
 #include <openssl/rand.h>
 #include <stdio.h>
 #include <string.h>
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #define TEST_SIZE       128
index 704b180..6deea8d 100644 (file)
@@ -16,7 +16,7 @@
 #include <openssl/err.h>
 #include <openssl/crypto.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 /*
index cda8469..df55932 100644 (file)
@@ -9,7 +9,7 @@
 
 #include <string.h>
 
-#include "../e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #ifndef OPENSSL_NO_MD2
index 2c8d22d..c5e74ea 100644 (file)
@@ -14,7 +14,7 @@
 
 #include <openssl/mdc2.h>
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 typedef struct {
     const char *input;
index 4f29953..5dfe60e 100644 (file)
@@ -9,7 +9,7 @@
 
 #include <string.h>
 
-#include "../e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #if defined(OPENSSL_NO_DES) && !defined(OPENSSL_NO_MDC2)
index e2eb3b6..3fd0498 100644 (file)
@@ -16,7 +16,7 @@
 #include <openssl/modes.h>
 #include "../crypto/modes/modes_lcl.h"
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 typedef struct {
     size_t size;
index 592f4b6..55b7644 100644 (file)
@@ -15,7 +15,7 @@
 #include "testutil.h"
 #include "internal/poly1305.h"
 #include "../crypto/poly1305/poly1305_local.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 typedef struct {
     size_t size;
index 6323d81..fc00076 100644 (file)
@@ -7,7 +7,7 @@
  * https://www.openssl.org/source/license.html
  */
 
-#include "../e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #ifndef OPENSSL_NO_RC2
index b2032fd..e3f41b6 100644 (file)
@@ -9,7 +9,7 @@
 
 #include <string.h>
 
-#include "../e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #ifndef OPENSSL_NO_RC4
index 16a0364..9aa214f 100644 (file)
@@ -9,7 +9,7 @@
 
 #include <string.h>
 
-#include "../e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #ifndef OPENSSL_NO_RC5
index 55339f9..13dcde1 100644 (file)
@@ -12,7 +12,7 @@
 #include <stdio.h>
 #include <string.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 
 #include <openssl/crypto.h>
 #include <openssl/err.h>
index e0fe4d5..f1b66ee 100644 (file)
@@ -21,7 +21,7 @@
 #include "../ssl/packet_locl.h"
 
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 #define CLIENT_VERSION_LEN      2
 
index ca24339..60a2d8d 100644 (file)
@@ -16,7 +16,7 @@
 #include "testutil.h"
 #include "internal/siphash.h"
 #include "../crypto/siphash/siphash_local.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 static BIO* b_stderr = NULL;
 static BIO* b_stdout = NULL;
index 9d6d2b5..9f52015 100644 (file)
@@ -14,7 +14,7 @@
 
 #include <openssl/ssl.h>
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 #ifdef __VMS
 # pragma names save
index b581cdf..9dc3395 100644 (file)
@@ -12,7 +12,7 @@
 #include <openssl/e_os2.h>
 #include <openssl/crypto.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "ssl_test_ctx.h"
 #include "testutil.h"
 
index f7ffe40..2376ce1 100644 (file)
@@ -15,7 +15,7 @@
 #include <stdio.h>
 #include <string.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "ssl_test_ctx.h"
 #include "testutil.h"
 #include <openssl/e_os2.h>
index 1c43686..dfcbf11 100644 (file)
@@ -17,7 +17,7 @@
 
 #include "ssltestlib.h"
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "../ssl/ssl_locl.h"
 
 static char *cert = NULL;
index 3a8d9b2..9e3bb7a 100644 (file)
@@ -25,6 +25,8 @@
 #include <string.h>
 #include <time.h>
 
+#include <internal/nelem.h>
+
 #define USE_SOCKETS
 #include "e_os.h"
 
index 64acea3..410d323 100644 (file)
@@ -9,7 +9,7 @@
 
 #include <string.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "ssltestlib.h"
 #include "testutil.h"
 
index ba2ceaf..922dfdd 100644 (file)
@@ -16,7 +16,7 @@
 #include <openssl/err.h>
 #include <openssl/crypto.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 /* The macros below generate unused functions which error out one of the clang
index ae16d82..742cd61 100644 (file)
@@ -16,7 +16,7 @@
 #include <openssl/crypto.h>
 #include <openssl/bn.h>
 
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 #define TEST(expected, test) test_case((expected), #test, (test))
index 274d3cd..1859706 100644 (file)
@@ -14,7 +14,7 @@
 #include <string.h>
 #include <assert.h>
 
-#include "../../e_os.h"
+#include <internal/nelem.h>
 #include <openssl/bio.h>
 
 /*
index ae3bf0e..13123d5 100644 (file)
@@ -13,7 +13,7 @@
 
 #include <string.h>
 #include <ctype.h>
-#include "../../e_os.h"
+#include <internal/nelem.h>
 
 /* The size of memory buffers to display on failure */
 #define MEM_BUFFER_SIZE     (2000)
index 34abcb2..2f4eeb6 100644 (file)
@@ -8,7 +8,7 @@
  */
 
 #include "../testutil.h"
-#include "../../e_os.h"
+#include <internal/nelem.h>
 #include "output.h"
 #include "tu_local.h"
 
index 8a8f0cc..3596752 100644 (file)
@@ -12,7 +12,7 @@
 #include <string.h>
 #include <ctype.h>
 
-#include "e_os.h"
+#include "internal/nelem.h"
 #include "../testutil.h"
 #include "tu_local.h"
 
index b2d95db..cea81ca 100644 (file)
@@ -14,7 +14,7 @@
 #include <errno.h>
 #include <string.h>
 #include <ctype.h>
-#include "../../e_os.h"
+#include <internal/nelem.h>
 
 /*
  * Output a failed test first line.
index b7f1e87..8814ccb 100644 (file)
@@ -16,7 +16,7 @@
 #include <openssl/asn1.h>
 #include <openssl/x509.h>
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 typedef struct {
     const char *data;
index 0f89260..36f8df6 100644 (file)
@@ -8,7 +8,7 @@
  */
 
 #include <string.h>
-#include "e_os.h"
+#include <internal/nelem.h>
 #include <openssl/x509.h>
 #include <openssl/x509v3.h>
 #include "testutil.h"
index 17517cd..930c01c 100644 (file)
@@ -15,7 +15,7 @@
 #include <openssl/x509.h>
 #include <openssl/x509v3.h>
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 /**********************************************************************
  *
index 7790b82..e8cec0c 100644 (file)
@@ -15,7 +15,7 @@
 #include <openssl/asn1.h>
 #include <openssl/x509.h>
 #include "testutil.h"
-#include "e_os.h"
+#include <internal/nelem.h>
 
 typedef struct {
     const char *data;
index 13a2ffc..d38aeb0 100644 (file)
@@ -16,7 +16,7 @@
 #include <openssl/pem.h>
 #include <openssl/conf.h>
 #include <openssl/err.h>
-#include "e_os.h"
+#include <internal/nelem.h>
 #include "testutil.h"
 
 static int test_certs(int num)