[svn-commits] jrose: branch 12 r425503 - /branches/12/main/sched.c

[SVN commits to the Digium repositories]

Author: jrose
Date: Tue Oct 14 13:49:25 2014
New Revision: 425503

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=425503
Log:
Scheduler: Fix a nasty scheduler caching bug which makes new tasks not execute

Tasks that were marked for pending deletion in the scheduler would be moved to
the cache for later reuse, but after being recycled the deleted mark wouldn't
be removed resulting in fresh tasks being deleted without reason... and
immediately moved back into the cache where they could be reused again. This
could cause horrendous things to happen in just about anything that used a
scheduler.

ASTERISK-24321 #close
Reported by: Steve Pitts
Review: https://reviewboard.asterisk.org/r/4071/


Modified:
    branches/12/main/sched.c

Modified: branches/12/main/sched.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/main/sched.c?view=diff&rev=425503&r1=425502&r2=425503
==============================================================================
--- branches/12/main/sched.c (original)
+++ branches/12/main/sched.c Tue Oct 14 13:49:25 2014
@@ -404,6 +404,7 @@
 		tmp->resched = when;
 		tmp->variable = variable;
 		tmp->when = ast_tv(0, 0);
+		tmp->deleted = 0;
 		if (sched_settime(&tmp->when, when)) {
 			sched_release(con, tmp);
 		} else {