bddd17
diff -Naur tiff-4.0.3.orig/tools/tiff2pdf.c tiff-4.0.3/tools/tiff2pdf.c
bddd17
--- tiff-4.0.3.orig/tools/tiff2pdf.c	2012-07-25 22:56:43.000000000 -0400
bddd17
+++ tiff-4.0.3/tools/tiff2pdf.c	2013-05-02 12:04:49.057090227 -0400
bddd17
@@ -3341,33 +3341,56 @@
bddd17
 	uint32 height){
bddd17
 
bddd17
 	tsize_t i=0;
bddd17
-	uint16 ri =0;
bddd17
-	uint16 v_samp=1;
bddd17
-	uint16 h_samp=1;
bddd17
-	int j=0;
bddd17
-	
bddd17
-	i++;
bddd17
-	
bddd17
-	while(i<(*striplength)){
bddd17
+
bddd17
+	while (i < *striplength) {
bddd17
+		tsize_t datalen;
bddd17
+		uint16 ri;
bddd17
+		uint16 v_samp;
bddd17
+		uint16 h_samp;
bddd17
+		int j;
bddd17
+		int ncomp;
bddd17
+
bddd17
+		/* marker header: one or more FFs */
bddd17
+		if (strip[i] != 0xff)
bddd17
+			return(0);
bddd17
+		i++;
bddd17
+		while (i < *striplength && strip[i] == 0xff)
bddd17
+			i++;
bddd17
+		if (i >= *striplength)
bddd17
+			return(0);
bddd17
+		/* SOI is the only pre-SOS marker without a length word */
bddd17
+		if (strip[i] == 0xd8)
bddd17
+			datalen = 0;
bddd17
+		else {
bddd17
+			if ((*striplength - i) <= 2)
bddd17
+				return(0);
bddd17
+			datalen = (strip[i+1] << 8) | strip[i+2];
bddd17
+			if (datalen < 2 || datalen >= (*striplength - i))
bddd17
+				return(0);
bddd17
+		}
bddd17
 		switch( strip[i] ){
bddd17
-			case 0xd8:
bddd17
-				/* SOI - start of image */
bddd17
+			case 0xd8:	/* SOI - start of image */
bddd17
 				_TIFFmemcpy(&(buffer[*bufferoffset]), &(strip[i-1]), 2);
bddd17
 				*bufferoffset+=2;
bddd17
-				i+=2;
bddd17
 				break;
bddd17
-			case 0xc0:
bddd17
-			case 0xc1:
bddd17
-			case 0xc3:
bddd17
-			case 0xc9:
bddd17
-			case 0xca:
bddd17
+			case 0xc0:	/* SOF0 */
bddd17
+			case 0xc1:	/* SOF1 */
bddd17
+			case 0xc3:	/* SOF3 */
bddd17
+			case 0xc9:	/* SOF9 */
bddd17
+			case 0xca:	/* SOF10 */
bddd17
 				if(no==0){
bddd17
-					_TIFFmemcpy(&(buffer[*bufferoffset]), &(strip[i-1]), strip[i+2]+2);
bddd17
-					for(j=0;j
bddd17
-						if( (buffer[*bufferoffset+11+(2*j)]>>4) > h_samp) 
bddd17
-							h_samp = (buffer[*bufferoffset+11+(2*j)]>>4);
bddd17
-						if( (buffer[*bufferoffset+11+(2*j)] & 0x0f) > v_samp) 
bddd17
-							v_samp = (buffer[*bufferoffset+11+(2*j)] & 0x0f);
bddd17
+					_TIFFmemcpy(&(buffer[*bufferoffset]), &(strip[i-1]), datalen+2);
bddd17
+					ncomp = buffer[*bufferoffset+9];
bddd17
+					if (ncomp < 1 || ncomp > 4)
bddd17
+						return(0);
bddd17
+					v_samp=1;
bddd17
+					h_samp=1;
bddd17
+					for(j=0;j
bddd17
+						uint16 samp = buffer[*bufferoffset+11+(3*j)];
bddd17
+						if( (samp>>4) > h_samp) 
bddd17
+							h_samp = (samp>>4);
bddd17
+						if( (samp & 0x0f) > v_samp) 
bddd17
+							v_samp = (samp & 0x0f);
bddd17
 					}
bddd17
 					v_samp*=8;
bddd17
 					h_samp*=8;
bddd17
@@ -3381,45 +3404,43 @@
bddd17
                                           (unsigned char) ((height>>8) & 0xff);
bddd17
 					buffer[*bufferoffset+6]=
bddd17
                                             (unsigned char) (height & 0xff);
bddd17
-					*bufferoffset+=strip[i+2]+2;
bddd17
-					i+=strip[i+2]+2;
bddd17
-
bddd17
+					*bufferoffset+=datalen+2;
bddd17
+					/* insert a DRI marker */
bddd17
 					buffer[(*bufferoffset)++]=0xff;
bddd17
 					buffer[(*bufferoffset)++]=0xdd;
bddd17
 					buffer[(*bufferoffset)++]=0x00;
bddd17
 					buffer[(*bufferoffset)++]=0x04;
bddd17
 					buffer[(*bufferoffset)++]=(ri >> 8) & 0xff;
bddd17
 					buffer[(*bufferoffset)++]= ri & 0xff;
bddd17
-				} else {
bddd17
-					i+=strip[i+2]+2;
bddd17
 				}
bddd17
 				break;
bddd17
-			case 0xc4:
bddd17
-			case 0xdb:
bddd17
-				_TIFFmemcpy(&(buffer[*bufferoffset]), &(strip[i-1]), strip[i+2]+2);
bddd17
-				*bufferoffset+=strip[i+2]+2;
bddd17
-				i+=strip[i+2]+2;
bddd17
+			case 0xc4: /* DHT */
bddd17
+			case 0xdb: /* DQT */
bddd17
+				_TIFFmemcpy(&(buffer[*bufferoffset]), &(strip[i-1]), datalen+2);
bddd17
+				*bufferoffset+=datalen+2;
bddd17
 				break;
bddd17
-			case 0xda:
bddd17
+			case 0xda: /* SOS */
bddd17
 				if(no==0){
bddd17
-					_TIFFmemcpy(&(buffer[*bufferoffset]), &(strip[i-1]), strip[i+2]+2);
bddd17
-					*bufferoffset+=strip[i+2]+2;
bddd17
-					i+=strip[i+2]+2;
bddd17
+					_TIFFmemcpy(&(buffer[*bufferoffset]), &(strip[i-1]), datalen+2);
bddd17
+					*bufferoffset+=datalen+2;
bddd17
 				} else {
bddd17
 					buffer[(*bufferoffset)++]=0xff;
bddd17
 					buffer[(*bufferoffset)++]=
bddd17
                                             (unsigned char)(0xd0 | ((no-1)%8));
bddd17
-					i+=strip[i+2]+2;
bddd17
 				}
bddd17
-				_TIFFmemcpy(&(buffer[*bufferoffset]), &(strip[i-1]), (*striplength)-i-1);
bddd17
-				*bufferoffset+=(*striplength)-i-1;
bddd17
+				i += datalen + 1;
bddd17
+				/* copy remainder of strip */
bddd17
+				_TIFFmemcpy(&(buffer[*bufferoffset]), &(strip[i]), *striplength - i);
bddd17
+				*bufferoffset+= *striplength - i;
bddd17
 				return(1);
bddd17
 			default:
bddd17
-				i+=strip[i+2]+2;
bddd17
+				/* ignore any other marker */
bddd17
+				break;
bddd17
 		}
bddd17
+		i += datalen + 1;
bddd17
 	}
bddd17
-	
bddd17
 
bddd17
+	/* failed to find SOS marker */
bddd17
 	return(0);
bddd17
 }
bddd17
 #endif