ANNOUNCEMENT: Live Wireshark University & Allegro Packets online APAC Wireshark Training Session
April 17th, 2024 | 14:30-16:00 SGT (UTC+8) | Online

Wireshark-dev: [Wireshark-dev] Drop sql-oracle dissector?

From: Evan Huus <eapache@xxxxxxxxx>
Date: Sat, 16 Mar 2013 22:17:04 -0400
packet-sqloracle.c is currently not being built (not listed in either
Makefile.common or CMakeLists.txt).

It was mentioned in bug #8449 because Moshe noticed that it had a
really egregious buffer overflow condition while doing random code
review.

https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=8449

Having taken a look at the dissector, it is an absolute mess. Tons of
obvious bugs, bad practices, etc. There is no way it would be accepted
as a patch, were it filed new today (and nobody seems to know who
wrote it anyways).

So, since nobody has complained about the fact that it's disabled, and
it looks like nearly as much work to rewrite as it is to fix, I'm
happy just removing it completely.

Thoughts?