Wireshark-bugs: [Wireshark-bugs] [Bug 7729] Full support of RFC2428 in FTP dissector
https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7729
Alexander Koeppe <format_c@xxxxxxxxx> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #9210|0 |1
is obsolete| |
--- Comment #14 from Alexander Koeppe <format_c@xxxxxxxxx> 2012-09-25 14:20:37 PDT ---
Created attachment 9225
--> https://bugs.wireshark.org/bugzilla/attachment.cgi?id=9225
Latest stable patch implementing RFC2428
(In reply to comment #12)
>
> Unfortunately, I still think we need to fix some things.
>
OK sure.
>
> For such malformed addresses, we must either flag up an error or at least stop
> the dissection at that point.
>
I would choose option 1.
>
> I worked on your patch a bit, please see my attachment.
>
Thanks. Looks good. I just removed the g_warning for debug output.
> The main problem that remains is that addresses like
>
> 1:2:3:4:5:6:7:::::8 are not recognized as malformed. Could you have a look?
OK fixed.
The new attached patch includes the above fix and the report to the user using
the expert interface.
So that your fuzzed pcap is now also being claimed as malformed (8 words + ::
== invalid IPv6).
--
Configure bugmail: https://bugs.wireshark.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are watching all bug changes.
