Closed Bug 216482 Opened 21 years ago Closed 21 years ago

Firebird Crashes when picked "View -> By Date and Site" or View-> By Site" in history sidebar

Categories

(Core :: XUL, defect)

x86
All
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla1.5beta

People

(Reporter: geoff, Assigned: neil)

References

Details

(Keywords: crash, fixed1.5, regression)

Attachments

(2 files)

User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5b) Gecko/20030817 Mozilla Firebird/0.6.1+ Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5b) Gecko/20030817 Mozilla Firebird/0.6.1+ This has only appeared in todays nightly 17th August, when I search for a history item, then try and go back to the full history list using "View -> By Site" etc.. Firebird crashes with no error, only the netscape feedback window thing. Reproducible: Always Steps to Reproduce: 1. Open Firebird 2. Search for a history item 3. pick "View -> By Date & Site" Actual Results: Firebird disappeared (crashed) Netscape Quality Feeback window appeared Expected Results: show all History items
FYI: new install / clean profile running default theme /w no Extensions, still crashes every time.
Confirming on Windows 2000, build 20030817. Talkback ID TB22829351E. I couldn't get my previous build (20030816) to crash, though.
Status: UNCONFIRMED → NEW
Ever confirmed: true
FWIW, Firebird manages to redraw the tree under the text box with what appears to be the correct stuff, and then crashes - /before/ redrawing the main content (so when it crashes the half of the menu over the web page is still visible, while the left bit, over the history view, isn't). Another Talkback ID, TB22829613K.
Attached file stack (deleted) —
Blew up with a stack overflow. Can also be reproduced by selecting view by date. WFM Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5b) Gecko/20030802 Mozilla Firebird/0.6.1, so it is a fairly recent introduction.
Depends on: 197486
This crash happens whatever option you select in the dropdown box - not just "By Date or Site" or "By Site". Should this be targeted for 0.7 as this is likely to be a common crash and results in data loss (whatever pages you are looking at)?
Attached patch Proposed patch (deleted) — Splinter Review
Attachment #130053 - Flags: superreview?(jag)
Attachment #130053 - Flags: review?(varga)
Your patch works for me. For some reason, the results list is scrolled down beyond the end of actual results, but that was the behavior in my older build too.
Comment on attachment 130053 [details] [diff] [review] Proposed patch Hmm, I thought you had already removed this? Anyway, sr=jag
Attachment #130053 - Flags: superreview?(jag) → superreview+
Verifying for Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.5b) Gecko/20030820 Mozilla Firebird/0.6.1+
The same thing happens on Linux i686, with a segmentation fault (MozillaFirebird/run-mozilla.sh: line 454: 822 Segmentation fault "$prog" ${1+"$@"}) ... Either way -- a somewhat unrelated issue, however it is mentioned in this bug and I am reluctant to file a new one in case this is as intended, however shouldn't "By Date and Site" be "By Date and Time"?
*** Bug 216892 has been marked as a duplicate of this bug. ***
Keywords: crash, regression
OS: Windows XP → All
Attachment #130053 - Flags: review?(varga) → review+
Moving to correct component as this is actually an XPFE bug that hit the bird.
Assignee: blake → hyatt
Component: History → XP Toolkit/Widgets: XUL
Product: Firebird → Browser
QA Contact: mozilla → shrir
Target Milestone: --- → mozilla1.5beta
Version: unspecified → Trunk
Comment on attachment 130053 [details] [diff] [review] Proposed patch Need this in trunk to remove potential crasher (problem is more noticable in firebird).
Attachment #130053 - Flags: approval1.5b?
*** Bug 216987 has been marked as a duplicate of this bug. ***
What exactly are you doing? In the future, if you want approval for something urgently, it'd be nice if you described what your code was doing in the bug.
Comment on attachment 130053 [details] [diff] [review] Proposed patch a=mkaply for 1.5b. Get in ASAP For the record, these calls are being removed because they lead to recursive runaway.
Attachment #130053 - Flags: approval1.5b? → approval1.5b+
over to neil, since it is his patch. neil, can you get this in for 1.5 final?
Assignee: hyatt → neil.parkwaycc.co.uk
Flags: blocking1.5+
Status: NEW → ASSIGNED
Fix checked in to the trunk.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Keywords: fixed1.5
Component: XP Toolkit/Widgets: XUL → XUL
QA Contact: shrir → xptoolkit.widgets
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: