diff options
author | Thomas Gerbet <thomas@gerbet.me> | 2024-05-14 08:39:05 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-05-14 08:39:05 +0200 |
commit | 9ddcaffecdf098822d944d4147dd8da30b4e6843 (patch) | |
tree | c31bb09a94d4cb9c45bb25295d8af8515114a18e | |
parent | 8a4282c38b6cbea9f0989c0eafc6ce1837a26442 (diff) | |
parent | 433a15b9c73eb89d61f5ca2279c1e7119dbbc4b9 (diff) |
Merge pull request #310892 from risicle/ris-nanomq-CVE-2024-31040-r23.11
[23.11] nanomq: add patches for CVE-2024-31040 & CVE-2024-31041
-rw-r--r-- | pkgs/servers/mqtt/nanomq/0.18.2-CVE-2024-31040.patch | 15 | ||||
-rw-r--r-- | pkgs/servers/mqtt/nanomq/default.nix | 4 |
2 files changed, 19 insertions, 0 deletions
diff --git a/pkgs/servers/mqtt/nanomq/0.18.2-CVE-2024-31040.patch b/pkgs/servers/mqtt/nanomq/0.18.2-CVE-2024-31040.patch new file mode 100644 index 0000000000000..0453415e88d64 --- /dev/null +++ b/pkgs/servers/mqtt/nanomq/0.18.2-CVE-2024-31040.patch @@ -0,0 +1,15 @@ +Based on upstream +https://github.com/nanomq/NanoNNG/commit/543ce93017a488c5001ac450c2a2c1ff6dca19d1 +with conflicting formatting changes removed + +--- a/nng/src/sp/protocol/mqtt/mqtt_parser.c ++++ b/nng/src/sp/protocol/mqtt/mqtt_parser.c +@@ -583,7 +583,7 @@ + if (cparam->pro_ver == MQTT_PROTOCOL_VERSION_v5) { + // check length + log_trace("Decoding MQTT V5 Properties"); +- if (pos >= max) ++ if (pos + 4 >= max) + return PROTOCOL_ERROR; + cparam->prop_len = (uint32_t) get_var_integer(packet + pos, + &len_of_var); diff --git a/pkgs/servers/mqtt/nanomq/default.nix b/pkgs/servers/mqtt/nanomq/default.nix index 902b5369df07c..b85b4af1b95b6 100644 --- a/pkgs/servers/mqtt/nanomq/default.nix +++ b/pkgs/servers/mqtt/nanomq/default.nix @@ -52,6 +52,10 @@ in stdenv.mkDerivation (finalAttrs: { fetchSubmodules = true; }; + patches = [ + ./0.18.2-CVE-2024-31040.patch + ]; + postPatch = '' substituteInPlace CMakeLists.txt \ --replace "DESTINATION /etc" "DESTINATION $out/etc" |