]> git.ipfire.org Git - thirdparty/asterisk.git/commit
Fix improper usage of scheduler exposed by 5c713fdf18f 79/1379/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:39:57 +0000 (07:39 -0500)
commit50fa9ff9972e67899dfc4e7e6766c5977d4aae7a
tree66b861f45acb11e60611316041ebbd67218f7979
parentb39e92f036188664aefa494a798253f4f78d11f4
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