Wireshark-dev: Re: [Wireshark-dev] [PATCH] packet-mysql.c: Trivial change

From: Jaap Keuter <jaap.keuter@xxxxxxxxx>
Date: Thu, 30 Nov 2006 08:15:50 +0100 (CET)
Hi,

Checked in.

Thanx,
Jaap

On Wed, 29 Nov 2006, Martin Warnes wrote:

> Attached updated patch to use "dissect_mysql_pdu" instead of
> "dissect_mysql" when registering the protocol dissector.
>
> Regards .. Martin
>
> Martin Warnes wrote the following on 29/11/2006 10:50:
> > Obviously not so trivial as dissect_mysql does reassembly, so it looks
> > like I shoud register dissect_mysql_pdu instead
> >
> > ... or would this only work if the higher level dissector also did
> > reassembly which mine does?
> >
> >
> >
> > Martin Warnes wrote the following on 29/11/2006 09:37:
> >
> >> Add register_dissector() call to allow being called as a subdissector
> >>
> >> ------------------------------------------------------------------------
> >>
> >> Index: packet-mysql.c
> >> ===================================================================
> >> --- packet-mysql.c	(revision 20004)
> >> +++ packet-mysql.c	(working copy)
> >> @@ -901,6 +901,8 @@
> >>  				       "Whether the MySQL dissector should reassemble MySQL buffers spanning multiple TCP segments."
> >>  				       " To use this option, you must also enable \"Allow subdissectors to reassemble TCP streams\" in the TCP protocol settings.",
> >>  				       &mysql_desegment);
> >> +
> >> +	register_dissector("mysql", dissect_mysql, proto_mysql);
> >>  }
> >>
> >>
> >>
>
>