|
|
1ad933 |
diff --git a/libtiff/tif_next.c b/libtiff/tif_next.c
|
|
|
1ad933 |
index b014685..13c0848 100644
|
|
|
1ad933 |
--- a/libtiff/tif_next.c
|
|
|
1ad933 |
+++ b/libtiff/tif_next.c
|
|
|
1ad933 |
@@ -37,7 +37,7 @@
|
|
|
1ad933 |
case 0: op[0] = (unsigned char) ((v) << 6); break; \
|
|
|
1ad933 |
case 1: op[0] |= (v) << 4; break; \
|
|
|
1ad933 |
case 2: op[0] |= (v) << 2; break; \
|
|
|
1ad933 |
- case 3: *op++ |= (v); break; \
|
|
|
1ad933 |
+ case 3: *op++ |= (v); op_offset++; break; \
|
|
|
1ad933 |
} \
|
|
|
1ad933 |
}
|
|
|
1ad933 |
|
|
|
1ad933 |
@@ -107,6 +107,8 @@ NeXTDecode(TIFF* tif, uint8* buf, tmsize_t occ, uint16 s)
|
|
|
1ad933 |
if( isTiled(tif) )
|
|
|
1ad933 |
imagewidth = tif->tif_dir.td_tilewidth;
|
|
|
1ad933 |
|
|
|
1ad933 |
+ tmsize_t op_offset = 0;
|
|
|
1ad933 |
+
|
|
|
1ad933 |
/*
|
|
|
1ad933 |
* The scanline is composed of a sequence of constant
|
|
|
1ad933 |
* color ``runs''. We shift into ``run mode'' and
|
|
|
1ad933 |
@@ -120,10 +122,15 @@ NeXTDecode(TIFF* tif, uint8* buf, tmsize_t occ, uint16 s)
|
|
|
1ad933 |
* bounds, potentially resulting in a security
|
|
|
1ad933 |
* issue.
|
|
|
1ad933 |
*/
|
|
|
1ad933 |
- while (n-- > 0 && npixels < imagewidth)
|
|
|
1ad933 |
+ while (n-- > 0 && npixels < imagewidth && op_offset < scanline)
|
|
|
1ad933 |
SETPIXEL(op, grey);
|
|
|
1ad933 |
if (npixels >= imagewidth)
|
|
|
1ad933 |
break;
|
|
|
1ad933 |
+ if (op_offset >= scanline ) {
|
|
|
1ad933 |
+ TIFFErrorExt(tif->tif_clientdata, module, "Invalid data for scanline %ld",
|
|
|
1ad933 |
+ (long) tif->tif_row);
|
|
|
1ad933 |
+ return (0);
|
|
|
1ad933 |
+ }
|
|
|
1ad933 |
if (cc == 0)
|
|
|
1ad933 |
goto bad;
|
|
|
1ad933 |
n = *bp++, cc--;
|
|
|
1ad933 |
@@ -162,9 +169,9 @@ NeXTPreDecode(TIFF* tif, uint16 s)
|
|
|
1ad933 |
TIFFInitNeXT(TIFF* tif, int scheme)
|
|
|
1ad933 |
{
|
|
|
1ad933 |
(void) scheme;
|
|
|
1ad933 |
- tif->tif_predecode = NeXTPreDecode;
|
|
|
1ad933 |
- tif->tif_decoderow = NeXTDecode;
|
|
|
1ad933 |
- tif->tif_decodestrip = NeXTDecode;
|
|
|
1ad933 |
+ tif->tif_predecode = NeXTPreDecode;
|
|
|
1ad933 |
+ tif->tif_decoderow = NeXTDecode;
|
|
|
1ad933 |
+ tif->tif_decodestrip = NeXTDecode;
|
|
|
1ad933 |
tif->tif_decodetile = NeXTDecode;
|
|
|
1ad933 |
return (1);
|
|
|
1ad933 |
}
|