65058 - NamedMap::remove hash horkage with BucketItem::prev

Status: VERIFIED FIXED

(Core :: XSLT, defect)

Description

[Markus Hübner]

When going to the url
http://www.world-direct.com/gia/test
with the build 2001011020 Mozilla crashes.

Comment 1

[Fabian Guisset]

<?xml version='1.0' encoding='iso-8859-1'?><?xml-stylesheet version='1.0'
type='text/xsl' href='GolfClubsOverview.xsl'?><sal_output version="1.0">
First line is this. Doesn't crash for me in 0.7 and 2001010904 win32.

Comment 2

[Markus Hübner]

got the info on #transformiix that in xsl-sorting it releases an nsString 
that's null.

Comment 3

[Doron Rosenberg (IBM)]

not crashing cvs pull from today...

Comment 4

[Matthias Versen [:Matti]]

Does not crash for me on win2k/ build 2001010620

Comment 5

[Keyser Sose]

WORKSFORME
Platform: PC
OS: Windows 98
Mozilla Build: 2001010904

Marking WORKSFORME.

Reopen if this reoccurs in the latest nightlies.

Comment 6

[Markus Hübner]

Just tried build 20010111 and it still crashes.

Comment 7

[Peter Van der Beken]

Just to clear things up, what the original bug report fails to mention is that
you need to have the Transformiix XSLT module installed for the crash. If the
module is not installed, the XML file just loads and there's no crash. Markus,
please include more details in your bug reports.

Comment 8

[Markus Hübner]

Sorry about that ... when I go to the page Mozilla just closes.
No error message or crash info at all!

Comment 9

[Axel Hecht]

I don't think we can deal with this with the wrappers as they are, marking 
a dependency.
And of course, it's more a Peter issue than Keith, so re-assigning.
Markus, that's just who would work on it, not that work started. Then the bug
will change to ASSIGNED.

Axel

Comment 10

[Axel Hecht]

find the reason for this crasher. When removing a BucketItem from the NamedMap,
we didn't care about prev.
Patch coming in a minute, checkin tomorrow.

Axel
... updating alot of components

Comment 11

[Axel Hecht]

Attachment: fixing NamedMap, securing MozillaString wrapper

Comment 12

[Axel Hecht]

checked in, r=peterv

Comment 13

[Markus Hübner]

Thank you Axel & Peter. You did a great job :)

Comment 14

[Peter Van der Beken [:peterv]]

Verified