Skip to content

Commit

Permalink
Release 1.17.7
Browse files Browse the repository at this point in the history
[BUGFIX] Do not unset PING alarm before ringing expired alarms.  This
prevented PING from ever being sent.
  • Loading branch information
Dmitri Tikhonov committed Dec 3, 2018
1 parent 8252b0b commit 18237fa
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 7 deletions.
5 changes: 5 additions & 0 deletions CHANGELOG
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
2018-12-03
- 1.17.7
- [BUGFIX] Do not unset PING alarm before ringing expired alarms.
This prevented PING from ever being sent.

2018-11-29
- 1.17.6
- Add failsafe: resume sending packets after some time
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 6
#define LSQUIC_PATCH_VERSION 7

/**
* Engine flags:
Expand Down
6 changes: 0 additions & 6 deletions src/liblsquic/lsquic_full_conn.c
Original file line number Diff line number Diff line change
Expand Up @@ -2818,12 +2818,6 @@ full_conn_ci_tick (lsquic_conn_t *lconn, lsquic_time_t now)
lsquic_send_ctl_set_buffer_stream_packets(&conn->fc_send_ctl, 1);
CLOSE_IF_NECESSARY();

if (!(conn->fc_flags & FC_SERVER))
{
lsquic_alarmset_unset(&conn->fc_alset, AL_PING);
lsquic_send_ctl_sanity_check(&conn->fc_send_ctl);
}

lsquic_alarmset_ring_expired(&conn->fc_alset, now);
CLOSE_IF_NECESSARY();

Expand Down

0 comments on commit 18237fa

Please sign in to comment.