]> git.ipfire.org Git - thirdparty/asterisk.git/commit
Fix improper usage of scheduler exposed by 5c713fdf18f 80/1380/2
authorMatt Jordan <mjordan@digium.com>
Tue, 6 Oct 2015 02:34:41 +0000 (21:34 -0500)
committerMatt Jordan <mjordan@digium.com>
Tue, 6 Oct 2015 12:40:29 +0000 (07:40 -0500)
commit8cb614fe203f0cf0d40652e1c32549a64dc8cbf8
tree2eaf6fdf6105e7854cf01dccd18f1814043f6166
parent89dec7675d0df2db273a5f3450b0d78732ed0b63
Fix improper usage of scheduler exposed by 5c713fdf18f

When 5c713fdf18f was merged, it allowed for scheduled items to have an ID of
'0' returned. While this was valid per the documentation for the API, it was
apparently never returned previously. As a result, several users of the
scheduler API viewed the result as being invalid, causing them to reschedule
already scheduled items or otherwise fail in interesting ways.

This patch corrects the users such that they view '0' as valid, and a returned
ID of -1 as being invalid.

Note that the failing HEP RTCP tests now pass with this patch. These tests
failed due to a duplicate scheduling of the RTCP transmissions.

ASTERISK-25449 #close

Change-Id: I019a9aa8b6997584f66876331675981ac9e07e39
channels/chan_sip.c
channels/chan_skinny.c
res/res_rtp_asterisk.c