78134 - ###!!! ASSERTION: OnStopDecode called multiple times.: '!(mState & onStopDecode)', file

Status: RESOLVED INVALID

(Core :: Graphics: ImageLib, defect, P2)

Description

[timeless]

SunOS 5.7 Generic_106541-15 sun4u sparc SUNW,Ultra-2
mozilla cvs build for xlib from this morning.

steps to reproduce run choffman's browser buster, this assertion happens for 
choffman urls: 66, 69, 86, 91, and 100.

Comment 1

[timeless]

Attachment: snipets from running choffman browser buster.

Comment 2

[Stuart Parmenter]

can you please post at least 1 real url that points to a page where you see 
this?

Comment 3

[John Bandhauer]

I see this consistenly on NT at http://www.kcbs.com

I'll attach a stack.

Comment 4

[John Bandhauer]

Attachment: assertion stack

Comment 5

[Stuart Parmenter]

works for me today.. I think that saari fixed this bug.

Comment 6

[Terri Preston]

Saw this assertion on peterl's build from yesterday (06/04/01), reopening

Comment 7

[Stuart Parmenter]

argh.  where?

Comment 8

[Terri Preston]

http://www.kcbs.com

Comment 9

[John Bandhauer]

Yeah. I still see this on that URL. I didn't realize this had been closed.

Comment 10

[Chris Abbey]

another URL that's showing this is http://sinfest.net/ (no, it's NOT what you
think it is... ;)  

###!!! ASSERTION: OnStopDecode called multiple times.: '!(mState &
onStopDecode)', file imgRequest.cpp, line 512
###!!! Break: at file imgRequest.cpp, line 512
###!!! ASSERTION: OnStopDecode called multiple times.: '!(mState &
onStopDecode)', file imgRequest.cpp, line 512
###!!! Break: at file imgRequest.cpp, line 512
###!!! ASSERTION: Multiple GetIIDs function for the same nsDOMClassInfoID:
'sClassInfoData[aID].mGetIIDsFptr == aGetIIDsFptr', file nsDOMClassInfo.cpp,
line 993
###!!! Break: at file nsDOMClassInfo.cpp, line 993

Comment 11

[Chris Abbey]

Attachment: current stack, from linux, via XPCOM_DEBUG_BREAK instead of gdb

Comment 12

[Stuart Parmenter]

0.9.3

Comment 13

[Jesse Ruderman]

Also http://sharkysoft.com/.

Comment 14

[Jason Kersey]

-> 0.9.4

Comment 15

[Stuart Parmenter]

I believe I fixed this when I checked in a patch to handle GIF errors properly.

Comment 16

[Stuart Parmenter]

*** Bug 94241 has been marked as a duplicate of this bug. ***

Comment 17

[David Epstein]

Using Mozilla 0.9.9 Gecko 20020304 debug build, the assert happens when loading:

http://sinfest.net/ (2nd assert; after nsCookieService assert)
http://www.monsanto.com

From bug 94241:

http://www.airliners.net/search/photo.search

Other urls in this bug work fine.

Comment 18

[David Epstein]

reopening as some urls cause the assert (see above)

Comment 19

[ArronM (:paper)]

I've had this assertion occur when I wouldn't let an image fully load (I'd go to
another page before it finished loading).  Then, when going back to the page,
the assertion would fire (after the image was fully loaded)

Comment 20

[Heikki Toivonen (remove -bugzilla when emailing directly)]

I see these assertions on http://www.keskisuomalainen.net as well. Beware,
because of another bug the product will stop handling most events meaning you
will have to kill it.

Comment 21

[Nicholas Allen]

*** Bug 193472 has been marked as a duplicate of this bug. ***

Comment 22

[Tim]

http://i.real.com/pics/real/main/curve.gif from http://www.real.com is an
example (the cause here is a corrupt image--it has ~600K of arbitrary data after
the ~1.4K image). The problem is generally that
http://lxr.mozilla.org/seamonkey/source/modules/libpr0n/decoders/gif/GIF2.cpp#996
(gif_write) gets called more than once with a gif_done status.

Perhaps gif_write should return failure if it gets entered again after gif_done;
however, that's a guess, since I don't know this code.

Comment 23

[Benoit Girard (:BenWa)]

This assert is no longer in the codebase.