Wireshark-commits: [Wireshark-commits] master b44f3b7: RTPS: Updated code path to handle multiple v
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 3 Mar 2016 05:35:56 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=b44f3b7d18c21f4a9f617d701c013fe41df24f8f
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

b44f3b7 by Juanjo Martin (juanjo@xxxxxxx):

    RTPS: Updated code path to handle multiple vendors
    
    Before, the parameter ids were handled incorrectly. A vendor specific
    parameter definition was used for all the vendors. This is wrong for
    ids starting at 0x8000. This commits aims to fix that problem and
    make easier the addition of new parameters or vendors.
    
    Change-Id: I0d40aa8cbfa44d5bb2928075001fe39e6f14abc2
    Reviewed-on: https://code.wireshark.org/review/14007
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    Petri-Dish: Michael Mann <mmann78@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  5cc8d43   L2TP: Typo, Decimal display of session ID
    adds  b44f3b7   RTPS: Updated code path to handle multiple vendors


Summary of changes:
 epan/dissectors/packet-rtps.c |  128 +++++++++++++++++++++++++++++------------
 1 file changed, 91 insertions(+), 37 deletions(-)