pdf2ps wrapper cannot translate from STDIN (patch included)

Bug #17124 reported by sullr
4
Affects Status Importance Assigned to Milestone
cupsys (Ubuntu)
Fix Released
Medium
Martin Pitt

Bug Description

the pdf2ps wrapper in /usr/lib/cups/filter/pdf2ps tries to copy STDIN to
tempfile if no file name is given.
It looks like that with the perl in hoary the expression File::Copy::copy(
STDIN,... ) tries to access STDIN as file name instead of file handle. Thus the
filter fails. The simple fix is:

--- /usr/lib/cups/filter/pdftops.orig 2005-05-15 21:20:29.843200216 +0200
+++ /usr/lib/cups/filter/pdftops 2005-05-15 21:05:33.205509864 +0200
@@ -89,7 +89,7 @@
 {
        my $tmpfile = $ENV{TMPDIR} . "pdfin.$$.tmp";
        open (TEMP, ">$tmpfile" ) || die ("ERROR: pdftops wrapper: $tmpfile: $!\n");
- if (! copy (STDIN, TEMP))
+ if (! copy ( \*STDIN, \*TEMP))
        {
                close (TEMP);
                unlink $tmpfile;

Revision history for this message
Martin Pitt (pitti) wrote :

Taking.

Changed in cupsys:
assignee: ogra → pitti
status: Unconfirmed → Confirmed
Revision history for this message
Martin Pitt (pitti) wrote :

Current development version is fixed, will upload soon.

Changed in cupsys:
status: Confirmed → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

 cupsys (1.1.99.b1.r4929-0ubuntu5) dapper; urgency=low
 .
   * debian/patches/51_dont_log_ipp_printer_query.dpatch: Suppress logging of
     successful CUPS_GET_DEFAULT messages, too. Closes: LP#29895
   * debian/pdftops: Fix reading from stdin. Closes: LP#17124

Changed in cupsys:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.