Closed
Bug 37909
Opened 25 years ago
Closed 25 years ago
image multi mixed triggers assert
Categories
(SeaMonkey :: General, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
M16
People
(Reporter: pnunn, Assigned: pnunn)
References
()
Details
(Whiteboard: [nsbeta2+])
Attachments
(1 file)
|
(deleted),
patch
|
Details | Diff | Splinter Review |
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
Status: NEW → ASSIGNED
Summary: image multi mixed triggers assert → image multi mixed triggers assert
Target Milestone: --- → M16
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
|
||
Updated•25 years ago
|
Target Milestone: M16 → M17
|
|
||
Comment 2•25 years ago
|
||
Marking nsbeta2 (didn't mean to change Milestone)
Keywords: nsbeta2
Target Milestone: M17 → M16
PDT requests more info on this for nsbeta2. What's the user-visible impact of
this bug?
|
||
Comment 4•25 years ago
|
||
server push sites (weather, traffic, surf cams, mozilla examples, more sites
that you reealize) don't work.
Putting on [nsbeta2+] radar for beta2 fix.
Whiteboard: [NEED INFO] → [nsbeta2+]
|
|
||
Comment 7•25 years ago
|
||
patch looks good to me. I don't see the leak potential though.
Ok. Let me prove it to myself that it isn't leaking
the image container and I'll check it in.
-P
Checked in. Verified that image container is destroyed properly.
-P
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
|
||
Comment 10•25 years ago
|
||
Marking Verified per pnunns last comment.
|
||
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•