ActiveJobs loop in cupsdCheckJobs() skips active pending jobs

[michaelrsweet]

Version: 2.0-current
CUPS.org User: twaugh.redhat

When looping over the active jobs, if a pending job is started it will
have looped through the ActiveJobs array, meaning further pending
active jobs will be skipped for another 10 seconds (the senddoc
delay).

Fix attached.

[michaelrsweet]

CUPS.org User: mike

Fixed in Subversion repository.

[michaelrsweet]

"0001-Fix-the-ActiveJobs-loop-in-cupsdCheckJobs-STR-4646.patch":

From 4999ea81613f41e34715953b6d077e9b7522a238 Mon Sep 17 00:00:00 2001
From: Tim Waugh twaugh@redhat.com
Date: Tue, 23 Jun 2015 16:26:06 +0100
Subject: [PATCH] Fix the ActiveJobs loop in cupsdCheckJobs() (STR #4646)

When looping over the active jobs, if a pending job is started it will
have looped through the ActiveJobs array, meaning further pending
active jobs will be skipped for another 10 seconds (the senddoc

delay).

scheduler/job.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/scheduler/job.c b/scheduler/job.c
index 956f9ff..82c5306 100644
--- a/scheduler/job.c
+++ b/scheduler/job.c
@@ -392,7 +392,9 @@ cupsdCheckJobs(void)
* Start the job...
*/

• cupsArraySave(ActiveJobs);
  

  start_job(job, printer);

• cupsArrayRestore(ActiveJobs);
  

  }
  }
  }

  2.4.3