Fix some clang warnings
authorMatt Caswell <matt@openssl.org>
Tue, 19 Jul 2016 10:34:21 +0000 (11:34 +0100)
committerMatt Caswell <matt@openssl.org>
Fri, 19 Aug 2016 12:52:40 +0000 (13:52 +0100)
Clang was complaining about some unused functions. Moving the stack
declaration to the header seems to sort it. Also the certstatus variable
in dtlstest needed to be declared static.

Reviewed-by: Richard Levitte <levitte@openssl.org>
test/dtlstest.c
test/ssltestlib.c
test/ssltestlib.h

index 06db5d2..4d497af 100644 (file)
@@ -23,7 +23,7 @@ static char *privkey = NULL;
 
 #define DUMMY_CERT_STATUS_LEN  12
 
-unsigned char certstatus[] = {
+static unsigned char certstatus[] = {
     SSL3_RT_HANDSHAKE, /* Content type */
     0xfe, 0xfd, /* Record version */
     0, 1, /* Epoch */
index a6ca5ff..ff5cbed 100644 (file)
@@ -238,8 +238,6 @@ typedef struct mempacket_st {
     unsigned int type;
 } MEMPACKET;
 
-DEFINE_STACK_OF(MEMPACKET)
-
 static void mempacket_free(MEMPACKET *pkt)
 {
     if (pkt->data != NULL)
index 27f7547..bd9272f 100644 (file)
@@ -33,4 +33,8 @@ void bio_s_mempacket_test_free(void);
 int mempacket_test_inject(BIO *bio, const char *in, int inl, int pktnum,
                           int type);
 
+typedef struct mempacket_st MEMPACKET;
+
+DEFINE_STACK_OF(MEMPACKET)
+
 #endif /* HEADER_SSLTESTLIB_H */