diff options
Diffstat (limited to 'src')
| -rw-r--r-- | src/lib/libssl/ssl_tlsext.c | 20 |
1 files changed, 3 insertions, 17 deletions
diff --git a/src/lib/libssl/ssl_tlsext.c b/src/lib/libssl/ssl_tlsext.c index 14cf6fce84..6649baf291 100644 --- a/src/lib/libssl/ssl_tlsext.c +++ b/src/lib/libssl/ssl_tlsext.c | |||
| @@ -1,4 +1,4 @@ | |||
| 1 | /* $OpenBSD: ssl_tlsext.c,v 1.145 2024/03/27 22:27:09 beck Exp $ */ | 1 | /* $OpenBSD: ssl_tlsext.c,v 1.146 2024/03/28 00:22:35 beck Exp $ */ |
| 2 | /* | 2 | /* |
| 3 | * Copyright (c) 2016, 2017, 2019 Joel Sing <jsing@openbsd.org> | 3 | * Copyright (c) 2016, 2017, 2019 Joel Sing <jsing@openbsd.org> |
| 4 | * Copyright (c) 2017 Doug Hogan <doug@openbsd.org> | 4 | * Copyright (c) 2017 Doug Hogan <doug@openbsd.org> |
| @@ -324,22 +324,8 @@ static int | |||
| 324 | tlsext_supportedgroups_client_process(SSL *s, uint16_t msg_type, CBS *cbs, | 324 | tlsext_supportedgroups_client_process(SSL *s, uint16_t msg_type, CBS *cbs, |
| 325 | int *alert) | 325 | int *alert) |
| 326 | { | 326 | { |
| 327 | /* | 327 | /* Servers should not send this extension per the RFC. */ |
| 328 | * Servers should not send this extension per the RFC. | 328 | return 0; |
| 329 | * | ||
| 330 | * However, certain F5 BIG-IP systems incorrectly send it. This bug is | ||
| 331 | * from at least 2014 but as of 2017, there are still large sites with | ||
| 332 | * this unpatched in production. As a result, we need to currently skip | ||
| 333 | * over the extension and ignore its content: | ||
| 334 | * | ||
| 335 | * https://support.f5.com/csp/article/K37345003 | ||
| 336 | */ | ||
| 337 | if (!CBS_skip(cbs, CBS_len(cbs))) { | ||
| 338 | *alert = SSL_AD_INTERNAL_ERROR; | ||
| 339 | return 0; | ||
| 340 | } | ||
| 341 | |||
| 342 | return 1; | ||
| 343 | } | 329 | } |
| 344 | 330 | ||
| 345 | /* | 331 | /* |
