From 975e37cd016f86985d16f1ee646e88213494854a Mon Sep 17 00:00:00 2001 From: Nan Xiao Date: Mon, 29 Mar 2021 17:24:01 +0800 Subject: [PATCH] Remove unnecessary BIO_do_handshake()s Since BIO_do_connect() and BIO_do_handshake() are same, no need to invoke BIO_do_handshake() once more after BIO_do_connect(). Reviewed-by: Paul Dale Reviewed-by: Tomas Mraz (Merged from https://github.com/openssl/openssl/pull/14725) --- demos/bio/client-arg.c | 6 ------ demos/bio/client-conf.c | 6 ------ doc/man3/BIO_f_ssl.pod | 5 ----- 3 files changed, 17 deletions(-) diff --git a/demos/bio/client-arg.c b/demos/bio/client-arg.c index 976fefff5d..27c4c98f96 100644 --- a/demos/bio/client-arg.c +++ b/demos/bio/client-arg.c @@ -94,12 +94,6 @@ int main(int argc, char **argv) goto end; } - if (BIO_do_handshake(sbio) <= 0) { - fprintf(stderr, "Error establishing SSL connection\n"); - ERR_print_errors_fp(stderr); - goto end; - } - /* Could examine ssl here to get connection info */ BIO_puts(sbio, "GET / HTTP/1.0\n\n"); diff --git a/demos/bio/client-conf.c b/demos/bio/client-conf.c index 84edc796dc..270df377e7 100644 --- a/demos/bio/client-conf.c +++ b/demos/bio/client-conf.c @@ -102,12 +102,6 @@ int main(int argc, char **argv) goto end; } - if (BIO_do_handshake(sbio) <= 0) { - fprintf(stderr, "Error establishing SSL connection\n"); - ERR_print_errors_fp(stderr); - goto end; - } - /* Could examine ssl here to get connection info */ BIO_puts(sbio, "GET / HTTP/1.0\n\n"); diff --git a/doc/man3/BIO_f_ssl.pod b/doc/man3/BIO_f_ssl.pod index 6a21e0c41c..8eee317c7a 100644 --- a/doc/man3/BIO_f_ssl.pod +++ b/doc/man3/BIO_f_ssl.pod @@ -188,11 +188,6 @@ unencrypted example in L. ERR_print_errors_fp(stderr); exit(1); } - if (BIO_do_handshake(sbio) <= 0) { - fprintf(stderr, "Error establishing SSL connection\n"); - ERR_print_errors_fp(stderr); - exit(1); - } /* XXX Could examine ssl here to get connection info */