Wireshark-commits: [Wireshark-commits] master d4f7f21: Clean up error checking and handling.
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: Sun, 3 Jan 2016 03:26:44 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=d4f7f21b51532c1a45a43764fa92cd55e17931de
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

d4f7f21 by Guy Harris (guy@xxxxxxxxxxxx):

    Clean up error checking and handling.
    
    The TLV length includes the T and the V, so it must be at least 4; fail
    if it's not.
    
    If the IPv4 or IPv6 prefix length isn't valid, don't bother trying to
    parse the rest of the addresses, just give up.
    
    Don't bother returning offset values that aren't used.
    
    Rename some routines to indicate that they process more than one item.
    
    Add some comments while we're at it.
    
    Change-Id: I6825216f0e7218e230d8d60d958b3c2453a3bb62
    Reviewed-on: https://code.wireshark.org/review/13016
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  6326894   IPv6: Minor rename, cleanup (Shim6)
    adds  d4f7f21   Clean up error checking and handling.


Summary of changes:
 epan/dissectors/packet-eigrp.c |  126 ++++++++++++++++++----------------------
 1 file changed, 55 insertions(+), 71 deletions(-)