Check explicitly that padding field is really read
This commit is contained in:
parent
d5da7611fa
commit
1093b3eeab
|
@ -5114,6 +5114,10 @@ ssize_t nghttp2_session_mem_recv(nghttp2_session *session,
|
||||||
readlen, iframe->payloadleft,
|
readlen, iframe->payloadleft,
|
||||||
nghttp2_buf_mark_avail(&iframe->sbuf)));
|
nghttp2_buf_mark_avail(&iframe->sbuf)));
|
||||||
|
|
||||||
|
if(nghttp2_buf_mark_avail(&iframe->sbuf)) {
|
||||||
|
return in - first;
|
||||||
|
}
|
||||||
|
|
||||||
/* Pad Length field is subject to flow control */
|
/* Pad Length field is subject to flow control */
|
||||||
rv = session_update_recv_connection_window_size(session, readlen);
|
rv = session_update_recv_connection_window_size(session, readlen);
|
||||||
if(nghttp2_is_fatal(rv)) {
|
if(nghttp2_is_fatal(rv)) {
|
||||||
|
@ -5140,8 +5144,6 @@ ssize_t nghttp2_session_mem_recv(nghttp2_session *session,
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
assert(nghttp2_buf_mark_avail(&iframe->sbuf) == 0);
|
|
||||||
|
|
||||||
busy = 1;
|
busy = 1;
|
||||||
|
|
||||||
padlen = inbound_frame_compute_pad(iframe);
|
padlen = inbound_frame_compute_pad(iframe);
|
||||||
|
|
Loading…
Reference in New Issue