URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=98586d964c53a8fb7b8562341e3a7fbd4c068b4d
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-1.12
Repository: wireshark
Commits:
98586d9 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>
(cherry picked from commit d4f7f21b51532c1a45a43764fa92cd55e17931de)
Reviewed-on: https://code.wireshark.org/review/13018
Actions performed:
from 50fa34b Welcome in 2016, part 2
adds 98586d9 Clean up error checking and handling.
Summary of changes:
epan/dissectors/packet-eigrp.c | 126 ++++++++++++++++++----------------------
1 file changed, 55 insertions(+), 71 deletions(-)