MLK: nsHTMLEditor leaks rules

VERIFIED FIXED in M8

Status

()

Core
Editor
P3
normal
VERIFIED FIXED
19 years ago
17 years ago

People

(Reporter: Bruce Mitchener, Assigned: Brade)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

19 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

19 years ago
Assignee: kostello → buster

Comment 1

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

Updated

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

Updated

19 years ago
Status: NEW → ASSIGNED
(Assignee)

Updated

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

Comment 2

19 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

19 years ago
Target Milestone: M8
(Assignee)

Comment 3

19 years ago
M8

Comment 4

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

Comment 5

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

Comment 6

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

Comment 7

19 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.