diff options
| author | jsing <> | 2019-01-21 14:19:51 +0000 |
|---|---|---|
| committer | jsing <> | 2019-01-21 14:19:51 +0000 |
| commit | 6f68d2c628cdec2b70bb5312a5f2ea31489070b5 (patch) | |
| tree | 0153bc1e4a8ee619d0024aaf45d51f4ed42f9684 /src | |
| parent | d21bff07f89718efaa729372134210287d078760 (diff) | |
| download | openbsd-6f68d2c628cdec2b70bb5312a5f2ea31489070b5.tar.gz openbsd-6f68d2c628cdec2b70bb5312a5f2ea31489070b5.tar.bz2 openbsd-6f68d2c628cdec2b70bb5312a5f2ea31489070b5.zip | |
Use the actual handshake message type in the certificate request handler.
Diffstat (limited to 'src')
| -rw-r--r-- | src/lib/libssl/tls13_handshake.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/lib/libssl/tls13_handshake.c b/src/lib/libssl/tls13_handshake.c index 160202421c..95f30ded7d 100644 --- a/src/lib/libssl/tls13_handshake.c +++ b/src/lib/libssl/tls13_handshake.c | |||
| @@ -1,4 +1,4 @@ | |||
| 1 | /* $OpenBSD: tls13_handshake.c,v 1.18 2019/01/21 13:45:57 jsing Exp $ */ | 1 | /* $OpenBSD: tls13_handshake.c,v 1.19 2019/01/21 14:19:51 jsing Exp $ */ |
| 2 | /* | 2 | /* |
| 3 | * Copyright (c) 2018-2019 Theo Buehler <tb@openbsd.org> | 3 | * Copyright (c) 2018-2019 Theo Buehler <tb@openbsd.org> |
| 4 | * Copyright (c) 2019 Joel Sing <jsing@openbsd.org> | 4 | * Copyright (c) 2019 Joel Sing <jsing@openbsd.org> |
| @@ -509,15 +509,13 @@ tls13_server_certificate_send(struct tls13_ctx *ctx) | |||
| 509 | int | 509 | int |
| 510 | tls13_server_certificate_request_recv(struct tls13_ctx *ctx) | 510 | tls13_server_certificate_request_recv(struct tls13_ctx *ctx) |
| 511 | { | 511 | { |
| 512 | uint8_t msg_type = 0; /* XXX */ | ||
| 513 | |||
| 514 | /* | 512 | /* |
| 515 | * Thanks to poor state design in the RFC, this function can be called | 513 | * Thanks to poor state design in the RFC, this function can be called |
| 516 | * when we actually have a certificate message instead of a certificate | 514 | * when we actually have a certificate message instead of a certificate |
| 517 | * request... in that case we call the certificate handler after | 515 | * request... in that case we call the certificate handler after |
| 518 | * switching state, to avoid advancing state. | 516 | * switching state, to avoid advancing state. |
| 519 | */ | 517 | */ |
| 520 | if (msg_type == TLS13_MT_CERTIFICATE) { | 518 | if (tls13_handshake_msg_type(ctx->hs_msg) == TLS13_MT_CERTIFICATE) { |
| 521 | ctx->handshake_stage.hs_type |= WITHOUT_CR; | 519 | ctx->handshake_stage.hs_type |= WITHOUT_CR; |
| 522 | return tls13_server_certificate_recv(ctx); | 520 | return tls13_server_certificate_recv(ctx); |
| 523 | } | 521 | } |
