Wireshark · Ethereal-dev: Re: [Ethereal-dev] patch for packet-sctp.c

[Guy Harris <guy@xxxxxxxxxx>]

> please find attached a patch file against the current CVS version for
> packet-sctp.c.

Checked in.

I changed the name of the new preference to "checksum" - the full name
is "sctp.checksum", so there's no need for an extra "sctp_" in it.

(From some patches submitted over the past couple of months, it appears
that people might not realize that the preference name in the
"prefs_register_XXX_preferences" call is *not* the full name of the
preference - the protocol name, and a dot, are automatically prepended,
so if you register a preference named "foobar" for a protocol named
"bletch", the name that shows up in the preferences file, and that would
be used with the "-o" command-line option, is "bletch.foobar", not just
"foobar", so, for example, registering the preference with the name
"bletch.foobar" means its real name is "bletch.bletch.foobar", which
isn't what you want.)

I also made the label in the preferences page say "Checksum type" (to
indicate that it's the type of checksum), and expanded the blurb (which
is written to the preferences file) a bit.