]> git.ipfire.org Git - thirdparty/asterisk.git/commit
res_rtp_asterisk: Fix dtls timer issues causing FRACKs and SEGVs
authorGeorge Joseph <gjoseph@sangoma.com>
Mon, 16 Sep 2024 21:17:28 +0000 (15:17 -0600)
committerGeorge Joseph <gjoseph@sangoma.com>
Tue, 1 Oct 2024 14:22:40 +0000 (14:22 +0000)
commitec2e26cfe77b93ac93203ec5ee7d4a237e72de2c
tree1d861bbb287a9b28918b09d4499f5e9585b1ab30
parent5a237235c00a3a408ddb65874f39dab76f3824cf
res_rtp_asterisk: Fix dtls timer issues causing FRACKs and SEGVs

In dtls_srtp_handle_timeout(), when DTLSv1_get_timeout() returned
success but with a timeout of 0, we were stopping the timer and
decrementing the refcount on instance but not resetting the
timeout_timer to -1.  When dtls_srtp_stop_timeout_timer()
was later called, it was atempting to stop a stale timer and could
decrement the refcount on instance again which would then cause
the instance destructor to run early.  This would result in either
a FRACK or a SEGV when ast_rtp_stop(0 was called.

According to the OpenSSL docs, we shouldn't have been stopping the
timer when DTLSv1_get_timeout() returned success and the new timeout
was 0 anyway.  We should have been calling DTLSv1_handle_timeout()
again immediately so we now reschedule the timer callback for
1ms (almost immediately).

Additionally, instead of scheduling the timer callback at a fixed
interval returned by the initial call to DTLSv1_get_timeout()
(usually 999 ms), we now reschedule the next callback based on
the last call to DTLSv1_get_timeout().

Resolves: #487
res/res_rtp_asterisk.c