Wireshark-commits: [Wireshark-commits] master-3.0 cd6f3bb: Back up to the beginning of the file if
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: Thu, 05 Dec 2019 06:50:39 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=cd6f3bb6037f1d28a6ac0f05b73350f3bc71a881
Submitter: "Guy Harris <guy@xxxxxxxxxxxx>"
Changed: branch: master-3.0
Repository: wireshark

Commits:

cd6f3bb 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>
    (cherry picked from commit ffabb74e2c75fd9cf314c28a89552f5b40cb18d1)
    Reviewed-on: https://code.wireshark.org/review/35317
    

Actions performed:

    from  e0db2fb   3.0.7 → 3.0.8.
     add  cd6f3bb   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(-)