Closed Bug 7217 Opened 25 years ago Closed 25 years ago

MLK: nsHTMLEditor leaks rules

Categories

(Core :: DOM: Editor, defect, P3)

Product:
Core
Component:
DOM: Editor
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: bruce, Assigned: Brade)

References

Details

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]
Assignee: kostello → buster
changing ownership to buster as the new lead for the editor module
Assignee: buster → brade
Depends on: 5806
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
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?
Target Milestone: M8
M8
need help verifying this bug...
Next week with luck I will be able to verify bugs if no one else has verified it
by then.
we need to get this bug off the verification radar...BRuce, any spare time
this week?
changing qa_contact to bruce@cybersight.com
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.