X-Git-Url: http://git.asbjorn.biz/?a=blobdiff_plain;f=lib%2Fmodules%2Fswfbits.c;h=c0c4bb8d73d172eec554ffffbd74e8074b9fde8d;hb=6885c9905b83d8facc69f7043a226d6c8584f558;hp=b301532b6a1cfc702324518e85b06b548760fbf8;hpb=2f3e7cf3da2d5ffa26cd770269e4401b1d27d909;p=swftools.git diff --git a/lib/modules/swfbits.c b/lib/modules/swfbits.c index b301532..c0c4bb8 100644 --- a/lib/modules/swfbits.c +++ b/lib/modules/swfbits.c @@ -300,24 +300,11 @@ int swf_SetLosslessBits(TAG * t,U16 width,U16 height,void * bitmap,U8 bitmap_fla fprintf(stderr, "rfxswf: unknown bitmap type %d\n", bitmap_flags); return -1; } - + swf_SetU8(t,bitmap_flags); swf_SetU16(t,width); swf_SetU16(t,height); - - /* fix for buggy flash players which can't handle plain-color bitmaps - TODO: is there a better solution? - */ - { int s; - int l=32; - for(s=0;sl) - break; - } - } - { z_stream zs; memset(&zs,0x00,sizeof(z_stream)); @@ -333,6 +320,17 @@ int swf_SetLosslessBits(TAG * t,U16 width,U16 height,void * bitmap,U8 bitmap_fla } else res = -3; // zlib error } + + while(t->len < 64) { /* actually, 63 and above is o.k., but let's stay on the safe side */ + + /* Flash players up to MX crash or do strange things if they encounter a + DefineLossless Tag with a payload of less than 63 bytes. They also + substitute the whole bitmap by a red rectangle. + + This loop fills up the tag with zeroes so that this doesn't happen. + */ + swf_SetU8(t, 0); + } return res; } @@ -340,7 +338,6 @@ int swf_SetLosslessBits(TAG * t,U16 width,U16 height,void * bitmap,U8 bitmap_fla int swf_SetLosslessBitsIndexed(TAG * t,U16 width,U16 height,U8 * bitmap,RGBA * palette,U16 ncolors) { RGBA * pal = palette; int bps = BYTES_PER_SCANLINE(width); - U8 * data; int res = 0; if (!pal) // create default palette for grayscale images @@ -415,12 +412,9 @@ int swf_SetLosslessBitsIndexed(TAG * t,U16 width,U16 height,U8 * bitmap,RGBA * p deflateEnd(&zs); - if (zs.next_out>data) swf_SetBlock(t,data,zs.next_out-data); - free(zpal); } else res = -2; // memory error } else res = -3; // zlib error - free(data); } if (!palette) free(pal);