Ticket #190: patchtiff.diff

libavcodec/tiff.c

@@ -166 +166 @@
         return ret;
     }
     for(line = 0; line < lines; line++){
+        int i;
         if(src - ssrc > size){
             av_log(s->avctx, AV_LOG_ERROR, "Source data overread\n");
             return -1;
@@ -173 +174 @@
         switch(s->compr){
         case TIFF_RAW:
             if (!s->fill_order) {
-                memcpy(dst, src, width);
+                if (s->bpp == 2) {
+                    for(i = 0; i < width; i++) {
+                        dst[i*4+0] = src[i] >> 6;
+                        dst[i*4+1] = src[i] >> 4 & 0x3;
+                        dst[i*4+2] = src[i] >> 2 & 0x3;
+                        dst[i*4+3] = src[i] & 0x3;
+                    }
+                } else if (s->bpp == 4) {
+                    for(i = 0; i < width; i++) {
+                        dst[i*2+0] = src[i] >> 4;
+                        dst[i*2+1] = src[i] & 0xF;
+                    }
+                } else {
+                    memcpy(dst, src, width);
+                }
             } else {
-                int i;
                 for (i = 0; i < width; i++)
                     dst[i] = av_reverse[src[i]];
             }
@@ -190 +204 @@
                         av_log(s->avctx, AV_LOG_ERROR, "Copy went out of bounds\n");
                         return -1;
                     }
-                    memcpy(dst + pixels, src, code);
+                    if (s->bpp == 2) {
+                        for (i = 0; i < code; i++) {
+                            dst[(pixels+i)*4+0] = src[i] >> 6;
+                            dst[(pixels+i)*4+1] = src[i] >> 4 & 0x3;
+                            dst[(pixels+i)*4+2] = src[i] >> 2 & 0x3;
+                            dst[(pixels+i)*4+3] = src[i] & 0x3;
+                        }
+                    } else if (s->bpp == 4) {
+                        for(i = 0; i < code; i++) {
+                            dst[(pixels+i)*2+0] = src[i] >> 4;
+                            dst[(pixels+i)*2+1] = src[i] & 0xF;
+                        }
+                    } else {
+                        memcpy(dst + pixels, src, code);
+                    }
                     src += code;
                     pixels += code;
                 }else if(code != -128){ // -127..-1
@@ -200 +228 @@
                         return -1;
                     }
                     c = *src++;
-                    memset(dst + pixels, c, code);
+                    if (s->bpp == 2) {
+                        for (i = 0; i < code; i++) {
+                            dst[(pixels+i)*4+0] = c >> 6;
+                            dst[(pixels+i)*4+1] = c >> 4 & 0x3;
+                            dst[(pixels+i)*4+2] = c >> 2 & 0x3;
+                            dst[(pixels+i)*4+3] = c & 0x3;
+                        }
+                    } else if (s->bpp == 4) {
+                        for(i = 0; i < code; i++) {
+                            dst[(pixels+i)*2+0] = c >> 4;
+                            dst[(pixels+i)*2+1] = c & 0xF;
+                        }
+                    } else {
+                        memset(dst + pixels, c, code);
+                    }
                     pixels += code;
                 }
             }
@@ -227 +269 @@
     case 11:
         s->avctx->pix_fmt = PIX_FMT_MONOBLACK;
         break;
+    case 21:
+    case 41:
     case 81:
         s->avctx->pix_fmt = PIX_FMT_PAL8;
         break;