Resolves: #2152064 - CVE-2022-3190 wireshark: f5ethtrailer Infinite loop in legacy style dissector
This commit is contained in:
Michal Ruprich
parent
1950bdf882
commit
0ef4b9de7d
142
wireshark-0011-cve-2022-3190.patch
Normal file
142
wireshark-0011-cve-2022-3190.patch
Normal file
@ -0,0 +1,142 @@
|
|||||||
|
From 0f27a83c5692b2afebe6e6934c1051f76aa2ecf9 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Jason Cohen <kryojenik2@gmail.com>
|
||||||
|
Date: Wed, 31 Aug 2022 11:10:17 -0500
|
||||||
|
Subject: [PATCH] f5ethtrailer: Improve "old-style" heuristic
|
||||||
|
|
||||||
|
Remove a chance for an infinate loop in the disection heuristic.
|
||||||
|
---
|
||||||
|
epan/dissectors/packet-f5ethtrailer.c | 108 +++++++++++++-------------
|
||||||
|
1 file changed, 56 insertions(+), 52 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/epan/dissectors/packet-f5ethtrailer.c b/epan/dissectors/packet-f5ethtrailer.c
|
||||||
|
index b2ba8f899d..915348ea83 100644
|
||||||
|
--- a/epan/dissectors/packet-f5ethtrailer.c
|
||||||
|
+++ b/epan/dissectors/packet-f5ethtrailer.c
|
||||||
|
@@ -2751,69 +2751,73 @@ dissect_dpt_trailer(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *d
|
||||||
|
static gint
|
||||||
|
dissect_old_trailer(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data)
|
||||||
|
{
|
||||||
|
- proto_tree *type_tree = NULL;
|
||||||
|
- proto_item *ti = NULL;
|
||||||
|
guint offset = 0;
|
||||||
|
- guint processed = 0;
|
||||||
|
- f5eth_tap_data_t *tdata = (f5eth_tap_data_t *)data;
|
||||||
|
- guint8 type;
|
||||||
|
- guint8 len;
|
||||||
|
- guint8 ver;
|
||||||
|
|
||||||
|
/* While we still have data in the trailer. For old format trailers, this needs
|
||||||
|
* type, length, version (3 bytes) and for new format trailers, the magic header (4 bytes).
|
||||||
|
* All old format trailers are at least 4 bytes long, so just check for length of magic.
|
||||||
|
*/
|
||||||
|
- while (tvb_reported_length_remaining(tvb, offset)) {
|
||||||
|
- type = tvb_get_guint8(tvb, offset);
|
||||||
|
- len = tvb_get_guint8(tvb, offset + F5_OFF_LENGTH) + F5_OFF_VERSION;
|
||||||
|
- ver = tvb_get_guint8(tvb, offset + F5_OFF_VERSION);
|
||||||
|
-
|
||||||
|
- if (len <= tvb_reported_length_remaining(tvb, offset) && type >= F5TYPE_LOW
|
||||||
|
- && type <= F5TYPE_HIGH && len >= F5_MIN_SANE && len <= F5_MAX_SANE
|
||||||
|
- && ver <= F5TRAILER_VER_MAX) {
|
||||||
|
- /* Parse out the specified trailer. */
|
||||||
|
- switch (type) {
|
||||||
|
- case F5TYPE_LOW:
|
||||||
|
- ti = proto_tree_add_item(tree, hf_low_id, tvb, offset, len, ENC_NA);
|
||||||
|
- type_tree = proto_item_add_subtree(ti, ett_f5ethtrailer_low);
|
||||||
|
-
|
||||||
|
- processed = dissect_low_trailer(tvb, pinfo, type_tree, offset, len, ver, tdata);
|
||||||
|
- if (processed > 0) {
|
||||||
|
- tdata->trailer_len += processed;
|
||||||
|
- tdata->noise_low = 1;
|
||||||
|
- }
|
||||||
|
- break;
|
||||||
|
- case F5TYPE_MED:
|
||||||
|
- ti = proto_tree_add_item(tree, hf_med_id, tvb, offset, len, ENC_NA);
|
||||||
|
- type_tree = proto_item_add_subtree(ti, ett_f5ethtrailer_med);
|
||||||
|
-
|
||||||
|
- processed = dissect_med_trailer(tvb, pinfo, type_tree, offset, len, ver, tdata);
|
||||||
|
- if (processed > 0) {
|
||||||
|
- tdata->trailer_len += processed;
|
||||||
|
- tdata->noise_med = 1;
|
||||||
|
- }
|
||||||
|
- break;
|
||||||
|
- case F5TYPE_HIGH:
|
||||||
|
- ti = proto_tree_add_item(tree, hf_high_id, tvb, offset, len, ENC_NA);
|
||||||
|
- type_tree = proto_item_add_subtree(ti, ett_f5ethtrailer_high);
|
||||||
|
-
|
||||||
|
- processed =
|
||||||
|
- dissect_high_trailer(tvb, pinfo, type_tree, offset, len, ver, tdata);
|
||||||
|
- if (processed > 0) {
|
||||||
|
- tdata->trailer_len += processed;
|
||||||
|
- tdata->noise_high = 1;
|
||||||
|
- }
|
||||||
|
- break;
|
||||||
|
+ while (tvb_reported_length_remaining(tvb, offset) >= F5_MIN_SANE) {
|
||||||
|
+ /* length field does not include the type and length bytes. Add them back in */
|
||||||
|
+ guint8 len = tvb_get_guint8(tvb, offset + F5_OFF_LENGTH) + F5_OFF_VERSION;
|
||||||
|
+ if (len > tvb_reported_length_remaining(tvb, offset)
|
||||||
|
+ || len < F5_MIN_SANE || len > F5_MAX_SANE) {
|
||||||
|
+ /* Invalid length - either a malformed trailer, corrupt packet, or not f5ethtrailer */
|
||||||
|
+ return offset;
|
||||||
|
+ }
|
||||||
|
+ guint8 type = tvb_get_guint8(tvb, offset);
|
||||||
|
+ guint8 ver = tvb_get_guint8(tvb, offset + F5_OFF_VERSION);
|
||||||
|
+
|
||||||
|
+ /* Parse out the specified trailer. */
|
||||||
|
+ proto_tree *type_tree = NULL;
|
||||||
|
+ proto_item *ti = NULL;
|
||||||
|
+ f5eth_tap_data_t *tdata = (f5eth_tap_data_t *)data;
|
||||||
|
+ guint processed = 0;
|
||||||
|
+
|
||||||
|
+ switch (type) {
|
||||||
|
+ case F5TYPE_LOW:
|
||||||
|
+ ti = proto_tree_add_item(tree, hf_low_id, tvb, offset, len, ENC_NA);
|
||||||
|
+ type_tree = proto_item_add_subtree(ti, ett_f5ethtrailer_low);
|
||||||
|
+
|
||||||
|
+ processed = dissect_low_trailer(tvb, pinfo, type_tree, offset, len, ver, tdata);
|
||||||
|
+ if (processed > 0) {
|
||||||
|
+ tdata->trailer_len += processed;
|
||||||
|
+ tdata->noise_low = 1;
|
||||||
|
}
|
||||||
|
- if (processed == 0) {
|
||||||
|
- proto_item_set_len(ti, 1);
|
||||||
|
- return offset;
|
||||||
|
+ break;
|
||||||
|
+ case F5TYPE_MED:
|
||||||
|
+ ti = proto_tree_add_item(tree, hf_med_id, tvb, offset, len, ENC_NA);
|
||||||
|
+ type_tree = proto_item_add_subtree(ti, ett_f5ethtrailer_med);
|
||||||
|
+
|
||||||
|
+ processed = dissect_med_trailer(tvb, pinfo, type_tree, offset, len, ver, tdata);
|
||||||
|
+ if (processed > 0) {
|
||||||
|
+ tdata->trailer_len += processed;
|
||||||
|
+ tdata->noise_med = 1;
|
||||||
|
+ }
|
||||||
|
+ break;
|
||||||
|
+ case F5TYPE_HIGH:
|
||||||
|
+ ti = proto_tree_add_item(tree, hf_high_id, tvb, offset, len, ENC_NA);
|
||||||
|
+ type_tree = proto_item_add_subtree(ti, ett_f5ethtrailer_high);
|
||||||
|
+
|
||||||
|
+ processed =
|
||||||
|
+ dissect_high_trailer(tvb, pinfo, type_tree, offset, len, ver, tdata);
|
||||||
|
+ if (processed > 0) {
|
||||||
|
+ tdata->trailer_len += processed;
|
||||||
|
+ tdata->noise_high = 1;
|
||||||
|
}
|
||||||
|
+ break;
|
||||||
|
+ default:
|
||||||
|
+ /* Unknown type - malformed trailer, corrupt packet, or not f5ethtrailer - bali out*/
|
||||||
|
+ return offset;
|
||||||
|
+ }
|
||||||
|
+ if (processed == 0) {
|
||||||
|
+ /* couldn't process trailer - bali out */
|
||||||
|
+ proto_item_set_len(ti, 1);
|
||||||
|
+ return offset;
|
||||||
|
}
|
||||||
|
offset += processed;
|
||||||
|
}
|
||||||
|
-return offset;
|
||||||
|
+ return offset;
|
||||||
|
} /* dissect_old_trailer() */
|
||||||
|
|
||||||
|
/*---------------------------------------------------------------------------*/
|
||||||
|
--
|
||||||
|
GitLab
|
||||||
@ -6,7 +6,7 @@
|
|||||||
Summary: Network traffic analyzer
|
Summary: Network traffic analyzer
|
||||||
Name: wireshark
|
Name: wireshark
|
||||||
Version: 3.4.10
|
Version: 3.4.10
|
||||||
Release: 3%{?dist}
|
Release: 4%{?dist}
|
||||||
Epoch: 1
|
Epoch: 1
|
||||||
License: GPL+
|
License: GPL+
|
||||||
Url: http://www.wireshark.org/
|
Url: http://www.wireshark.org/
|
||||||
@ -29,6 +29,7 @@ Patch0007: wireshark-0007-cmakelists.patch
|
|||||||
#Patch0008: wireshark-0008-move-glib.patch
|
#Patch0008: wireshark-0008-move-glib.patch
|
||||||
Patch0009: wireshark-0009-smc-support.patch
|
Patch0009: wireshark-0009-smc-support.patch
|
||||||
Patch0010: wireshark-0010-fips-ripemd160.patch
|
Patch0010: wireshark-0010-fips-ripemd160.patch
|
||||||
|
Patch0011: wireshark-0011-cve-2022-3190.patch
|
||||||
|
|
||||||
#install tshark together with wireshark GUI
|
#install tshark together with wireshark GUI
|
||||||
Requires: %{name}-cli = %{epoch}:%{version}-%{release}
|
Requires: %{name}-cli = %{epoch}:%{version}-%{release}
|
||||||
@ -276,6 +277,9 @@ fi
|
|||||||
%{_libdir}/pkgconfig/%{name}.pc
|
%{_libdir}/pkgconfig/%{name}.pc
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Fri Jan 20 2023 Michal Ruprich <mruprich@redhat.com> - 1:3.4.10-4
|
||||||
|
- Resolves: #2152064 - CVE-2022-3190 wireshark: f5ethtrailer Infinite loop in legacy style dissector
|
||||||
|
|
||||||
* Thu Jan 19 2023 Michal Ruprich <mruprich@redhat.com> - 1:3.4.10-3
|
* Thu Jan 19 2023 Michal Ruprich <mruprich@redhat.com> - 1:3.4.10-3
|
||||||
- Resolves: #2083581 - capinfos aborts in FIPS
|
- Resolves: #2083581 - capinfos aborts in FIPS
|
||||||
|
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user