106857 - Memory Leak of 90 bytes in nsLDAPAutoCompleteSession::OnLDAPMessage

Status: RESOLVED INVALID

(MailNews Core :: LDAP Integration, defect)

Description

[[e-mail] (gone - use [e-mail] instead)]

While verifying bug 103085, I found the foll
owing leak: 

Trunk CVS pull @8pm (tonight), Windows 2000.

Using the autocomplete setup provided by [e-mail].

dmose had asked a long time ago if I could run this, sorry it took so long.

Steps to Reproduce:

1.  Mozilla.exe -compose.
2.  Type dmos (it'll complete to Dan Mosedale <[e-mail]>).
3.  Tab to subject.
4.  Exit.

[W] MLK: Memory leak of 90 bytes from 3 blocks allocated in ber_init
    Distribution of leaked blocks
    Allocation location
    malloc+0xc           [C:\WINNT\SYSTEM32\msvcrt.DLL ip=0x580013eb]
    ber_init+0x64        
[C:\moz_src\mozilla\dist\WIN32_O.OBJ\bin\NSLDAP32V40.dll ip=0x0d7ea574]
    ???            [ip=0x08ec0c40]
    nsLDAPAutoCompleteSession::OnLDAPMessage(nsILDAPMessage *)+0x145 
[c:\moz_src\mozilla\xpfe\components\autocomplete\src\nsLDAPAutoCompleteSession.c
pp:392 ip=0x089899df]
    
            // a search entry has been returned
            //
 =>         return OnLDAPSearchEntry(aMessage);
    
        case nsILDAPMessage::RES_SEARCH_RESULT:
    
    XPTC_InvokeByIndex+0x75 
[c:\moz_src\mozilla\xpcom\reflect\xptcall\src\md\win32\xptcinvoke.cpp:152 
ip=0x100d36e2]
            mov     eax,methodIndex
            shl     eax,2               // *= 4
            add     edx,eax
 =>         call    [edx]               // stdcall, i.e. callee cleans up stack.
        }
    }
    #pragma warning(default : 4035) // restore default
    EventHandler+0xb4    
[c:\moz_src\mozilla\xpcom\proxy\src\nsProxyEvent.cpp:514 ip=0x100e5df8]
            nsresult rv = XPTC_InvokeByIndex( proxyObject->GetRealObject(),
                                              info->GetMethodIndex(),
                                              info->GetParameterCount(),
 =>                                           info->GetParameterList());
            info->SetResult(rv);
        }
        else
    md_EventReceiverProc+0x61 [c:\moz_src\mozilla\xpcom\threads\plevent.c:1071 
ip=0x100cc829]
        {
            PREventQueue *queue = (PREventQueue *)lParam;
            queue->removeMsg = PR_FALSE;
 =>         PL_ProcessPendingEvents(queue);
            queue->removeMsg = PR_TRUE;
        #ifdef XP_OS2
                return MRFROMLONG(TRUE);
    ScrollDC+0x490       [C:\WINNT\SYSTEM32\user32.dll ip=0x77e12e98]
    ScrollDC+0x6d8       [C:\WINNT\SYSTEM32\user32.dll ip=0x77e130e0]
    DispatchMessageA+0xb [C:\WINNT\SYSTEM32\user32.dll ip=0x77e15824]
    DispatchMessageA+0xb [C:\WINNT\SYSTEM32\USER32.DLL ip=0x67e125a2]
    nsAppShell::Run(void)+0x37c 
[c:\moz_src\mozilla\widget\src\windows\nsAppShell.cpp:121 ip=0x04cf3aab]
        //      if (!nsToolkit::gAIMMMsgPumpOwner || 
(nsToolkit::gAIMMMsgPumpOwner->OnTranslateMessage(&msg) != S_OK))
        //#endif
                TranslateMessage(&msg);
     =>         ::DispatchMessage(&msg);
                if (mDispatchListener)
                  mDispatchListener->AfterDispatch();
              }
    nsAppShellService::Run(void)+0x3f 
[c:\moz_src\mozilla\xpfe\appshell\src\nsAppShellService.cpp:301 ip=0x04622e05]
        
        NS_IMETHODIMP
        nsAppShellService::Run(void)
     => {
          return mAppShell->Run();
        }
        
    main1+0xbe1          
[c:\moz_src\mozilla\xpfe\bootstrap\nsAppRunner.cpp:1285 ip=0x004044ed]
    main+0x258           
[c:\moz_src\mozilla\xpfe\bootstrap\nsAppRunner.cpp:1607 ip=0x004035d1]
    WinMain+0x78         
[c:\moz_src\mozilla\xpfe\bootstrap\nsAppRunner.cpp:1625 ip=0x0040a020]

Comment 1

[Srilatha Moturi]

over to dmose

Comment 2

[Dan Mosedale (:dmosedale, :dmose)]

Early Christmas present for leif.

Comment 3

[[e-mail] (gone - use [e-mail] instead)]

Dan, should this go back to you now?

Comment 4

[Dan Mosedale (:dmosedale, :dmose)]

Yeah.  Taking.

Comment 5

[Dan Mosedale (:dmosedale, :dmose)]

Mass change: pushing out; as I'm concentrating on feature work at the moment.

Comment 6

[David :Bienvenu]

taking - I've been working on fixes for some ldap memory leaks that cause
connections not to close, and this might be related.

Comment 7

[David :Bienvenu]

I'm not planning on looking at this anytime soon - I don't know if this leak still exists or not...

Comment 8

[Wayne Mery (:wsmwk)]

ldap has been reimplemented