Wireshark-commits: [Wireshark-commits] master 99db1c5: Remove superfluous pointer checks (CID-28059
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 9 Mar 2016 13:20:23 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=99db1c5c982360c5f8307f41eccc6417f83e6a84
Submitter: Michael Mann (mmann78@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

99db1c5 by Jaap Keuter (jaap.keuter@xxxxxxxxx):

    Remove superfluous pointer checks (CID-280599, 280600 and 280601)
    
    The preceding code takes every effort to make sure that the
    frac and end pointers are relevant pointers. Don't go back and
    check for NULL yet again.
    
    Change-Id: If805aee5a77a8c23eac7b074bd11d3743c5e1a3b
    Reviewed-on: https://code.wireshark.org/review/14398
    Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    Reviewed-by: Michael Mann <mmann78@xxxxxxxxxxxx>
    

Actions performed:

    from  301fa00   Use field info length instead of packet length for protocol hierarchy stats.
    adds  99db1c5   Remove superfluous pointer checks (CID-280599, 280600 and 280601)


Summary of changes:
 editcap.c |   33 +++++++++++++++------------------
 1 file changed, 15 insertions(+), 18 deletions(-)