Wireshark-commits: [Wireshark-commits] master 2ed4743: RTPS: APP_ACK_CONF submessage dissection fix

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Tue, 13 Nov 2018 19:06:56 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=2ed4743915a7768959aff59631357a7b13a16dbb
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

2ed4743 by ismaelrti (ismael@xxxxxxx):

    RTPS: APP_ACK_CONF submessage dissection fixed.
    
    Count field of APP_ACK_CONF submessage was dissected using a signed
    integer rather than unsigned. That avoids the dissection to be concluded
    due to a wrong type error.
    
    Change-Id: Ie5f85ce5b3d745d74e1b50d96a77560fb854034b
    Reviewed-on: https://code.wireshark.org/review/30605
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  d2227b7   extcap: Allow ASCII '0' in preference name
     add  2ed4743   RTPS: APP_ACK_CONF submessage dissection fixed.


Summary of changes:
 epan/dissectors/packet-rtps.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)