Commit c9de4a20 authored by Matt Caswell's avatar Matt Caswell
Browse files

Fix empty NewSessionTicket processing 



Rebasing the state machine code introduced a problem with empty
NewSessionTicket processing. The return value from the
tls_process_new_session_ticket() is supposed to be an enum, but a bare
integer was being used. Unfortunately this is valid C so the compiler
doesn't pick it up.

Reviewed-by: default avatarTim Hudson <tjh@openssl.org>
Reviewed-by: default avatarRichard Levitte <levitte@openssl.org>
parent d13dd4be

ssl/statem/statem_clnt.c

+1 −1
Original line number Diff line number Diff line
@@ -2184,7 +2184,7 @@ enum MSG_PROCESS_RETURN tls_process_new_session_ticket(SSL *s, PACKET *pkt) 


    /* Server is allowed to change its mind and send an empty ticket. */

    if (ticklen == 0)

        return 1;

        return MSG_PROCESS_CONTINUE_READING;



    if (s->session->session_id_length > 0) {

        int i = s->session_ctx->session_cache_mode;