Closed
Bug 5637
Opened 25 years ago
Closed 25 years ago
releasing RDF observers in destructor
Categories
(Core Graveyard :: RDF, defect, P3)
Tracking
(Not tracked)
VERIFIED
FIXED
M5
People
(Reporter: jevering, Assigned: waterson)
Details
(Whiteboard: fix in hand, awaiting code review)
Attachments
(1 file)
|
(deleted),
patch
|
Details | Diff | Splinter Review |
Go to yahoo.com
open sidebar
open related links
select one
after the page loads, apprunner crashes.
It appears that it is attempting access an already freed element.
Stack trace:
nsXULTreeElement::GetDatabase(nsXULTreeElement * const 0x03af6418,
nsIRDFCompositeDataSource * * 0x0088f99c) line 35 + 24 bytes
RDFTreeBuilderImpl::CheckRDFGraphForUpdates(nsIContent * 0x03b08cc0)
line 456 + 62 bytes
RDFTreeBuilderImpl::UpdateContainer(nsIContent * 0x03afa1e0) line 406
RDFTreeBuilderImpl::Notify(nsITimer * 0x03f70e50) line 669
TimerImpl::Fire(unsigned long 14596501) line 313
TimerImpl::ProcessTimeouts(unsigned long 14596501) line 187
FireTimeout(HWND__ * 0x00000000, unsigned int 275, unsigned int 260,
unsigned long 14596501) line 101 + 9 bytes
KERNEL32! bff72317()
RAPTORBASE!
|
Assignee | |
Updated•25 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M5
|
|
Assignee | |
Comment 1•25 years ago
|
||
|
|
Assignee | |
Updated•25 years ago
|
Summary: Selecting a related link from sidebar crashes apprunner → releasing RDF observers in destructor
Whiteboard: fix in hand, awaiting code review
|
|
Assignee | |
Comment 2•25 years ago
|
||
Fix in hand: we were NS_RELEASE-ing RDF observers that we didn't own. A
datasource only holds a weak reference to the observers that are
observing it.
Waiting for rjc and putterman to review diffs, which are attached above.
|
||
Comment 3•25 years ago
|
||
Diffs look good to me.
|
||
Comment 4•25 years ago
|
||
It looks good to me too.
|
|
Assignee | |
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
|
||
Updated•6 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•