Bug 807858

Created attachment 573488 [details]
slapdash patch that might fix problem

Description of problem:
The fix referenced in http://rhn.redhat.com/errata/RHSA-2012-0301.html
for the private bz #694922 breaks convert, when converting from PS
input files.

Version-Release number of selected component (if applicable):
ImageMagick-6.2.8.0-12.el5

How reproducible:
convert file.ps file.whatever

Steps to Reproduce:
1.
2.
3.
  
Actual results:
[ghostscript library] -q -dBATCH -dSAFER -dMaxBitmap=500000000 -dNOPAUSE -dAlignToPixels=0 -dEPSCrop "-sDEVICE=pnmraw" -dTextAlphaBits=4 -dGraphicsAlphaBits=4 "-r612x792" 72x72 "-sOutputFile=" "-f/tmp/magick-XXOcprc6" "-f/tmp/magick-XXeCsmoL"
Error: /undefinedfilename in (72x72)
...
convert: no decode delegate for this image format `file.ps'.
convert: Postscript delegate failed `file.ps'.
convert: missing an image filename `file.jpg'.

 
Expected results:
successful file conversion

Additional info:
It seems like the
    ImageMagick-6.2.8-rotate-PDF.patch
changes coders/pdf.c to insert one less string into the ``delegate
command'' format string

@@ -504,7 +517,7 @@ static Image *ReadPDFImage(const ImageIn
   (void) FormatMagickString(command,MaxTextExtent,
     GetDelegateCommands(delegate_info),
     read_info->antialias != MagickFalse ? 4 : 1,
-    read_info->antialias != MagickFalse ? 4 : 1,geometry,density,options,
+    read_info->antialias != MagickFalse ? 4 : 1,density,options,
     read_info->filename,postscript_filename,input_filename);
   status=InvokePostscriptDelegate(read_info->verbose,command);
   image=ReadImage(read_info,exception);

There's corresponding changes to the delegates.xml file, like

-  <delegate decode="ps:alpha" stealth="True" command='"@PSDelegate@" -q dBATCH -dSAFER -dMaxBitmap=500000000 -dNOPAUSE -dAlignToPixels=0 -dEPSCrop -sDEVICE=@GSAlphaDevice@" -dTextAlphaBits=%u -dGraphicsAlphaBits=%u "-g%s" -r%s" %s "-sOutputFile=%s" "-f%s" "-f%s"' />

+  <delegate decode="ps:alpha" stealth="True" command='"@PSDelegate@" -q dBATCH -dSAFER -dMaxBitmap=500000000 -dNOPAUSE -dAlignToPixels=0 -dEPSCrop -sDEVICE=@GSAlphaDevice@" -dTextAlphaBits=%u -dGraphicsAlphaBits=%u "-r%s" %s "-sOutputFile=%s" "-f%s" "-f%s"' />

where the "-g%s" token has been dropped

Unfortunately, coders/ps.c uses the same delegates, and wasn't updated
accordingly, so, at a minimum, something like the attached needs to be
done to that file.  (I say, ``like,'' because ps.c might also call for
the same ``rotate fix'' that pdf.c did, and not just this band-aid that
fixes the regression.  Also, i haven't actually compiled or tested the
patched code)