Wireshark-commits: [Wireshark-commits] master a750581: SMB: fix a V595 warning reported by PVS-Stud

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: Mon, 8 Jun 2015 21:07:22 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=a7505812107346b3912df2fba6f8f805def97b9e
Submitter: Pascal Quantin (pascal.quantin@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

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

    SMB: fix a V595 warning reported by PVS-Studio
    
    The 'si' pointer was utilized before it was verified against nullptr.
    
    Change-Id: I92faf43160698a548531dceb557cf4153d15d03f
    Reviewed-on: https://code.wireshark.org/review/8845
    Reviewed-by: Pascal Quantin <pascal.quantin@xxxxxxxxx>
    

Actions performed:

    from  f6f690c   LBMC: fix Truncation of constant value -16. The value range of unsigned char type: [0, 255] found by PVS Studio (V569)
    adds  a750581   SMB: fix a V595 warning reported by PVS-Studio


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