Closed Bug 83467 Opened 24 years ago Closed 23 years ago

nsXBLPrototypeHandler::ConstructMask crashes

Categories

(Core :: XBL, defect, P2)

Product:
Core
Component:
XBL
Platform:
PowerPC
Mac System 9.x
Type:
defect

Tracking

()

Status:
VERIFIED INVALID
Milestone:
mozilla1.0.1

People

(Reporter: Brade, Assigned: Brade)

Details

(Keywords: crash)

nsXBLPrototypeHandler::ConstructMask will cause a crash when it calls nsCRT::strtok with a null string. ConstructMask should check if str is null (after line 963) and not pass it to nsCRT::strtok if it is. This should be a safe fix which will prevent needless crashes.
->brade per hyatt, who sez this is probably only triggered by bad XUL, not going to occur in product, but "go for it, I'll sr= it"
Assignee: hyatt → brade
putting in 9.2
Keywords: crash
Priority: -- → P2
Target Milestone: --- → mozilla0.9.2
moving to 1.0 -- see Trudelle comment
Target Milestone: mozilla0.9.2 → mozilla1.0
Status: NEW → ASSIGNED
Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1 (you can query for this string to delete spam or retrieve the list of bugs I've moved)
Target Milestone: mozilla1.0 → mozilla1.0.1
I thought I could make this easy bulletproof fix, but when I looked at the file, I noticed that the function does not exist anymore. Marking as invalid.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → INVALID
Alrighty, then.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.