- Fixed (for the third time) patch for STR #3425 to correctly remove job

info files in /var/spool/cups (bug #571830).
This commit is contained in:
Jiří Popelka 2010-03-10 11:08:02 +00:00
parent 0a206ac418
commit 6484bf8fde
2 changed files with 42 additions and 11 deletions

View File

@ -1,6 +1,6 @@
diff -up cups-1.4.2/scheduler/job.c.str3425 cups-1.4.2/scheduler/job.c diff -up cups-1.4.2/scheduler/job.c.str3425 cups-1.4.2/scheduler/job.c
--- cups-1.4.2/scheduler/job.c.str3425 2009-12-23 11:00:06.070056898 +0000 --- cups-1.4.2/scheduler/job.c.str3425 2010-03-09 16:59:12.000000000 +0100
+++ cups-1.4.2/scheduler/job.c 2009-12-23 11:02:22.889056232 +0000 +++ cups-1.4.2/scheduler/job.c 2010-03-09 17:02:57.000000000 +0100
@@ -253,7 +253,7 @@ cupsdCancelJobs(const char *dest, /* I - @@ -253,7 +253,7 @@ cupsdCancelJobs(const char *dest, /* I -
if (purge) if (purge)
cupsdSetJobState(job, IPP_JOB_CANCELED, CUPSD_JOB_PURGE, cupsdSetJobState(job, IPP_JOB_CANCELED, CUPSD_JOB_PURGE,
@ -24,27 +24,54 @@ diff -up cups-1.4.2/scheduler/job.c.str3425 cups-1.4.2/scheduler/job.c
job = (cupsd_job_t *)cupsArrayNext(Jobs)) job = (cupsd_job_t *)cupsArrayNext(Jobs))
if (job->state_value >= IPP_JOB_CANCELED && !job->printer) if (job->state_value >= IPP_JOB_CANCELED && !job->printer)
cupsdDeleteJob(job, CUPSD_JOB_PURGE); cupsdDeleteJob(job, CUPSD_JOB_PURGE);
@@ -2498,8 +2498,6 @@ cupsdSetJobState( @@ -2286,12 +2286,14 @@ cupsdSetJobState(
job->dirty = 1; if (!cupsdLoadJob(job))
return;
- /*
- * Don't do anything if the state is unchanged...
- */
-
- if (newstate == (oldstate = job->state_value))
- return;
+ /*
+ * Don't do anything if the state is unchanged and we aren't purging the
+ * job...
+ */
+
+ oldstate = job->state_value;
+ if (newstate == oldstate && action != CUPSD_JOB_PURGE)
+ return;
/*
* Stop any processes that are working on the current job...
@@ -2499,7 +2501,14 @@ cupsdSetJobState(
cupsdMarkDirty(CUPSD_DIRTY_JOBS); cupsdMarkDirty(CUPSD_DIRTY_JOBS);
} }
- else if (!job->printer) else if (!job->printer)
- cupsdDeleteJob(job, CUPSD_JOB_PURGE); + {
+ /*
+ * Delete the job immediately if not actively printing...
+ */
+
cupsdDeleteJob(job, CUPSD_JOB_PURGE);
+ job = NULL;
+ }
break; break;
} }
@@ -2507,7 +2505,7 @@ cupsdSetJobState( @@ -2507,7 +2516,7 @@ cupsdSetJobState(
* Finalize the job immediately if we forced things... * Finalize the job immediately if we forced things...
*/ */
- if (action == CUPSD_JOB_FORCE) - if (action == CUPSD_JOB_FORCE)
+ if (action >= CUPSD_JOB_FORCE && job->printer) + if (action >= CUPSD_JOB_FORCE && job && job->printer)
finalize_job(job); finalize_job(job);
/* /*
diff -up cups-1.4.2/scheduler/main.c.str3425 cups-1.4.2/scheduler/main.c diff -up cups-1.4.2/scheduler/main.c.str3425 cups-1.4.2/scheduler/main.c
--- cups-1.4.2/scheduler/main.c.str3425 2009-12-23 11:00:05.687056970 +0000 --- cups-1.4.2/scheduler/main.c.str3425 2010-03-09 16:59:12.000000000 +0100
+++ cups-1.4.2/scheduler/main.c 2009-12-23 11:00:06.156057773 +0000 +++ cups-1.4.2/scheduler/main.c 2010-03-09 16:59:12.000000000 +0100
@@ -389,7 +389,7 @@ main(int argc, /* I - Number of comm @@ -389,7 +389,7 @@ main(int argc, /* I - Number of comm
* parent's file descriptors to be blocking. This is a workaround for a * parent's file descriptors to be blocking. This is a workaround for a
* limitation of userland libpthread on OpenBSD. * limitation of userland libpthread on OpenBSD.

View File

@ -8,7 +8,7 @@
Summary: Common Unix Printing System Summary: Common Unix Printing System
Name: cups Name: cups
Version: 1.4.2 Version: 1.4.2
Release: 34%{?dist} Release: 35%{?dist}
License: GPLv2 License: GPLv2
Group: System Environment/Daemons Group: System Environment/Daemons
Source: http://ftp.easysw.com/pub/cups/%{version}/cups-%{version}-source.tar.bz2 Source: http://ftp.easysw.com/pub/cups/%{version}/cups-%{version}-source.tar.bz2
@ -624,6 +624,10 @@ rm -rf $RPM_BUILD_ROOT
%{php_extdir}/phpcups.so %{php_extdir}/phpcups.so
%changelog %changelog
* Wed Mar 10 2010 Jiri Popelka <jpopelka@redhat.com> 1:1.4.2-35
- Fixed (for the third time) patch for STR #3425 to correctly
remove job info files in /var/spool/cups (bug #571830).
* Fri Mar 5 2010 Tim Waugh <twaugh@redhat.com> - 1:1.4.2-34 * Fri Mar 5 2010 Tim Waugh <twaugh@redhat.com> - 1:1.4.2-34
- Applied patch for CVE-2010-0302 (incomplete fix for CVE-2009-3553, - Applied patch for CVE-2010-0302 (incomplete fix for CVE-2009-3553,
bug #557775). bug #557775).