37909 - image multi mixed triggers assert

Status: VERIFIED FIXED

(SeaMonkey :: General, defect, P3)

Description

[pnunn]

page with cgi delivering stream of gifs gives assert in
il_gif_write.

Looks like stream has not gone thru il_gif_init (no gs structure).

The cgi:
http://cgi.netscape.com/cgi-bin/doit.cgi

Comment 1

[pnunn]

Looks like ic->state is not getting set to IC_MULTI. And need
a test to be sure il_gif_abort isn't triggered in
process_buffered_gif_input_data(). Can use IC_MULTI for test.
-pn

Comment 2

[Suresh Duddi (gone)]

Marking nsbeta2 (didn't mean to change Milestone)

Comment 3

[leger]

PDT requests more info on this for nsbeta2. What's the user-visible impact of 
this bug?

Comment 4

[Judson Valeski]

server push sites (weather, traffic, surf cams, mozilla examples, more sites 
that you reealize) don't work.

Comment 5

[leger]

Putting on [nsbeta2+] radar for beta2 fix.

Comment 6

[pnunn]

Attachment: diff file for bug fix. May leak.

Comment 7

[Judson Valeski]

patch looks good to me. I don't see the leak potential though.

Comment 8

[pnunn]

Ok. Let me prove it to myself that it isn't leaking
the image container and I'll check it in.
-P

Comment 9

[pnunn]

Checked in. Verified that image container is destroyed properly.
-P

Comment 10

[leger]

Marking Verified per pnunns last comment.