Wireshark-dev: Re: [Wireshark-dev] proto_filter_names hash collision

From: Jaap Keuter <jaap.keuter@xxxxxxxxx>
Date: Wed, 21 Nov 2007 02:15:18 +0100
Guy Harris wrote:
Kukosa, Tomas wrote:
It seems that we have reached critical number of protocols.
I have met collision of built in "p7" protoco with my private "nu" protocol. They both have the same g_str_hash() value. Does it make sence to create hash from the protocol filter name which has usually 2-4 characters? BTW the g_str_hash() for full protocol name takes quite much time during startup. Does it make sence to check full protocol name duplicity? Is not it enough to check short name and filter name duplicity?

Does it make sense to do all those sanity checks only if a special command-line flag is supplied to TShark, so that the checks are done as part of the build process, rather than being done every time you run Wireshark or TShark?

Yes, because you ever know what cocktail of plugins the user has added to its installation.

Thanx,
Jaap