Wireshark-commits: [Wireshark-commits] master cc795d4: WSP: check tvb_get_guintvar() success
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: Fri, 09 Mar 2018 14:11:36 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cc795d4ddfaca382bcd0e6b4862a432eddd06f54
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

cc795d4 by Pascal Quantin (pascal.quantin@xxxxxxxxx):

    WSP: check tvb_get_guintvar() success
    
    4630b4fcf8 sets octetCount to 0 in case of error. Let's check this
    return value to avoid an infinite loop
    
    Bug: 14519
    Change-Id: Ie3519067d609afb1bbf9e67f7f15f95911d2b173
    Reviewed-on: https://code.wireshark.org/review/26388
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Petri-Dish: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  1ac90d5   Register and initialize the ZCL clusters in a uniform way.
    adds  cc795d4   WSP: check tvb_get_guintvar() success


Summary of changes:
 epan/dissectors/packet-wsp.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)