Wireshark · Wireshark-bugs: [Wireshark-bugs] [Bug 3884] Assertion caused by fuzz test file

[bugzilla-daemon@xxxxxxxxxxxxx]

Alexis La Goutte changed bug 3884

What	Removed	Added
CC		alexis.lagoutte@gmail.com

Comment # 18 on bug 3884 from Alexis La Goutte

May be close this bug ?

Also... Gerald any reason to have different code for X25 ? (Revision 44439)

                case FT_AX25:
-                       DISSECTOR_ASSERT(length == 7);
+                       if (length != 7) {
+                               length_error = length < 7 ? TRUE : FALSE;
+                               expert_add_info_format(NULL, tree,
PI_MALFORMED, PI_ERROR, "Trying to fetch an AX.25 address with length %d
+                               THROW(ReportedBoundsError);
+                       }
                        proto_tree_set_ax25_tvb(new_fi, tvb, start);
                        break;

(for others : 
                case FT_IPv6:
-                       DISSECTOR_ASSERT(length >= 0 && length <= FT_IPv6_LEN);
+                       if (length != FT_IPv6_LEN) {
+                               length_error = length < FT_IPv6_LEN ? TRUE :
FALSE;
+                               report_type_length_mismatch(tree, "an IPv6
address", length, length_error);
+                       }

---

      You are receiving this mail because:

• You are the assignee for the bug.
• You are watching all bug changes.