Skip to content

Commit

Permalink
Release 1.17.3
Browse files Browse the repository at this point in the history
[BUGFIX] Do not send STOP_WAITING frames when using Q044
  • Loading branch information
Dmitri Tikhonov committed Nov 16, 2018
1 parent feecb11 commit bf2c703
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 2 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
2018-11-16
- 1.17.3
- [BUGFIX] Do not send STOP_WAITING frames when using Q044

2018-10-19
- 1.17.2
- [BUGFIX] Memory leak in test_frame_rw unit test.
Expand Down
2 changes: 1 addition & 1 deletion include/lsquic.h
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ extern "C" {

#define LSQUIC_MAJOR_VERSION 1
#define LSQUIC_MINOR_VERSION 17
#define LSQUIC_PATCH_VERSION 2
#define LSQUIC_PATCH_VERSION 3

/**
* Engine flags:
Expand Down
2 changes: 2 additions & 0 deletions src/liblsquic/lsquic_engine.c
Original file line number Diff line number Diff line change
Expand Up @@ -1444,3 +1444,5 @@ lsquic_engine_count_attq (lsquic_engine_t *engine, int from_now)
now += from_now;
return attq_count_before(engine->attq, now);
}


3 changes: 2 additions & 1 deletion src/liblsquic/lsquic_full_conn.c
Original file line number Diff line number Diff line change
Expand Up @@ -2001,7 +2001,8 @@ process_incoming_packet (struct full_conn *conn, lsquic_packet_in_t *packet_in)
if (conn->fc_conn.cn_version >= LSQVER_039)
{
assert(!(conn->fc_flags & FC_NSTP)); /* This bit off at start */
if (conn->fc_settings->es_support_nstp)
if (conn->fc_conn.cn_version >= LSQVER_044
|| conn->fc_settings->es_support_nstp)
{
conn->fc_flags |= FC_NSTP;
lsquic_send_ctl_turn_nstp_on(&conn->fc_send_ctl);
Expand Down
3 changes: 3 additions & 0 deletions src/liblsquic/lsquic_handshake.c
Original file line number Diff line number Diff line change
Expand Up @@ -965,6 +965,9 @@ lsquic_enc_session_gen_chlo (lsquic_enc_session_t *enc_session,
return -1;

n_opts = 0;
/* CHLO is not regenerated during version negotiation. Hence we always
* include this option to cover the case when Q044 gets negotiated down.
*/
if (settings->es_support_nstp)
opts[ n_opts++ ] = QTAG_NSTP;

Expand Down

0 comments on commit bf2c703

Please sign in to comment.