Wireshark · Wireshark-commits: [Wireshark-commits] master f4ad204: Remove now-redundant check.

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f4ad204745e6ccbb0ae420561b8e159dc4a3a824
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

f4ad204 by Guy Harris (guy@xxxxxxxxxxxx):

    Remove now-redundant check.
    
    We now check much earlier for an invalid message length; remove the
    check done afterwards.
    
    Also, note that dissect_netlink_error() should also check the message
    length, to make sure it doesn't run past the end of the message, and
    indicate why we are assuming an "integer" is 4 bytes (it's because the
    RFC is vague here).
    
    Change-Id: Ie0b5074acc852cdeaa008fee1125130a6c8771a1
    Reviewed-on: https://code.wireshark.org/review/17279
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  3479406   Get rid of trailing tabs.
    adds  f4ad204   Remove now-redundant check.


Summary of changes:
 epan/dissectors/packet-netlink.c |   13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)