Closed Bug 1971 Opened 26 years ago Closed 26 years ago

large jpeg image (>8000) not handled properly.

Categories

(Core :: Layout, defect, P3)

x86
Windows NT
defect

Tracking

()

VERIFIED FIXED

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: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
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
Setting all current Open/Normal to M4.
QA Contact: 1698
[QA Assigning to Self]
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!
Priority: P2 → P3
ok
Target Milestone: M4 → M5
pnunn's not here for the m4 endgame. moving to m5
Target Milestone: M5 → M6
Target Milestone: M6 → M8
Status: REOPENED → ASSIGNED
I have a small check in for this, if the tree ever opens.... -pn
Status: ASSIGNED → RESOLVED
Closed: 26 years ago26 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
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.