Wireshark-commits: [Wireshark-commits] master ffabb74: Back up to the beginning of the file if we d

Date Prev · Date Next · Thread Prev · Thread Next
Date Index · Thread Index · Other Months · All Mailing Lists
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Thu, 05 Dec 2019 06:49:55 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=ffabb74e2c75fd9cf314c28a89552f5b40cb18d1
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

ffabb74 by Guy Harris (guy@xxxxxxxxxxxx):

    Back up to the beginning of the file if we don't have a gzip header.
    
    If the first byte of the file is 31, and we advance to the next byte but
    find it's not 139, back up to the first byte before falling through and
    treating the file as uncompressed.
    
    Add/expand some comments while we're at it.
    
    Bug: 16252
    Change-Id: I292b51f9cc04173482a43b26b0ce73c9e7aee570
    Reviewed-on: https://code.wireshark.org/review/35315
    Petri-Dish: Guy Harris <guy@xxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    

Actions performed:

    from  4b673a1   tipc: check message bundle size before using it.
     add  ffabb74   Back up to the beginning of the file if we don't have a gzip header.


Summary of changes:
 wiretap/file_wrappers.c | 201 ++++++++++++++++++++++++++++++------------------
 1 file changed, 126 insertions(+), 75 deletions(-)