Status: Read
Received: from [207.176.88.85] by phillip.cerious.com (NTMail 3.03.0017/7.aaxh) with ESMTP id pcrews for <pcrews@cerious.com>; Mon, 27 Mar 2000 12:28:10 -0500
Received: from localhost (daemon@localhost)
	by gate7.olympiakos.com (8.9.3/8.9.3) with SMTP id MAA13150;
	Mon, 27 Mar 2000 12:06:03 -0500
Received: by gate7.olympiakos.com (bulk_mailer v1.9); Mon, 27 Mar 2000 12:04:35 -0500
Received: (from majordom@localhost)
	by gate7.olympiakos.com (8.9.3/8.9.3) id MAA13072
	for tiff-outgoing; Mon, 27 Mar 2000 12:04:34 -0500
X-Authentication-Warning: gate7.olympiakos.com: majordom set sender to owner-tiff@olympiakos.com using -f
Received: from devserv.devel.redhat.com (IDENT:root@devserv.devel.redhat.com [207.175.42.156])
	by gate7.olympiakos.com (8.9.3/8.9.3) with ESMTP id MAA13069
	for <tiff@olympiakos.com>; Mon, 27 Mar 2000 12:04:32 -0500
Received: (from nalin@localhost)
	by devserv.devel.redhat.com (8.9.3/8.9.3) id MAA14658;
	Mon, 27 Mar 2000 12:04:22 -0500
Date: Mon, 27 Mar 2000 12:04:22 -0500
From: Nalin Dahyabhai <nalin@redhat.com>
To: Frank Warmerdam <warmerda@home.com>
Cc: TIFF Mailing List <tiff@olympiakos.com>
Subject: Re: Some CCITT Group 3 TIFFs crash libtiff
Message-ID: <20000327120422.B14134@devserv.devel.redhat.com>
References: <38CFCC1B.B75FE060@med-web.com> <38CFCFAD.2B9824BC@home.com> <38D26078.C1991FF7@med-web.com> <38D27AA9.303107C4@home.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
X-Mailer: Mutt 1.0.1i
In-Reply-To: <38D27AA9.303107C4@home.com>; from warmerda@home.com on Fri, Mar 17, 2000 at 01:34:17PM -0500
X-Random-Fortune: Luke, I'm yer father, eh. Come over to the dark side, you hoser.  -- Dave Thomas, "Strange Brew"
Organization: Red Hat, Inc.
X-Department: OS Development
Sender: owner-tiff@olympiakos.com

On Fri, Mar 17, 2000 at 01:34:17PM -0500, Frank Warmerdam wrote:
> Good news, I have worked out what the bug is.  In the last few months the
> fax3 code was changed to handle files wider (or longer?) than 65536 pixels
> and part of this was converting the runs arrays from uint16 to uint32.

Turns out this change also broke fax2ps.  The fix is trivial (below).

Cheers,

Nalin

================================================================================
--- tools/fax2ps.c	Mon Mar 27 11:44:08 2000
+++ tools/fax2ps.c	Mon Mar 27 11:44:35 2000
@@ -62,7 +62,7 @@
  * March 13-15, 1995.
  */
 static void
-printruns(unsigned char* buf, uint16* runs, uint16* erun, uint32 lastx)
+printruns(unsigned char* buf, uint32* runs, uint32* erun, uint32 lastx)
 {
     static struct {
 	char white, black;
