Memory leak of 4 bytes from 1 block allocated in PR_Malloc

VERIFIED FIXED

Status

--
minor
VERIFIED FIXED
17 years ago
14 years ago

People

(Reporter: stephend, Assigned: naving)

Tracking

({memory-leak})

Trunk
x86
Windows 2000
memory-leak

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

652 bytes, patch
Bienvenu
: review+
Details | Diff | Splinter Review
Build ID: Even with Navin's fix for bug 111371, creating a new local folder in 
mail, I see:

[W] MLK: Memory leak of 4 bytes from 1 block allocated in PR_Malloc
        Distribution of leaked blocks
                 4 bytes from 1 block of 4 bytes (0x0908dbf8) 
        Allocation location
            malloc         [msvcrt.DLL]
            PR_Malloc      [prmem.c:50]
            nsMemory::Alloc(UINT) [nsMemoryImpl.cpp:556]
            nsMsgFolderDataSource::DoNewFolder(nsIMsgFolder *,nsISupportsArray 
*) [nsMsgFolderDataSource.cpp:1963]
            nsMsgFolderDataSource::DoCommand(nsISupportsArray *,nsIRDFResource 
*,nsISupportsArray *) [nsMsgFolderDataSource.cpp:743]
            CompositeDataSourceImpl::DoCommand(nsISupportsArray 
*,nsIRDFResource *,nsISupportsArray *) [nsCompositeDataSource.cpp:1434]
            XPTC_InvokeByIndex [xptcinvoke.cpp:152]
            XPCWrappedNative::CallMethod
(XPCCallContext&,CallMode::XPCWrappedNative) [xpcwrappednative.cpp:2009]
            XPC_WN_CallMethod(JSContext *,JSObject *,UINT,long *,long *) 
[xpcwrappednativejsops.cpp:1266]
            js_Invoke      [jsinterp.c:832]
            js_Interpret   [jsinterp.c:2791]
            js_Invoke      [jsinterp.c:849]
            js_InternalInvoke [jsinterp.c:924]
            JS_CallFunctionValue [jsapi.c:3405]
            nsJSContext::CallEventHandler(void *,void *,UINT,void *,int *,int) 
[nsJSEnvironment.cpp:988]
            nsJSEventListener::HandleEvent(nsIDOMEvent *) 
[nsJSEventListener.cpp:180]
            nsEventListenerManager::HandleEventSubType(nsListenerStruct 
*,nsIDOMEvent *,nsIDOMEventTarget *,UINT,UINT) [nsEventListenerManager.cpp:1213]
            nsEventListenerManager::HandleEvent(nsIPresContext *,nsEvent 
*,nsIDOMEvent * *,nsIDOMEventTarget *,UINT,nsEventStatus *) 
[nsEventListenerManager.cpp:2203]
            nsXULElement::HandleDOMEvent(nsIPresContext *,nsEvent *,nsIDOMEvent 
* *,UINT,nsEventStatus *) [nsXULElement.cpp:3401]
            PresShell::HandleDOMEventWithTarget(nsIContent *,nsEvent 
*,nsEventStatus *) [nsPresShell.cpp:5903]
Keywords: mlk
QA Contact: esther → stephend
(Assignee)

Comment 1

17 years ago
that was a leak in local folder. this is something not related to that one.
(Assignee)

Comment 2

17 years ago
Created attachment 59135 [details] [diff] [review]
proposed fix

use nsXPIDLString
(Assignee)

Comment 3

17 years ago
Created attachment 59136 [details] [diff] [review]
proposed fix

use nsXPIDLString
Attachment #59135 - Attachment is obsolete: true
(Assignee)

Comment 4

17 years ago
cc bienvenu for review.

Comment 5

17 years ago
Comment on attachment 59136 [details] [diff] [review]
proposed fix

r=bienvenu
Attachment #59136 - Flags: review+
(Assignee)

Comment 6

17 years ago
fixed.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED
excellent.  you guys are rockin'.

verified fixed.
Status: RESOLVED → VERIFIED
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.