Wireshark-commits: [Wireshark-commits] master deb687b: Return the same error for too-large packets
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: Sat, 30 Apr 2016 23:39:11 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=deb687b3763530faea3c34b244c58f1cd1660ec2
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

deb687b by Guy Harris (guy@xxxxxxxxxxxx):

    Return the same error for too-large packets as we do for other files.
    
    And note that our limit (which is what we use as the fixed buffer size)
    is less than WTAP_MAX_PACKET_SIZE, so we don't have to check against
    WTAP_MAX_PACKET_SIZE.
    
    Change-Id: I28cd95c40fd2fba9994a5d64ef323f1d8c1c4478
    Reviewed-on: https://code.wireshark.org/review/15204
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  b39f0f3   Note that we don't have to worry about too-large packets.
    adds  deb687b   Return the same error for too-large packets as we do for other files.


Summary of changes:
 wiretap/pppdump.c |   11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)