Agree completely that it's a nasty error that needs to get fixed.

The error is coming from these two lines in the get_details hash setup.

my ($width, $height, $packed, $bgcolor, $aspect) = unpack("vvCCC", $header);
   my $color_table_size = 1 << (($packed & 0x07) + 1);

Looking at older revisions of ImageInfo gives me no hints except that this has been there for a while.

Fundamentally, I don't know if a blank $packed is a legit return for the color table size or not.

If a blank $packed is legit, then something akin to (!defined $packed or $packed eq '') { $packed = 0;} might help.

My worry is that $packed ending up uninitialized is an issue where we should be aborting the imageinfo routines en masse.

Can you send me / point me towards the messages that might be getting the errors and I can look at the GIF's for more information?

Regards,
KAM

Reply via email to