MLK: nsHTMLEditor leaks rules

VERIFIED FIXED in M8

Status

()

P3
normal
VERIFIED FIXED
20 years ago
18 years ago

People

(Reporter: bruce, Assigned: Brade)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

20 years ago
Build from May 26, 1999.  Solaris 2.6.  Start and immediately exit apprunner
-editor and this leaks.


MLK: 12 bytes leaked at 0x96d048
  * This memory was allocated from:
        malloc         [rtlib.o]
        __bUiLtIn_nEw  [libgcc.a]
        __builtin_new  [rtlib.o]
        nsHTMLEditor::InitRules() [nsHTMLEditor.cpp:127]
        nsTextEditor::Init(nsIDOMDocument*,nsIPresShell*) [nsTextEditor.cpp:253]
        nsHTMLEditor::Init(nsIDOMDocument*,nsIPresShell*) [nsHTMLEditor.cpp:118]
        nsEditorAppCore::InstantiateEditor(nsIDOMDocument*,nsIPresShell*)
[nsEditorAppCore.cpp:335]
        nsEditorAppCore::DoEditorMode(nsIWebShell*) [nsEditorAppCore.cpp:387]
        nsEditorAppCore::PrepareDocumentForEditing() [nsEditorAppCore.cpp:681]
        nsEditorAppCore::OnEndDocumentLoad(nsIDocumentLoader*,nsIURL*,int)
[nsEditorAppCore.cpp:1987]
        nsWebShell::OnEndDocumentLoad(nsIDocumentLoader*,nsIURL*,int)
[nsWebShell.cpp:2486]
        nsDocLoaderImpl::FireOnEndDocumentLoad(nsIDocumentLoader*,int)
[nsDocLoader.cpp:840]
        nsDocLoaderImpl::LoadURLComplete(nsIURL*,nsISupports*,int)
[nsDocLoader.cpp:1018]
        nsDocumentBindInfo::OnStopBinding(nsIURL*,unsigned int,const unsigned
short*) [nsDocLoader.cpp:1539]
        stub_complete(_NET_StreamClass*) [nsStubContext.cpp:772]
        net_ProcessFile [mkfile.c:1360]
        NET_ProcessNet [mkgeturl.c:3355]
        NET_PollSockets [mkselect.c:298]
        nsNetlibService::NetPollSocketsCallback(nsITimer*,void*)
[nsNetService.cpp:1276]
        TimerImpl::FireTimeout() [nsTimer.cpp:73]
        nsTimerExpired [nsTimer.cpp:189]
        g_timeout_dispatch [gmain.c:1147]
        g_main_dispatch [gmain.c:647]
        g_main_iterate [gmain.c:854]
        g_main_run     [gmain.c:912]
        gtk_main       [gtkmain.c:475]
        nsAppShell::Run() [nsAppShell.cpp:197]
        nsAppShellService::Run() [nsAppShellService.cpp:402]
        main           [nsAppRunner.cpp:482]
        _start         [crt1.o]

Updated

20 years ago
Assignee: kostello → buster

Comment 1

20 years ago
changing ownership to buster as the new lead for the editor module
(Assignee)

Updated

20 years ago
Assignee: buster → brade
Depends on: 5806
(Assignee)

Updated

20 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

20 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 20 years ago
Resolution: --- → FIXED
(Assignee)

Comment 2

20 years ago
mRules are now in nsTextEditor.
Thanks to sfraser's fixes, the rules are now deleted in nsTextEditor's destructor

Bruce, can you verify this bug?
(Assignee)

Updated

20 years ago
Target Milestone: M8
(Assignee)

Comment 3

20 years ago
M8

Comment 4

20 years ago
need help verifying this bug...
(Reporter)

Comment 5

20 years ago
Next week with luck I will be able to verify bugs if no one else has verified it
by then.

Comment 6

20 years ago
we need to get this bug off the verification radar...BRuce, any spare time
this week?

Comment 7

20 years ago
changing qa_contact to bruce@cybersight.com

Comment 8

18 years ago
yeah, it sure seems like this is fixed, mRules is definetely being cleaned up.

anthonyd
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.