Wireshark-commits: [Wireshark-commits] master 1318d4c: http: ignore response body for HEAD requests

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Wed, 1 Oct 2014 19:14:55 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=1318d4cfe77fe58e766634eff8407091fe63fdb2
Submitter: Alexis La Goutte (alexis.lagoutte@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

1318d4c by Peter Wu (peter@xxxxxxxxxxxxx):

    http: ignore response body for HEAD requests
    
    Responses to HEAD requests must not have a message body, so do not
    attempt to use the Content-Length and similar headers that indicate a
    non-empty message body in the response. Those point to the entity that
    would be returned for non-HEAD requests.
    
    Requests do not have request_method set, so do not limit desegmentation
    by HEAD in that case. Verified by comparing the output of `tshark -O
    http` and by looking in the Wireshark GTK UI.
    
    Bug: 6064
    Change-Id: I7f027a06d8480673ca9754037d080c3b25cc5816
    Reviewed-on: https://code.wireshark.org/review/4357
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    Reviewed-by: Evan Huus <eapache@xxxxxxxxx>
    Reviewed-by: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
    

Actions performed:

    from  f351a58   Update the release notes.
    adds  1318d4c   http: ignore response body for HEAD requests


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