]> 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:42 +0000 (14:22 +0000)
commit8f82b8cfc1ad9e701c47b9905772b70b963016ae
tree5e913281ebf14013f1a593feaf9c9650e152c84d
parent98380812c0d8fdd74e604bde5fb175e7f4c67cef
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