Re: New openSAFETY dissector

[Guy Harris <guy () alum mit edu>]

On Mar 24, 2011, at 12:01 AM, Roland Knall wrote:

> Hello
>
> The dissectory has now been fuzz-tested as well, with nearly 250 MB
> worth of various capture files. It passed.
>
> The updated version of the patch is online.

It's checked in.

Unfortunately, the openSAFETY dissector, which is a built-in dissector, registers itself as a heuristic dissector for 
SERCOS III, the dissector for which is a plugin dissector.

Perhaps heur_dissector_add() should report an error to the user, rather than aborting, if the heuristic dissector table 
in question doesn't exist.  If it were to check the WIRESHARK_ABORT_ON_DISSECTOR_BUG environment variable, and:

        if it's set, abort if the heuristic dissector table doesn't exist;

        if it's not set, report an error to the user;

that should allow the build process to catch typos in the heuristic dissector table name but avoid having *Shark crash 
if plugins are missing.

However, if this is the cause of

        http://buildbot.wireshark.org/trunk/builders/Ubuntu-10.04-x64/builds/1452/steps/fuzz-menagerie/logs/stdio

then that raises the question "why isn't the SERCOS III plugin being loaded when we're doing the fuzz testing?"  If the 
plugins aren't getting loaded, that means they're not getting fuzz tested.

This also raises the question of whether the SERCOS III dissector should be made a built-in dissector.
___________________________________________________________________________
Sent via:    Wireshark-dev mailing list <wireshark-dev () wireshark org>
Archives:    http://www.wireshark.org/lists/wireshark-dev
Unsubscribe: https://wireshark.org/mailman/options/wireshark-dev
             mailto:wireshark-dev-request () wireshark org?subject=unsubscribe