diff options
author | Rich Salz <rsalz@akamai.com> | 2016-01-08 10:36:37 -0500 |
---|---|---|
committer | Rich Salz <rsalz@openssl.org> | 2016-01-08 12:48:15 -0500 |
commit | 7ff970ef55a1552e5a1acc6d337250c755b7fd0d (patch) | |
tree | bcac7dcc0807b4d3f032dfa5a5ce8d8c25f37b08 /apps | |
parent | 59fd40d4e5030a7257edd11d758eab1dcebb3787 (diff) |
Portability fix for apps/s_client.c
Make some local variables and a table of them be static.
Reviewed-by: Viktor Dukhovni <viktor@openssl.org>
Diffstat (limited to 'apps')
-rw-r--r-- | apps/s_client.c | 29 |
1 files changed, 16 insertions, 13 deletions
diff --git a/apps/s_client.c b/apps/s_client.c index 441da2417f..152b0fbff8 100644 --- a/apps/s_client.c +++ b/apps/s_client.c @@ -546,20 +546,20 @@ static ossl_ssize_t checked_uint8(const char **inptr, void *out) return 1; } +struct tlsa_field { + void *var; + const char *name; + ossl_ssize_t (*parser)(const char **, void *); +}; + static int tlsa_import_rr(SSL *con, const char *rrdata) { - int ret; - uint8_t usage; - uint8_t selector; - uint8_t mtype; - unsigned char *data = NULL; - const char *cp = rrdata; - ossl_ssize_t len = 0; - struct tlsa_field { - void *var; - const char *name; - ossl_ssize_t (*parser)(const char **, void *); - } tlsa_fields[] = { + /* Not necessary to re-init these values; the "parsers" do that. */ + static uint8_t usage; + static uint8_t selector; + static uint8_t mtype; + static unsigned char *data; + static tlsa_field tlsa_fields[] = { { &usage, "usage", checked_uint8 }, { &selector, "selector", checked_uint8 }, { &mtype, "mtype", checked_uint8 }, @@ -567,6 +567,9 @@ static int tlsa_import_rr(SSL *con, const char *rrdata) { NULL, } }; struct tlsa_field *f; + int ret; + const char *cp = rrdata; + ossl_ssize_t len = 0; for (f = tlsa_fields; f->var; ++f) { /* Returns number of bytes produced, advances cp to next field */ @@ -2424,7 +2427,7 @@ static void print_stuff(BIO *bio, SSL *s, int full) } if ((mdpth = SSL_get0_dane_authority(s, NULL, &mspki)) >= 0) { uint8_t usage, selector, mtype; - (void) SSL_get0_dane_tlsa(s, &usage, &selector, &mtype, NULL, NULL); + mdpth = SSL_get0_dane_tlsa(s, &usage, &selector, &mtype, NULL, NULL); BIO_printf(bio, "DANE TLSA %d %d %d %s at depth %d\n", usage, selector, mtype, (mspki != NULL) ? "TA public key verified certificate" : |