Commit 1b0286a3 authored by Matt Caswell's avatar Matt Caswell
Browse files

Fix a memory leak 



When we call tls_collect_extensions() we need to free up the raw
extensions data later.

Perl changes reviewed by Richard Levitte. Non-perl changes reviewed by Rich
Salz

Reviewed-by: default avatarRich Salz <rsalz@openssl.org>
Reviewed-by: default avatarRichard Levitte <levitte@openssl.org>
parent 7caf619f

ssl/statem/statem_clnt.c

+4 −0
Original line number Diff line number Diff line
@@ -1352,10 +1352,12 @@ MSG_PROCESS_RETURN tls_process_server_hello(SSL *s, PACKET *pkt) 
        goto f_err;

    }



    OPENSSL_free(extensions);

    return MSG_PROCESS_CONTINUE_READING;

 f_err:

    ssl3_send_alert(s, SSL3_AL_FATAL, al);

    ossl_statem_set_error(s);

    OPENSSL_free(extensions);

    return MSG_PROCESS_ERROR;

}
@@ -3117,11 +3119,13 @@ static MSG_PROCESS_RETURN tls_process_encrypted_extensions(SSL *s, PACKET *pkt) 
                                         rawexts, &al))

        goto err;



    OPENSSL_free(rawexts);

    return MSG_PROCESS_CONTINUE_READING;



 err:

    ssl3_send_alert(s, SSL3_AL_FATAL, al);

    ossl_statem_set_error(s);

    OPENSSL_free(rawexts);

    return MSG_PROCESS_ERROR;

}