diff options
| author | Nick Mathewson <nickm@torproject.org> | 2006-08-31 18:47:54 +0000 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2006-08-31 18:47:54 +0000 |
| commit | c4ac4bcba348c32af8a4125887fbd69480dd2378 (patch) | |
| tree | a2528ef6bdbd6349d112e30d59ff4d96b6d0a742 | |
| parent | 21a78803584c468f4c0b949d35444695266ff35f (diff) | |
| download | tor-c4ac4bcba348c32af8a4125887fbd69480dd2378.tar.gz tor-c4ac4bcba348c32af8a4125887fbd69480dd2378.zip | |
r8696@Kushana: nickm | 2006-08-31 14:43:44 -0400
Try to appease some warnings with newer gccs that believe that ignoring a return value is okay, but casting a return value and then ignoring it is a sign of madness.
svn:r8312
| -rw-r--r-- | src/common/crypto.c | 7 | ||||
| -rw-r--r-- | src/common/tortls.c | 2 |
2 files changed, 2 insertions, 7 deletions
diff --git a/src/common/crypto.c b/src/common/crypto.c index 6767b637dc..57b504f5ba 100644 --- a/src/common/crypto.c +++ b/src/common/crypto.c @@ -503,12 +503,7 @@ crypto_pk_write_public_key_to_string(crypto_pk_env_t *env, char **dest, } BIO_get_mem_ptr(b, &buf); - BIO_set_close(b, BIO_NOCLOSE); /* so BIO_free doesn't free buf */ - /* XXX The above line generates a warning on new gcc/openssls: - * crypto.c:506: warning: value computed is not used - * Perhaps in new openssls this is a more complex macro and - * we're no longer calling it quite right? -RD - */ + (void)BIO_set_close(b, BIO_NOCLOSE); /* so BIO_free doesn't free buf */ BIO_free(b); tor_assert(buf->length >= 0); diff --git a/src/common/tortls.c b/src/common/tortls.c index 623f460455..072f9709dc 100644 --- a/src/common/tortls.c +++ b/src/common/tortls.c @@ -692,7 +692,7 @@ log_cert_lifetime(X509 *cert, const char *problem) BIO_get_mem_ptr(bio, &buf); s1 = tor_strndup(buf->data, buf->length); - BIO_reset(bio); + (void)BIO_reset(bio); if (!(ASN1_TIME_print(bio, X509_get_notAfter(cert)))) { tls_log_errors(LOG_WARN, "printing certificate lifetime"); goto end; |