diff options
| author | Theo Buehler <tb@cvs.openbsd.org> | 2022-01-24 13:53:30 +0000 |
|---|---|---|
| committer | Theo Buehler <tb@cvs.openbsd.org> | 2022-01-24 13:53:30 +0000 |
| commit | d51fc4d482b91b37c8b4540780456268d0c0fbf4 (patch) | |
| tree | 3cd5b942832ae301ec6abeb836cd25e75a02010b /lib/libssl/ssl_clnt.c | |
| parent | 0853376869b39187468fcee7a40cdee025f641d5 (diff) | |
Garbage collect unused ret in ssl3_get_new_session_ticket()
ret is used to bubble up an error from ssl3_get_message() and is unused
otherwise. Zap dead store and silly ret = 1; return ret;
ok inoguchi jsing
Diffstat (limited to 'lib/libssl/ssl_clnt.c')
| -rw-r--r-- | lib/libssl/ssl_clnt.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/lib/libssl/ssl_clnt.c b/lib/libssl/ssl_clnt.c index 7e4ef44b9ca..a402535c7dc 100644 --- a/lib/libssl/ssl_clnt.c +++ b/lib/libssl/ssl_clnt.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ssl_clnt.c,v 1.138 2022/01/24 13:51:48 tb Exp $ */ +/* $OpenBSD: ssl_clnt.c,v 1.139 2022/01/24 13:53:29 tb Exp $ */ /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) * All rights reserved. * @@ -1602,8 +1602,6 @@ ssl3_get_new_session_ticket(SSL *s) SSL3_ST_CR_SESSION_TICKET_B, -1, 16384)) <= 0) return ret; - ret = 0; - if (S3I(s)->hs.tls12.message_type == SSL3_MT_FINISHED) { S3I(s)->hs.tls12.reuse_message = 1; return (1); @@ -1655,8 +1653,9 @@ ssl3_get_new_session_ticket(SSL *s) EVP_Digest(CBS_data(&session_ticket), CBS_len(&session_ticket), s->session->session_id, &s->session->session_id_length, EVP_sha256(), NULL); - ret = 1; - return (ret); + + return (1); + fatal_err: ssl3_send_alert(s, SSL3_AL_FATAL, al); err: |