The branch master has been updated
via af7e05c7c60b87723efccc01f6d03ebc07cdd93c (commit)
from a8f957686675194d786b41f6e1f7c48bb85723ec (commit)
- Log -----------------------------------------------------------------
commit af7e05c7c60b87723efccc01f6d03ebc07cdd93c
Author: Robert Scheck <rob...@fedoraproject.org>
Date: Tue Feb 14 21:47:25 2017 +0100
Handle negative reply for NNTP STARTTLS in s_client
Reviewed-by: Andy Polyakov <ap...@openssl.org>
Reviewed-by: Rich Salz <rs...@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/2629)
-----------------------------------------------------------------------
Summary of changes:
apps/s_client.c | 18 ++++++++++++++----
1 file changed, 14 insertions(+), 4 deletions(-)
diff --git a/apps/s_client.c b/apps/s_client.c
index 6d96012..2db985d 100644
--- a/apps/s_client.c
+++ b/apps/s_client.c
@@ -2218,7 +2218,16 @@ int s_client_main(int argc, char **argv)
"Didn't find STARTTLS in server response,"
" trying anyway...\n");
BIO_printf(sbio, "STARTTLS\r\n");
- BIO_read(sbio, sbuf, BUFSIZZ);
+ mbuf_len = BIO_read(sbio, mbuf, BUFSIZZ);
+ if (mbuf_len < 0) {
+ BIO_printf(bio_err, "BIO_read failed\n");
+ goto end;
+ }
+ mbuf[mbuf_len] = '\0';
+ if (strstr(mbuf, "382") == NULL) {
+ BIO_printf(bio_err, "STARTTLS failed: %s", mbuf);
+ goto shut;
+ }
}
break;
case PROTO_SIEVE:
@@ -2252,15 +2261,16 @@ int s_client_main(int argc, char **argv)
if (mbuf_len < 0) {
BIO_printf(bio_err, "BIO_read failed\n");
goto end;
- } else if (mbuf_len < 2) {
- BIO_printf(bio_err, "Server does not support STARTTLS.\n");
+ }
+ mbuf[mbuf_len] = '\0';
+ if (mbuf_len < 2) {
+ BIO_printf(bio_err, "STARTTLS failed: %s", mbuf);
goto shut;
}
/*
* According to RFC 5804 ยง 2.2, response codes are case-
* insensitive, make it uppercase but preserve the response.
*/
- mbuf[mbuf_len] = '\0';
strncpy(sbuf, mbuf, 2);
make_uppercase(sbuf);
if (strncmp(sbuf, "OK", 2) != 0) {
_____
openssl-commits mailing list
To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-commits
