From c9de4a20fa21101208cf311c5437d9c358ba1f07 Mon Sep 17 00:00:00 2001 From: Matt Caswell Date: Thu, 22 Oct 2015 15:02:14 +0100 Subject: [PATCH] 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: Tim Hudson Reviewed-by: Richard Levitte --- ssl/statem/statem_clnt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ssl/statem/statem_clnt.c b/ssl/statem/statem_clnt.c index dbba36bc9e..d0c6b1292e 100644 --- a/ssl/statem/statem_clnt.c +++ b/ssl/statem/statem_clnt.c @@ -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; -- 2.34.1