Wireshark-commits: [Wireshark-commits] master 3128589: Fix Ceph packet length determination.

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 19 Aug 2014 01:17:11 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=312858939d6e37385b117ac1d20d4e3af07e1619
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

3128589 by Kevin Cox (kevincox@xxxxxxxxxxx):

    Fix Ceph packet length determination.
    
    The authentication string was not being factored in on the length reply.
    Also there was an issue with different banners.  Now the banner length
    must match what we expect or the packet is rejected.  If the banner
    length changes the protocol is different and we won't be able to parse
    it anyways.
    
    Change-Id: I0c1a7379edaa203042486a0e6f9ce3642427da99
    Reviewed-on: https://code.wireshark.org/review/3710
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  cf0cb90   ceph: use col_add_str instead of col_set_str
    adds  3128589   Fix Ceph packet length determination.


Summary of changes:
 epan/dissectors/packet-ceph.c |   30 ++++++++++++++++++------------
 1 file changed, 18 insertions(+), 12 deletions(-)