187762 - Crash when using erroneous bookmarks.html file (PT Folder contains PT Folder)

Status: VERIFIED DUPLICATE of bug 158428

(SeaMonkey :: Bookmarks & History, defect)

Description

[Darren Stone]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.3b) Gecko/20030102
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.3b) Gecko/20030102

When the Personal Toolbar (PT) Folder contains a child folder that is also
marked as a PT Folder, the client crashes when attempting to open the Bookmark
Manager.  A simple testcase is attached.

Reproducible: Always

Steps to Reproduce:
1. Use attached bookmark.html file as your bookmarks file
2. Open Browser
3. Open Bookmark Manager

Actual Results:  
Crash.

Expected Results:  
The contents of bookmarks.html shouldn't cause the client to crash or prevent
the bookmark manager from operating.

Another (so far un-reproducible) bug from a few weeks ago caused the child PT
Folder to be added to the main PT Folder in the first place -- I watched the new
folder spontaneously appear on my Personal Toolbar as I moused over it.  Unable
to delete the new folder or use the Bookmark manager, I ended up editing
bookmarks.html by hand.  That's when I discovered the cause of the Bookmark
Manager crash.

Comment 1

[Darren Stone]

Attachment: Bookmarks.html Testcase

Comment 2

[Olivier Cahagne]

can you post Talkback ID for this crash "mozilla/bin/components/talkback.exe" ?

Comment 3

[Olivier Cahagne]

confirming using build 20030104 on Linux, no Talkback popped up when crashing,
will attach stacktrace with GDB.

Comment 4

[Olivier Cahagne]

I get this assertion using build 20020105 (CVS) on Linux in an infinite loop, I
don't "crash" with a stack after 5 minutes of this warning in console:

###!!! ASSERTION: overflow: 'Error', file nsTreeRows.cpp, line 353
Break: at file nsTreeRows.cpp, line 353
###!!! ASSERTION: overflow: 'Error', file nsTreeRows.cpp, line 368
Break: at file nsTreeRows.cpp, line 368

see bug 158428 comment 13.

*** This bug has been marked as a duplicate of 158428 ***

Comment 5

[timeless]

vrfy dupe