diff options
author | Peter Wu <peter@lekensteyn.nl> | 2014-11-28 15:52:07 +0100 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2014-12-16 15:23:41 +0000 |
commit | 6ee78f3be167807b9e2790b8f88f14ea1a4c9462 (patch) | |
tree | 42e1afe9b45ff2165e0457a737c0d5deb8f945ea /wiretap/5views.c | |
parent | db93414049b3094734e33a0da49f611c2a211410 (diff) |
http: properly calculate end of chunked response
It was previously assumed that the remainder of a packet contains a
chunked-body response. This does not have be the case, and if the
assumption is violated, then the dissector would add multiple parts to a
single "De-chunked entity body".
This patch properly calculates the end of a chunked-body response,
taking the optional trailer-part into account and adjusting the size of
the chunked-body data as needed.
The CRLF in last-chunk that was previously dissected as "Chunk Boundary"
is the last CRLF that closes chunked-body, it is not part of last-chunk
(as it has no chunk-data to terminate).
A new header field is added for this trailer-part (RFC 7230 sec. 4.1).
Bug: 10707
Change-Id: Ifef1cc7dd0443edca4198eb1c27f58719f85fa9f
Reviewed-on: https://code.wireshark.org/review/5526
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Reviewed-by: Evan Huus <eapache@gmail.com>
Petri-Dish: Evan Huus <eapache@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'wiretap/5views.c')
0 files changed, 0 insertions, 0 deletions