Last modified: 2009-06-09 20:56:42 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T21089, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 19089 - animated gif brought down wiki
animated gif brought down wiki
Status: RESOLVED DUPLICATE of bug 16451
Product: MediaWiki
Classification: Unclassified
File management (Other open bugs)
unspecified
All All
: Normal major (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks: gif
  Show dependency treegraph
 
Reported: 2009-06-05 11:04 UTC by Jools Wills
Modified: 2009-06-09 20:56 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Jools Wills 2009-06-05 11:04:16 UTC
A user uploaded an animated gif ~400k that has many frames to my wiki. Convert seemed to take forever to finish holding up fastcgi and causing it to stop responding (effectively being my site down)

Do we actually need to scale/thumbnail animated gifs? wouldn't it be better to use 

convert blah.gif[0] by default on all gifs to only process the first frame ?

the evil gif

http://malus.exotica.org.uk/~buzz/animated.gif

I;m marking this as major, as the defaults are not sensible enough to protect from this. The Bitmap.php code only processes the first frame if

if( $srcWidth * $srcHeight > $wgMaxAnimatedGifArea ) {

however this doesn't help for a gif with small dimensions but many frames.
Comment 1 Gurch 2009-06-06 09:26:51 UTC
Wikimedia wikis used to refuse to thumbnail GIFs at all (even non-animated ones, for some reason) owing to this, but they now do again, so I'm assuming (or at least hoping) they have some mechanism in place to prevent this.
Comment 2 Steve Sanbeg 2009-06-09 20:56:42 UTC
It should use the first frame if ( $srcWidth * $srcHeight * $srcFrames > $wgMaxAnimatedGifArea ), but once the functionality to count frames is added, this would be resolved one way or the other.


*** This bug has been marked as a duplicate of bug 16451 ***

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links