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;