Crash in ImageLib when a background image is applied to a textarea

VERIFIED FIXED in M17

Status

()

Core
ImageLib
P3
critical
VERIFIED FIXED
18 years ago
18 years ago

People

(Reporter: Pierre Saslawsky, Assigned: pnunn)

Tracking

({crash})

Trunk
crash
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [nsbeta2+], URL)

Attachments

(2 attachments)

(Reporter)

Description

18 years ago
This bug is extracted from bug 36128 "fixed value for background-image and 
Textarea".

http://basketball.jouh.at/Test.html causes a crash on WinNT and Mac. It crashes 
in if.cpp line 1925 because 'firstURI' is nil in the expression:
     firstURI->Equals(uri, &eq);)
(Reporter)

Updated

18 years ago
Keywords: crash
(Reporter)

Comment 1

18 years ago
I'm going to attach a better testcase. After fixing the crash, if you don't see a 
background-image inside the TEXTAREAs, please reassign to the Editor team.
(Reporter)

Comment 2

18 years ago
Created attachment 7705 [details]
testcase which shouldn't crash and should show images inside TEXTAREAs
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → M16

Updated

18 years ago
Target Milestone: M16 → M17

Updated

18 years ago
Keywords: nsbeta2

Comment 3

18 years ago
Putting on [nsbeta2+] radar for beta2 fix.  Can defensive coding stop the crash?
Whiteboard: [nsbeta2+]
(Assignee)

Comment 4

18 years ago
Created attachment 8874 [details] [diff] [review]
patch to test URI return;
(Assignee)

Comment 5

18 years ago
Note that fix is attached to bug.
-P

Comment 6

18 years ago
Oops, that was my fault, sorry.  Thanks for fixing it Pam -- your fix was 
obviously the right thing to do.
(Assignee)

Comment 7

18 years ago
Got a rev:Morse

I'll check in when the tree is green/open.
thnks,Steve.
-P
(Assignee)

Comment 8

18 years ago
checked in fix.
-P
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 9

18 years ago
verified in 5/23 build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.