Wireshark-commits: [Wireshark-commits] master 3a9f171: HTTP: start dissecting upgraded connections
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: Mon, 13 Aug 2018 07:59:39 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=3a9f1717634a331507e9ccae74108fd9e3c1222a
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master
Repository: wireshark

Commits:

3a9f171 by Peter Wu (peter@xxxxxxxxxxxxx):

    HTTP: start dissecting upgraded connections right after upgrade
    
    After a HTTP upgrade, some data may already follow the headers. Be sure
    to dissect this. Tested with a SSTP capture (bug 8239), HTTP proxy
    capture (bug 15043), no regressions were found. WebSocket traffic from
    the attached bug is now properly dissected.
    
    Bug: 13889
    Change-Id: Icc32871b4ebb2520769cb17505517d9d11543684
    Reviewed-on: https://code.wireshark.org/review/29111
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  1d4bb22   HTTP: really choose the correct server port for tunnels
     add  3a9f171   HTTP: start dissecting upgraded connections right after upgrade


Summary of changes:
 epan/dissectors/packet-http.c | 62 +++++++++++++++++++++++++++++--------------
 epan/dissectors/packet-http.h |  9 ++++---
 2 files changed, 48 insertions(+), 23 deletions(-)