73 lines
2.6 KiB
Diff
73 lines
2.6 KiB
Diff
|
Partial fix for issues filed upstream at
|
||
|
|
http://bugzilla.maptools.org/show_bug.cgi?id=2140
|
||
|
|
This stops the tiffcmp core dump noted in bug #460322, but isn't enough
|
||
|
|
to make tiffcmp return the right answer (it emits a bunch of error
|
||
|
|
messages instead).
|
||
|
|
|
||
|
|
|
||
|
|
diff -Naur tiff-3.9.2.orig/libtiff/tif_jpeg.c tiff-3.9.2/libtiff/tif_jpeg.c
|
||
|
|
--- tiff-3.9.2.orig/libtiff/tif_jpeg.c 2009-08-30 12:21:46.000000000 -0400
|
||
|
|
+++ tiff-3.9.2/libtiff/tif_jpeg.c 2010-01-05 22:40:40.000000000 -0500
|
||
|
|
@@ -988,8 +988,15 @@
|
||
|
|
tsize_t nrows;
|
||
|
|
(void) s;
|
||
|
|
|
||
|
|
- /* data is expected to be read in multiples of a scanline */
|
||
|
|
- if ( (nrows = sp->cinfo.d.image_height) ) {
|
||
|
|
+ nrows = cc / sp->bytesperline;
|
||
|
|
+ if (cc % sp->bytesperline)
|
||
|
|
+ TIFFWarningExt(tif->tif_clientdata, tif->tif_name, "fractional scanline not read");
|
||
|
|
+
|
||
|
|
+ if( nrows > (int) sp->cinfo.d.image_height )
|
||
|
|
+ nrows = sp->cinfo.d.image_height;
|
||
|
|
+
|
||
|
|
+ /* data is expected to be read in multiples of a scanline */
|
||
|
|
+ if (nrows) {
|
||
|
|
/* Cb,Cr both have sampling factors 1, so this is correct */
|
||
|
|
JDIMENSION clumps_per_line = sp->cinfo.d.comp_info[1].downsampled_width;
|
||
|
|
int samples_per_clump = sp->samplesperclump;
|
||
|
|
@@ -1087,8 +1094,7 @@
|
||
|
|
* TODO: resolve this */
|
||
|
|
buf += sp->bytesperline;
|
||
|
|
cc -= sp->bytesperline;
|
||
|
|
- nrows -= sp->v_sampling;
|
||
|
|
- } while (nrows > 0);
|
||
|
|
+ } while (--nrows > 0);
|
||
|
|
|
||
|
|
#ifdef JPEG_LIB_MK1
|
||
|
|
_TIFFfree(tmpbuf);
|
||
|
|
diff -Naur tiff-3.9.2.orig/libtiff/tif_strip.c tiff-3.9.2/libtiff/tif_strip.c
|
||
|
|
--- tiff-3.9.2.orig/libtiff/tif_strip.c 2006-03-25 13:04:35.000000000 -0500
|
||
|
|
+++ tiff-3.9.2/libtiff/tif_strip.c 2010-01-05 21:39:20.000000000 -0500
|
||
|
|
@@ -238,23 +238,19 @@
|
||
|
|
ycbcrsubsampling + 0,
|
||
|
|
ycbcrsubsampling + 1);
|
||
|
|
|
||
|
|
- if (ycbcrsubsampling[0] == 0) {
|
||
|
|
+ if (ycbcrsubsampling[0]*ycbcrsubsampling[1] == 0) {
|
||
|
|
TIFFErrorExt(tif->tif_clientdata, tif->tif_name,
|
||
|
|
"Invalid YCbCr subsampling");
|
||
|
|
return 0;
|
||
|
|
}
|
||
|
|
|
||
|
|
- scanline = TIFFroundup(td->td_imagewidth,
|
||
|
|
+ /* number of sample clumps per line */
|
||
|
|
+ scanline = TIFFhowmany(td->td_imagewidth,
|
||
|
|
ycbcrsubsampling[0]);
|
||
|
|
- scanline = TIFFhowmany8(multiply(tif, scanline,
|
||
|
|
- td->td_bitspersample,
|
||
|
|
- "TIFFScanlineSize"));
|
||
|
|
- return ((tsize_t)
|
||
|
|
- summarize(tif, scanline,
|
||
|
|
- multiply(tif, 2,
|
||
|
|
- scanline / ycbcrsubsampling[0],
|
||
|
|
- "TIFFVStripSize"),
|
||
|
|
- "TIFFVStripSize"));
|
||
|
|
+ /* number of samples per line */
|
||
|
|
+ scanline = multiply(tif, scanline,
|
||
|
|
+ ycbcrsubsampling[0]*ycbcrsubsampling[1] + 2,
|
||
|
|
+ "TIFFScanlineSize");
|
||
|
|
} else {
|
||
|
|
scanline = multiply(tif, td->td_imagewidth,
|
||
|
|
td->td_samplesperpixel,
|