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

Add int return value to on_frame_not_send_callback

This commit is contained in:
Tatsuhiro Tsujikawa 2013-08-29 21:51:58 +09:00
parent d4852b0f11
commit b9d2f9b6b0
5 changed files with 23 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/includes/nghttp2/nghttp2.h
View File

@ -879,8 +879,13 @@ typedef int (*nghttp2_on_frame_send_callback)
* sent because of the error. The error is indicated by the
* |lib_error_code|, which is one of the values defined in
* :type:`nghttp2_error`.
*
* The implementation of this function must return 0 if it
* succeeds. If nonzero is returned, it is treated as fatal error and
* `nghttp2_session_recv()` and `nghttp2_session_send()` functions
* immediately return :enum:`NGHTTP2_ERR_CALLBACK_FAILURE`.
*/
typedef void (*nghttp2_on_frame_not_send_callback)
typedef int (*nghttp2_on_frame_not_send_callback)
(nghttp2_session *session, nghttp2_frame *frame, int lib_error_code,
void *user_data);

6
lib/nghttp2_session.c
View File

@ -1474,8 +1474,10 @@ int nghttp2_session_send(nghttp2_session *session)
it. */
nghttp2_frame *frame = nghttp2_outbound_item_get_ctrl_frame(item);
if(frame->hd.type != NGHTTP2_WINDOW_UPDATE) {
session->callbacks.on_frame_not_send_callback
(session, frame, framebuflen, session->user_data);
if(session->callbacks.on_frame_not_send_callback
(session, frame, framebuflen, session->user_data) != 0) {
return NGHTTP2_ERR_CALLBACK_FAILURE;
}
}
}
nghttp2_outbound_item_free(item);

3
src/shrpx_http2_upstream.cc
View File

@ -331,7 +331,7 @@ int on_data_chunk_recv_callback(nghttp2_session *session,
} // namespace
namespace {
void on_frame_not_send_callback(nghttp2_session *session,
int on_frame_not_send_callback(nghttp2_session *session,
nghttp2_frame *frame,
int lib_error_code, void *user_data)
{
@ -348,6 +348,7 @@ void on_frame_not_send_callback(nghttp2_session *session,
upstream->rst_stream(downstream, NGHTTP2_INTERNAL_ERROR);
}
}
return 0;
}
} // namespace

7
src/shrpx_spdy_session.cc
View File

@ -970,7 +970,7 @@ int before_frame_send_callback(nghttp2_session *session,
} // namespace
namespace {
void on_frame_not_send_callback(nghttp2_session *session,
int on_frame_not_send_callback(nghttp2_session *session,
nghttp2_frame *frame,
int lib_error_code, void *user_data)
{
@ -986,19 +986,20 @@ void on_frame_not_send_callback(nghttp2_session *session,
auto sd = reinterpret_cast<StreamData*>
(nghttp2_session_get_stream_user_data(session, frame->hd.stream_id));
if(!sd) {
return;
return 0;
}
if(sd->dconn) {
auto downstream = sd->dconn->get_downstream();
if(!downstream ||
downstream->get_downstream_stream_id() != frame->hd.stream_id) {
return;
return 0;
}
downstream->set_response_state(Downstream::MSG_RESET);
call_downstream_readcb(spdy, downstream);
}
spdy->remove_stream_data(sd);
}
return 0;
}
} // namespace

3
tests/nghttp2_session_test.c
View File

@ -155,7 +155,7 @@ static int on_frame_send_callback(nghttp2_session *session,
return 0;
}
static void on_frame_not_send_callback(nghttp2_session *session,
static int on_frame_not_send_callback(nghttp2_session *session,
nghttp2_frame *frame,
int lib_error,
void *user_data)
@ -164,6 +164,7 @@ static void on_frame_not_send_callback(nghttp2_session *session,
++ud->frame_not_send_cb_called;
ud->not_sent_frame_type = frame->hd.type;
ud->not_sent_error = lib_error;
return 0;
}
static int on_data_chunk_recv_callback(nghttp2_session *session,