diff -Nru nghttp2-1.43.0/debian/changelog nghttp2-1.43.0/debian/changelog --- nghttp2-1.43.0/debian/changelog 2023-10-11 17:50:35.000000000 +0000 +++ nghttp2-1.43.0/debian/changelog 2024-04-18 07:14:38.000000000 +0000 @@ -1,3 +1,14 @@ +nghttp2 (1.43.0-1ubuntu0.2) jammy-security; urgency=medium + + * SECURITY UPDATE: HTTP/2 protocol denial of service + - debian/patches/CVE-2024-28182-1.patch: Add + nghttp2_option_set_max_continuations + - debian/patches/CVE-2024-28182-2.patch: Limit CONTINUATION frames + following an incoming HEADER frame + - CVE-2024-28182 + + -- Fabian Toepfer Thu, 18 Apr 2024 09:14:38 +0200 + nghttp2 (1.43.0-1ubuntu0.1) jammy-security; urgency=medium * SECURITY UPDATE: HTTP/2 protocol denial of service diff -Nru nghttp2-1.43.0/debian/patches/CVE-2024-28182-1.patch nghttp2-1.43.0/debian/patches/CVE-2024-28182-1.patch --- nghttp2-1.43.0/debian/patches/CVE-2024-28182-1.patch 1970-01-01 00:00:00.000000000 +0000 +++ nghttp2-1.43.0/debian/patches/CVE-2024-28182-1.patch 2024-04-17 13:24:26.000000000 +0000 @@ -0,0 +1,88 @@ +From d71a4668c6bead55805d18810d633fbb98315af9 Mon Sep 17 00:00:00 2001 +From: Tatsuhiro Tsujikawa +Date: Sat, 9 Mar 2024 16:48:10 +0900 +Subject: [PATCH] Add nghttp2_option_set_max_continuations + +--- + doc/Makefile.am | 1 + + lib/includes/nghttp2/nghttp2.h | 11 +++++++++++ + lib/nghttp2_option.c | 5 +++++ + lib/nghttp2_option.h | 5 +++++ + lib/nghttp2_session.c | 4 ++++ + 5 files changed, 26 insertions(+) + +--- nghttp2-1.43.0.orig/doc/Makefile.am ++++ nghttp2-1.43.0/doc/Makefile.am +@@ -68,6 +68,7 @@ APIDOCS= \ + nghttp2_option_set_no_recv_client_magic.rst \ + nghttp2_option_set_peer_max_concurrent_streams.rst \ + nghttp2_option_set_user_recv_extension_type.rst \ ++ nghttp2_option_set_max_continuations.rst \ + nghttp2_option_set_max_outbound_ack.rst \ + nghttp2_option_set_max_settings.rst \ + nghttp2_option_set_stream_reset_rate_limit.rst \ +--- nghttp2-1.43.0.orig/lib/includes/nghttp2/nghttp2.h ++++ nghttp2-1.43.0/lib/includes/nghttp2/nghttp2.h +@@ -2739,6 +2739,17 @@ nghttp2_option_set_stream_reset_rate_lim + /** + * @function + * ++ * This function sets the maximum number of CONTINUATION frames ++ * following an incoming HEADER frame. If more than those frames are ++ * received, the remote endpoint is considered to be misbehaving and ++ * session will be closed. The default value is 8. ++ */ ++NGHTTP2_EXTERN void nghttp2_option_set_max_continuations(nghttp2_option *option, ++ size_t val); ++ ++/** ++ * @function ++ * + * Initializes |*session_ptr| for client use. The all members of + * |callbacks| are copied to |*session_ptr|. Therefore |*session_ptr| + * does not store |callbacks|. The |user_data| is an arbitrary user +--- nghttp2-1.43.0.orig/lib/nghttp2_option.c ++++ nghttp2-1.43.0/lib/nghttp2_option.c +@@ -133,3 +133,8 @@ void nghttp2_option_set_stream_reset_rat + option->stream_reset_burst = burst; + option->stream_reset_rate = rate; + } ++ ++void nghttp2_option_set_max_continuations(nghttp2_option *option, size_t val) { ++ option->opt_set_mask |= NGHTTP2_OPT_MAX_CONTINUATIONS; ++ option->max_continuations = val; ++} +--- nghttp2-1.43.0.orig/lib/nghttp2_option.h ++++ nghttp2-1.43.0/lib/nghttp2_option.h +@@ -69,6 +69,7 @@ typedef enum { + NGHTTP2_OPT_MAX_OUTBOUND_ACK = 1 << 11, + NGHTTP2_OPT_MAX_SETTINGS = 1 << 12, + NGHTTP2_OPT_STREAM_RESET_RATE_LIMIT = 1 << 15, ++ NGHTTP2_OPT_MAX_CONTINUATIONS = 1 << 16, + } nghttp2_option_flag; + + /** +@@ -97,6 +98,10 @@ struct nghttp2_option { + */ + size_t max_settings; + /** ++ * NGHTTP2_OPT_MAX_CONTINUATIONS ++ */ ++ size_t max_continuations; ++ /** + * Bitwise OR of nghttp2_option_flag to determine that which fields + * are specified. + */ +--- nghttp2-1.43.0.orig/lib/nghttp2_session.c ++++ nghttp2-1.43.0/lib/nghttp2_session.c +@@ -538,6 +538,10 @@ static int session_new(nghttp2_session * + option->stream_reset_burst, + option->stream_reset_rate); + } ++ ++ if (option->opt_set_mask & NGHTTP2_OPT_MAX_CONTINUATIONS) { ++ (*session_ptr)->max_continuations = option->max_continuations; ++ } + } + + rv = nghttp2_hd_deflate_init2(&(*session_ptr)->hd_deflater, diff -Nru nghttp2-1.43.0/debian/patches/CVE-2024-28182-2.patch nghttp2-1.43.0/debian/patches/CVE-2024-28182-2.patch --- nghttp2-1.43.0/debian/patches/CVE-2024-28182-2.patch 1970-01-01 00:00:00.000000000 +0000 +++ nghttp2-1.43.0/debian/patches/CVE-2024-28182-2.patch 2024-04-17 13:24:30.000000000 +0000 @@ -0,0 +1,103 @@ +From 00201ecd8f982da3b67d4f6868af72a1b03b14e0 Mon Sep 17 00:00:00 2001 +From: Tatsuhiro Tsujikawa +Date: Sat, 9 Mar 2024 16:26:42 +0900 +Subject: [PATCH] Limit CONTINUATION frames following an incoming HEADER frame + +--- + lib/includes/nghttp2/nghttp2.h | 7 ++++++- + lib/nghttp2_helper.c | 2 ++ + lib/nghttp2_session.c | 7 +++++++ + lib/nghttp2_session.h | 10 ++++++++++ + 4 files changed, 25 insertions(+), 1 deletion(-) + +diff --git a/lib/includes/nghttp2/nghttp2.h b/lib/includes/nghttp2/nghttp2.h +index 889176097d..a9629c7823 100644 +--- a/lib/includes/nghttp2/nghttp2.h ++++ b/lib/includes/nghttp2/nghttp2.h +@@ -466,7 +466,12 @@ typedef enum { + * exhaustion on server side to send these frames forever and does + * not read network. + */ +- NGHTTP2_ERR_FLOODED = -904 ++ NGHTTP2_ERR_FLOODED = -904, ++ /** ++ * When a local endpoint receives too many CONTINUATION frames ++ * following a HEADER frame. ++ */ ++ NGHTTP2_ERR_TOO_MANY_CONTINUATIONS = -905, + } nghttp2_error; + + /** +diff --git a/lib/nghttp2_helper.c b/lib/nghttp2_helper.c +index 93dd4754b7..b3563d98e0 100644 +--- a/lib/nghttp2_helper.c ++++ b/lib/nghttp2_helper.c +@@ -336,6 +336,8 @@ const char *nghttp2_strerror(int error_code) { + "closed"; + case NGHTTP2_ERR_TOO_MANY_SETTINGS: + return "SETTINGS frame contained more than the maximum allowed entries"; ++ case NGHTTP2_ERR_TOO_MANY_CONTINUATIONS: ++ return "Too many CONTINUATION frames following a HEADER frame"; + default: + return "Unknown error code"; + } +diff --git a/lib/nghttp2_session.c b/lib/nghttp2_session.c +index ea4fcbec57..fc4c77a3f0 100644 +--- a/lib/nghttp2_session.c ++++ b/lib/nghttp2_session.c +@@ -497,6 +497,7 @@ static int session_new(nghttp2_session **session_ptr, + (*session_ptr)->max_send_header_block_length = NGHTTP2_MAX_HEADERSLEN; + (*session_ptr)->max_outbound_ack = NGHTTP2_DEFAULT_MAX_OBQ_FLOOD_ITEM; + (*session_ptr)->max_settings = NGHTTP2_DEFAULT_MAX_SETTINGS; ++ (*session_ptr)->max_continuations = NGHTTP2_DEFAULT_MAX_CONTINUATIONS; + + if (option) { + if ((option->opt_set_mask & NGHTTP2_OPT_NO_AUTO_WINDOW_UPDATE) && +@@ -6826,6 +6827,8 @@ nghttp2_ssize nghttp2_session_mem_recv2(nghttp2_session *session, + } + } + session_inbound_frame_reset(session); ++ ++ session->num_continuations = 0; + } + break; + } +@@ -6947,6 +6950,10 @@ nghttp2_ssize nghttp2_session_mem_recv2(nghttp2_session *session, + } + #endif /* DEBUGBUILD */ + ++ if (++session->num_continuations > session->max_continuations) { ++ return NGHTTP2_ERR_TOO_MANY_CONTINUATIONS; ++ } ++ + readlen = inbound_frame_buf_read(iframe, in, last); + in += readlen; + +diff --git a/lib/nghttp2_session.h b/lib/nghttp2_session.h +index b119329a04..ef8f7b27d6 100644 +--- a/lib/nghttp2_session.h ++++ b/lib/nghttp2_session.h +@@ -110,6 +110,10 @@ typedef struct { + #define NGHTTP2_DEFAULT_STREAM_RESET_BURST 1000 + #define NGHTTP2_DEFAULT_STREAM_RESET_RATE 33 + ++/* The default max number of CONTINUATION frames following an incoming ++ HEADER frame. */ ++#define NGHTTP2_DEFAULT_MAX_CONTINUATIONS 8 ++ + /* Internal state when receiving incoming frame */ + typedef enum { + /* Receiving frame header */ +@@ -290,6 +294,12 @@ struct nghttp2_session { + size_t max_send_header_block_length; + /* The maximum number of settings accepted per SETTINGS frame. */ + size_t max_settings; ++ /* The maximum number of CONTINUATION frames following an incoming ++ HEADER frame. */ ++ size_t max_continuations; ++ /* The number of CONTINUATION frames following an incoming HEADER ++ frame. This variable is reset when END_HEADERS flag is seen. */ ++ size_t num_continuations; + /* Next Stream ID. Made unsigned int to detect >= (1 << 31). */ + uint32_t next_stream_id; + /* The last stream ID this session initiated. For client session, diff -Nru nghttp2-1.43.0/debian/patches/series nghttp2-1.43.0/debian/patches/series --- nghttp2-1.43.0/debian/patches/series 2023-10-11 17:43:38.000000000 +0000 +++ nghttp2-1.43.0/debian/patches/series 2024-04-17 13:24:30.000000000 +0000 @@ -2,3 +2,5 @@ 0002-Workaround-for-963648.patch 0003-beutify-theme.js-to-show-better-what-it-does.patch CVE-2023-44487.patch +CVE-2024-28182-1.patch +CVE-2024-28182-2.patch