walkero
/
nghttp2
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

Don't show PAD_HIGH and PAD_LOW flags to user callback

This commit is contained in:
Tatsuhiro Tsujikawa 2014-03-14 21:53:03 +09:00
parent 2ec4b10805
commit 5b81f7c713
1 changed files with 10 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
lib/nghttp2_session.c
View File

@ -3962,10 +3962,12 @@ ssize_t nghttp2_session_mem_recv(nghttp2_session *session,
if(iframe->payloadleft) { if(iframe->payloadleft) {
break; break;
} }
/* Clear PAD_HIGH and PAD_LOW, because we rely on those flags /* Clear PAD_HIGH and PAD_LOW, because we rely on those flags in
in the next CONTINUATION frame. */ the next CONTINUATION frame. Also we don't show these flags
to user callback */
iframe->frame.hd.flags &= iframe->frame.hd.flags &=
~(NGHTTP2_FLAG_PAD_HIGH | NGHTTP2_FLAG_PAD_LOW); ~(NGHTTP2_FLAG_PAD_HIGH | NGHTTP2_FLAG_PAD_LOW);
if((iframe->frame.hd.flags & NGHTTP2_FLAG_END_HEADERS) == 0) { if((iframe->frame.hd.flags & NGHTTP2_FLAG_END_HEADERS) == 0) {
inbound_frame_reset_left(iframe, NGHTTP2_FRAME_HDLEN); inbound_frame_reset_left(iframe, NGHTTP2_FRAME_HDLEN);
iframe->padlen = 0; iframe->padlen = 0;
@ -4258,6 +4260,12 @@ ssize_t nghttp2_session_mem_recv(nghttp2_session *session,
if(iframe->payloadleft) { if(iframe->payloadleft) {
break; break;
} }
/* Clear PAD_HIGH and PAD_LOW, because we don't show these flags
to user callback */
session->iframe.frame.hd.flags &=
~(NGHTTP2_FLAG_PAD_HIGH | NGHTTP2_FLAG_PAD_LOW);
rv = nghttp2_session_process_data_frame(session); rv = nghttp2_session_process_data_frame(session);
if(nghttp2_is_fatal(rv)) { if(nghttp2_is_fatal(rv)) {
return rv; return rv;