]> git.ipfire.org Git - thirdparty/asterisk.git/commit
Fix improper usage of scheduler exposed by 5c713fdf18f 82/1382/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:04 +0000 (07:40 -0500)
commitd61da57428e767036155377d606a322e0cef2d9d
tree395d2df301a74776864ccbaa3a02b86dbcf3843c
parent5d12653d2ad420bcad7a06321253a9a30d78363a
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