Closed
Bug 88736
Opened 24 years ago
Closed 24 years ago
FilterExpr needs to check result type before casting.
Categories
(Core :: XSLT, defect)
Core
XSLT
Tracking
()
VERIFIED
FIXED
mozilla1.0
People
(Reporter: sicking, Assigned: sicking)
Details
Attachments
(2 files)
|
3.24 KB,
patch
|
Details | Diff | Splinter Review | |
|
3.23 KB,
patch
|
Details | Diff | Splinter Review |
FilterExpr::evaluate and FilterExpr::matches needs to check if the ExprResult
returned from expr is a nodeset before casting. Otherwise we'll crash on
expressions like "9[false()]"
Patch coming up...
(I've snuck in a small bugfix aswell, the nodeset needs to be sorted before
it's filtered, see axes14 test)
|
Assignee | |
Comment 1•24 years ago
|
||
|
|
Assignee | |
Comment 2•24 years ago
|
||
|
||
Comment 3•24 years ago
|
||
r=peterv if you accept the bug and set the milestone and the priority.
|
|
Assignee | |
Updated•24 years ago
|
Status: NEW → ASSIGNED
Whiteboard: have r=peterv, need sr=
Target Milestone: --- → mozilla1.0
|
||
Comment 4•24 years ago
|
||
sr=jst
|
|
Assignee | |
Comment 5•24 years ago
|
||
checked in
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Whiteboard: have r=peterv, need sr=
|
|
Assignee | |
Comment 6•24 years ago
|
||
We don't crash any more on stuff like "(2)[1]"
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•