tor

commit adaf3e9b89f62d68ab631b8f672d9bff996689b9
parent df312b3cf637d2c5487adc60ed541823bef56f06
Author: David Goulet <dgoulet@torproject.org>
Date:   Wed, 31 Jan 2018 13:46:31 -0500

sched: Avoid adding the same channel twice to the KIST pending list

This is the quick fix that is keeping the channel in PENDING state so if we
ever try to reschedule the same channel, it won't happened.

Fixes #24700

Signed-off-by: David Goulet <dgoulet@torproject.org>

Diffstat:
A
changes/bug24700
 | 
4
++++
M
src/or/scheduler_kist.c
 | 
1
-

2 files changed, 4 insertions(+), 1 deletion(-)
diff --git a/changes/bug24700 b/changes/bug24700
@@ -0,0 +1,4 @@
+  o Minor bugfixes (scheduler, KIST):
+    - Avoid adding the same channel twice in the KIST scheduler pending list
+      wasting CPU cycles at handling the same channel twice. Fixes bug 24700;
+      bugfix on 0.3.2.1-alpha.
diff --git a/src/or/scheduler_kist.c b/src/or/scheduler_kist.c
@@ -693,7 +693,6 @@ kist_scheduler_run(void)
        * after the scheduling loop is over. They can hopefully be taken care of
        * in the next scheduling round.
        */
-      chan->scheduler_state = SCHED_CHAN_WAITING_TO_WRITE;
       if (!to_readd) {
         to_readd = smartlist_new();
       }