Build from May 26, 1999. Solaris 2.6. Start and immediately exit apprunner -editor and this leaks. Multiple traces follow. MLK: 12 bytes leaked at 0x96d2a8 * This memory was allocated from: malloc [rtlib.o] __bUiLtIn_nEw [libgcc.a] __builtin_new [rtlib.o] NS_NewEditorKeyListener(nsIDOMEventListener**,nsITextEditor*) [nsEditorEventListeners.cpp:1241] nsTextEditor::Init(nsIDOMDocument*,nsIPresShell*) [nsTextEditor.cpp:255] 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] MLK: 12 bytes leaked at 0x96d370 * This memory was allocated from: malloc [rtlib.o] __bUiLtIn_nEw [libgcc.a] __builtin_new [rtlib.o] NS_NewEditorMouseListener(nsIDOMEventListener**,nsITextEditor*) [nsEditorEventListeners.cpp:1259] nsTextEditor::Init(nsIDOMDocument*,nsIPresShell*) [nsTextEditor.cpp:259] 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] MLK: 12 bytes leaked at 0x96d508 * This memory was allocated from: malloc [rtlib.o] __bUiLtIn_nEw [libgcc.a] __builtin_new [rtlib.o] NS_NewEditorCompositionListener(nsIDOMEventListener**,nsITextEditor*) [nsEditorEventListeners.cpp:1307] nsTextEditor::Init(nsIDOMDocument*,nsIPresShell*) [nsTextEditor.cpp:280] 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] MLK: 12 bytes leaked at 0x96d5d0 * This memory was allocated from: malloc [rtlib.o] __bUiLtIn_nEw [libgcc.a] __builtin_new [rtlib.o] NS_NewEditorDragListener(nsIDOMEventListener**,nsITextEditor*) [nsEditorEventListeners.cpp:1292] nsTextEditor::Init(nsIDOMDocument*,nsIPresShell*) [nsTextEditor.cpp:292] 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]

changing ownership to buster as the new lead for the editor module

this should get fixed soon, set milestone for M8 to match up with XP IDL and related ownership changes on the schedule.

I think sfraser's fixes, fixed these as well (the destructor for nsTextEditor is now being called and I can step to see that the lines like the following are being called: erP->RemoveEventListenerByIID(mDragListenerP, nsIDOMDragListener::GetIID()) resolving as fixed. Bruce--can you verify this?

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

changing qa_contact to bruce@cybersight.com

verified in todays build.