106858 - Memory leak of 8 bytes in nsMemory::Alloc

Status: RESOLVED INVALID

(MailNews Core :: LDAP Integration, defect, P3)

Description

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

While verifying bug 103085, I found the following leak: 

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

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

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 8 bytes from 2 blocks allocated in PR_Malloc
    Distribution of leaked blocks
    Allocation location
    malloc+0xc           [C:\WINNT\SYSTEM32\msvcrt.DLL ip=0x580013eb]
    PR_Malloc+0x2c       [.\../../../../pr/src/malloc/prmem.c:50 ip=0x30018c5c]
    nsMemory::Alloc(UINT)+0x9a 
[c:\moz_src\mozilla\xpcom\base\nsMemoryImpl.cpp:556 ip=0x1003bba1]
        if (gMemory == nsnull) {
            EnsureGlobalMemoryService();
        }
 =>     return gMemory->Alloc(size);
    }
    
    NS_EXPORT void*
    nsAbLDAPAutoCompFormatter::GetAttributes(UINT *,char * * *)+0x204 
[c:\moz_src\mozilla\mailnews\addrbook\src\nsAbLDAPAutoCompFormatter.cpp:414 
ip=0x0d45e5b2]
        //
        while (rawSearchAttrsSize < count) {
            if (!(rawSearchAttrs[rawSearchAttrsSize] =
 =>               ToNewCString(*(mSearchAttrs.CStringAt(rawSearchAttrsSize))))) 
{
                NS_FREE_XPCOM_ALLOCATED_POINTER_ARRAY(rawSearchAttrsSize,
                                                      rawSearchAttrs);
                    NS_ERROR("nsAbLDAPAutoCompFormatter::GetAttributes(): out "
    nsLDAPAutoCompleteSession::SetFormatter(nsILDAPAutoCompFormatter *)+0xb1 
[c:\moz_src\mozilla\xpfe\components\autocomplete\src\nsLDAPAutoCompleteSession.c
pp:1368 ip=0x0898c797]
        
            // get and cache the attributes that will be used to do lookups
            //
     =>     nsresult rv = mFormatter->GetAttributes(&mSearchAttrsSize, 
&mSearchAttrs);
            if (NS_FAILED(rv)) {
                NS_ERROR("nsLDAPAutoCompleteSession::SetFormatter(): "
                         " mFormatter->GetAttributes failed");
    XPTC_InvokeByIndex+0x75 
[c:\moz_src\mozilla\xpcom\reflect\xptcall\src\md\win32\xptcinvoke.cpp:152 
ip=0x100d36e2]
    XPCWrappedNative::CallMethod(XPCCallContext&,CallMode::XPCWrappedNative)
+0x10bf [c:\moz_src\mozilla\js\src\xpconnect\src\xpcwrappednative.cpp:1951 
ip=0x0470a122]
    XPC_WN_GetterSetter(JSContext *,JSObject *,UINT,long *,long *)+0x1b0 
[c:\moz_src\mozilla\js\src\xpconnect\src\xpcwrappednativejsops.cpp:1290 
ip=0x04714ca2]
    js_Invoke+0xba4      [c:\moz_src\mozilla\js\src\jsinterp.c:809 
ip=0x0442cacc]
    js_InternalInvoke+0x1a8 [c:\moz_src\mozilla\js\src\jsinterp.c:901 
ip=0x0442d32b]
    js_SetProperty+0x10e4 [c:\moz_src\mozilla\js\src\jsobj.c:2681 ip=0x04452537]
    js_Interpret+0x9bad  [c:\moz_src\mozilla\js\src\jsinterp.c:2611 
ip=0x0443799e]
    js_Invoke+0xc57      [c:\moz_src\mozilla\js\src\jsinterp.c:826 
ip=0x0442cb7f]
    js_InternalInvoke+0x1a8 [c:\moz_src\mozilla\js\src\jsinterp.c:901 
ip=0x0442d32b]
    JS_CallFunctionValue+0x42 [c:\moz_src\mozilla\js\src\jsapi.c:3410 
ip=0x043ec41a]
    nsJSContext::CallEventHandler(void *,void *,UINT,void *,int *,int)+0x2ad 
[c:\moz_src\mozilla\dom\src\base\nsJSEnvironment.cpp:977 ip=0x05b4b132]
    nsJSEventListener::HandleEvent(nsIDOMEvent *)+0x64e 
[c:\moz_src\mozilla\dom\src\events\nsJSEventListener.cpp:155 ip=0x05b97daa]
    nsEventListenerManager::HandleEventSubType(nsListenerStruct *,nsIDOMEvent 
*,nsIDOMEventTarget *,UINT,UINT)+0x870 
[c:\moz_src\mozilla\content\events\src\nsEventListenerManager.cpp:1213 
ip=0x056ae22b]
    nsEventListenerManager::HandleEvent(nsIPresContext *,nsEvent *,nsIDOMEvent 
* *,nsIDOMEventTarget *,UINT,nsEventStatus *)+0x282d 
[c:\moz_src\mozilla\content\events\src\nsEventListenerManager.cpp:1814 
ip=0x056b0b7c]
    nsXULElement::HandleDOMEvent(nsIPresContext *,nsEvent *,nsIDOMEvent * 
*,UINT,nsEventStatus *)+0x144d 
[c:\moz_src\mozilla\content\xul\content\src\nsXULElement.cpp:3387 ip=0x055e309f]

Comment 1

[Srilatha Moturi]

over to dmose

Comment 2

[Dan Mosedale (:dmosedale, :dmose)]

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

Comment 3

[Wayne Mery (:wsmwk)]

how does one test this?

Comment 4

[Stephen Donner [:stephend] Not actively reading bugmail]

(In reply to comment #3)
> how does one test this?

One has IBM (Rational) Purify, and runs through the steps in comment 0... 

Comment 5

[Rich Megginson]

(In reply to comment #3)
> how does one test this?
> 

valgrind on linux is quite nice too for catching heap memory usage errors and leaks.

Comment 6

[Wayne Mery (:wsmwk)]

ldap has been reimplemented