rpms/cups/F-11 cups-str3431.patch,NONE,1.1 cups.spec,1.553,1.554

[Tim Waugh]

Author: twaugh

Update of /cvs/pkgs/rpms/cups/F-11
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv14029

Modified Files:
	cups.spec 
Added Files:
	cups-str3431.patch 
Log Message:
* Tue Dec  8 2009 Tim Waugh <twaugh at redhat.com> - 1:1.4.2-14
- The scheduler did not use the Get-Job-Attributes policy for a
  printer (STR #3431).


cups-str3431.patch:
 ipp.c |   15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

--- NEW FILE cups-str3431.patch ---
diff -up cups-1.4.2/scheduler/ipp.c.str3431 cups-1.4.2/scheduler/ipp.c
--- cups-1.4.2/scheduler/ipp.c.str3431	2009-12-08 12:28:21.095374048 +0000
+++ cups-1.4.2/scheduler/ipp.c	2009-12-08 12:32:03.829498891 +0000
@@ -7183,6 +7183,7 @@ get_job_attrs(cupsd_client_t  *con,	/* I
   ipp_attribute_t *attr;		/* Current attribute */
   int		jobid;			/* Job ID */
   cupsd_job_t	*job;			/* Current job */
+  cupsd_printer_t *printer;		/* Current printer */
   char		scheme[HTTP_MAX_URI],	/* Method portion of URI */
 		username[HTTP_MAX_URI],	/* Username portion of URI */
 		host[HTTP_MAX_URI],	/* Host portion of URI */
@@ -7257,7 +7258,19 @@ get_job_attrs(cupsd_client_t  *con,	/* I
   * Check policy...
   */
 
-  if ((status = cupsdCheckPolicy(DefaultPolicyPtr, con, NULL)) != HTTP_OK)
+  if ((printer = job->printer) == NULL)
+    printer = cupsdFindDest(job->dest);
+
+  if (printer)
+  {
+    if ((status = cupsdCheckPolicy(printer->op_policy_ptr, con,
+                                   NULL)) != HTTP_OK)
+    {
+      send_http_error(con, status, printer);
+      return;
+    }
+  }
+  else if ((status = cupsdCheckPolicy(DefaultPolicyPtr, con, NULL)) != HTTP_OK)
   {
     send_http_error(con, status, NULL);
     return;


Index: cups.spec
===================================================================
RCS file: /cvs/pkgs/rpms/cups/F-11/cups.spec,v
retrieving revision 1.553
retrieving revision 1.554
diff -u -p -r1.553 -r1.554
--- cups.spec	8 Dec 2009 12:30:16 -0000	1.553
+++ cups.spec	8 Dec 2009 12:33:34 -0000	1.554
@@ -72,6 +72,7 @@ Patch58: cups-str3435.patch
 Patch59: cups-str3436.patch
 Patch60: cups-str3425.patch
 Patch61: cups-str3428.patch
+Patch62: cups-str3431.patch
 
 Patch100: cups-lspp.patch
 Epoch: 1
@@ -248,6 +249,7 @@ module. 
 %patch59 -p1 -b .str3436
 %patch60 -p1 -b .str3425
 %patch61 -p1 -b .str3428
+%patch62 -p1 -b .str3431
 
 %if %lspp
 %patch100 -p1 -b .lspp
@@ -540,6 +542,8 @@ rm -rf $RPM_BUILD_ROOT
 
 %changelog
 * Tue Dec  8 2009 Tim Waugh <twaugh at redhat.com> - 1:1.4.2-14
+- The scheduler did not use the Get-Job-Attributes policy for a
+  printer (STR #3431).
 - The scheduler added two job-name attributes to each job object
   (STR #3428).
 - The scheduler did not clean out completed jobs when