Wireshark-commits: [Wireshark-commits] master dd424c6: Handle the case where the previous PDU has l
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: Fri, 4 Jul 2014 14:00:52 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=dd424c622581310b4d0d79bc57137f03d2573752
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

dd424c6 by AndersBroman (anders.broman@xxxxxxxxxxxx):

    Handle the case where the previous PDU has left an extra CRLF in the stream.RFC 2626   In the interest of robustness, servers SHOULD ignore any empty   line(s) received where a Request-Line is expected. In other words, if   the server is reading the protocol stream at the beginning of a   message and receives a CRLF first, it should ignore the CRLF.
    
    Change-Id: I97ba94f451463c8facd2c20bf6b7364f095119e7
    Reviewed-on: https://code.wireshark.org/review/2808
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    

Actions performed:

    from  34d09f7   Add missing /
    adds  dd424c6   Handle the case where the previous PDU has left an extra CRLF in the stream.RFC 2626   In the interest of robustness, servers SHOULD ignore any empty   line(s) received where a Request-Line is expected. In other words, if   the server is reading the protocol stream at the beginning of a   message and receives a CRLF first, it should ignore the CRLF.


Summary of changes:
 epan/dissectors/packet-http.c |   30 ++++++++++++++++++++++++++++--
 1 file changed, 28 insertions(+), 2 deletions(-)