Closed
Bug 1971
Opened 26 years ago
Closed 26 years ago
large jpeg image (>8000) not handled properly.
Categories
(Core :: Layout, defect, P3)
Tracking
()
VERIFIED
FIXED
M8
People
(Reporter: pnunn, Assigned: pnunn)
References
()
Details
large jpeg image (>8000) not handled properly. crashes.
The code in jpeg.cpp for il_jpeg_write() doesn't NULL check
ic->ds before it uses it. bad. bad. bad. I'm also taking out
the assert and putting in code that will actually handle the
problem.
-pn
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Updated•26 years ago
|
Status: RESOLVED → REOPENED
Updated•26 years ago
|
Resolution: FIXED → ---
Comment 1•26 years ago
|
||
Re-opening.
Image displays as blank in 1.28.99 builds of:
- Win32 viewer
- Linux Viewer
Displayed as broken image on:
- Mac viewer (possibly due to low memory on Mac), as well as on Mac
4.5.1m2.
Displays correctly on:
- Mac IE 4.5.
eli:
I don't think this needs to be reopened.
By design images in the browser must be less
than 8000 pixels in either dimension.
Are you concerned that you get a broken image icon
on one platform and blanks on some others. It would
be good to be consistant.
If we decide to change the spec so images greater than
8000 are viewable, we should change it AFTER we have
a basic working product.
I'll wait to close this for your comment.
-pn
Updated•26 years ago
|
QA Contact: 1698
Comment 4•26 years ago
|
||
[QA Assigning to Self]
Comment 5•26 years ago
|
||
Hi, Pam ---
Sorry; didn't CC self on this bug, and missed your comment on 2.1.99 until today.
In an ideal universe, yes, I'd think we should be able to handle whatever images
IE can handle.
In a somewhat ideal universe, yes, I think it would also help tech support & help
desk folks to at least provide the same error state cross-platform in this case,
if we won't be supporting it.
How would you feel about at least going for the somewhat ideal universe right
now, and bumping down the bug priority to a P3/Minor?
Thanks!
Updated•26 years ago
|
Target Milestone: M4 → M5
Comment 7•26 years ago
|
||
pnunn's not here for the m4 endgame. moving to m5
I have a small check in for this, if the tree ever opens....
-pn
Status: ASSIGNED → RESOLVED
Closed: 26 years ago → 26 years ago
Resolution: --- → FIXED
Updated•26 years ago
|
Status: RESOLVED → VERIFIED
Comment 9•26 years ago
|
||
Verified using 7.9.99 PM build; on Win32/Linux/Mac OS, the browser properly
displays the URL, rather than crashing on the image.
You need to log in
before you can comment on or make changes to this bug.
Description
•