browser hangs up forever on restart if seperator in bookmak manager contains double quote characters

RESOLVED FIXED in seamonkey2.0

Status

--
critical
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: tester, Assigned: neil)

Tracking

({fixed-seamonkey2.0})

Trunk
seamonkey2.0
x86
Windows XP
fixed-seamonkey2.0
Bug Flags:
blocking-seamonkey2.0 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.16) Gecko/20080702 SeaMonkey/1.1.11
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.16) Gecko/20080702 SeaMonkey/1.1.11

I am in Bookmark manager. If I use the double quote character in the rename a of
Seperator, the program crashes when I run the Seamonkey browser.

Reproducible: Always

Steps to Reproduce:
1.Open bookmark manager
2.Click new seperator
3.Click rename to add text to the seperator line.
4.Do it with the Rename text containing ""
5.Close SeaMonkey
Actual Results:  
1.Start SeaMonkey browser again. Program hangs forever

Expected Results:  
Progarm to run properly

Properly encode or drop the quote character (or any other character which may affect the HTML interpreter).
Version: unspecified → SeaMonkey 1.1 Branch

Comment 1

10 years ago
Mozilla/5.0 (Windows; U; Windows NT 5.1; cs; rv:1.9.0.1pre) Gecko/2008062408 SeaMonkey/2.0a1pre
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.16) Gecko/20080702 SeaMonkey/1.1.11

Hangs confirming, on trunk and 1.1.11 versions.

Firefox does not contain rename.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: browser hangs up forever on restart if some items in bookmak manager are edited (rename) → browser hangs up forever on restart if seperator in bookmak manager contains double quote characters
Version: SeaMonkey 1.1 Branch → Trunk

Updated

10 years ago
Flags: blocking-seamonkey2?

Updated

9 years ago
Flags: blocking-seamonkey2.0? → blocking-seamonkey2.0+
(Assignee)

Comment 2

9 years ago
Created attachment 399183 [details] [diff] [review]
Proposed patch

Escaped the keyword too, just in case.
Assignee: nobody → neil
Status: NEW → ASSIGNED
Attachment #399183 - Flags: review?(iann_bugzilla)

Comment 3

9 years ago
Comment on attachment 399183 [details] [diff] [review]
Proposed patch

>--- a/suite/browser/src/nsBookmarksService.cpp	Mon Sep 07 14:27:14 2009 -0700
>+                if (property == kNC_Name || property == kNC_ShortcutURL)
>+                {
>+                    EscapeHTML(attribute);
>+                }
Do we need the braces?
Attachment #399183 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 4

9 years ago
Pushed changeset 79163e3ee03f to comm-central.
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED

Updated

9 years ago
Attachment #399183 - Flags: approval-seamonkey2.0+

Updated

9 years ago
Keywords: fixed-seamonkey2.0
Target Milestone: --- → seamonkey2.0
You need to log in before you can comment on or make changes to this bug.