Wireshark-commits: [Wireshark-commits] master 3a0b4d6: vnc: fix issue found by PVS Studio (V1003)

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: Thu, 17 May 2018 05:28:15 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=3a0b4d643182c2bff3a2e3c480ede538ff692dc3
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

3a0b4d6 by Alexis La Goutte (alexis.lagoutte@xxxxxxxxx):

    vnc: fix issue found by PVS  Studio (V1003)
    
    Consider inspecting the 'for' operator. No more than one iteration of the loop will be performed
    
    Change-Id: I0fe1cb83c8dca935c729a94eb7f6b94d2c0d245a
    Reviewed-on: https://code.wireshark.org/review/27508
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  88f7dae   pkcs1: recognize the ecdsa-with-SHA1 OID
    adds  3a0b4d6   vnc: fix issue found by PVS  Studio (V1003)


Summary of changes:
 epan/dissectors/packet-vnc.c | 87 +++++++++++++++++++++-----------------------
 1 file changed, 42 insertions(+), 45 deletions(-)