Wireshark-dev: Re: [Wireshark-dev] [Wireshark-commits] rev 18766: /trunk/epan/dissectors/ /trun

From: Joerg Mayer <jmayer@xxxxxxxxx>
Date: Thu, 20 Jul 2006 14:18:32 +0200
On Wed, Jul 19, 2006 at 06:51:26PM +0000, etxrab@xxxxxxxxxxxxx wrote:
> http://anonsvn.wireshark.org/viewvc/viewvc.cgi?view=rev&revision=18766
> 
> User: etxrab
> Date: 2006/07/19 06:51 PM
> 
> Log:
>  From Martin Mathieson:
>  
>  This patch:
>  - adds headers found in later versions of the msrp drafts
>  - fixes a problem where wrong length values were used while parsing the 
>  request/status line and it was going beyond linelen
>  - "Transaktion" -> "Transaction"
>  - status code now appears as a numerical field
>  - removes unused parameters from check_msrp_header()
>  - tidies up some indentation
>  It has survived some fuzz-testing.

I'm still not sure that this is correct: which conditions may lead to
the function to return -1?

 ciao
      Joerg
-- 
Joerg Mayer                                           <jmayer@xxxxxxxxx>
We are stuck with technology when what we really want is just stuff that
works. Some say that should read Microsoft instead of technology.