292 URLName = (char*)PR_Malloc(PL_strlen(spec)+1); 293 PL_strcpy(URLName, spec); 1) that isn't freed up. 2) What's wrong with URLName = nsCRT::strdup(spec); 3) Don't need this at all really in a necko world since the URL accessors allocate the memory for you, so you own it. [W] MLK: Memory leak of 90 bytes from 1 block allocated in PR_Malloc Distribution of leaked blocks 90 bytes from 1 block of 90 bytes (0x09637278) allocation number 164080 Allocation location malloc [dbgheap.c:129] PR_Malloc [prmem.c:38] nsWalletlibService::OnEndDocumentLoad(nsIDocumentLoader *,nsIChannel *,UINT,nsIDocumentLoaderObserver *) [nsWalletService.cpp:292] nsDocLoaderImpl::FireOnEndDocumentLoad(nsDocLoaderImpl *,UINT) [nsDocLoader.cpp:1228] nsDocLoaderImpl::FireOnEndDocumentLoad(nsDocLoaderImpl *,UINT) [nsDocLoader.cpp:1235] nsDocLoaderImpl::FireOnEndDocumentLoad(nsDocLoaderImpl *,UINT) [nsDocLoader.cpp:1235] nsDocLoaderImpl::OnStopRequest(nsIChannel *,nsISupports *,UINT,WORD const*) [nsDocLoader.cpp:1098] nsLoadGroup::RemoveChannel(nsIChannel *,nsISupports *,UINT,WORD const*) [nsLoadGroup.cpp:560] nsFileChannel::OnStopRequest(nsIChannel *,nsISupports *,UINT,WORD const*) [nsFileChannel.cpp:827] nsOnStopRequestEvent::HandleEvent(void) [nsAsyncStreamListener.cpp:268] nsStreamListenerEvent::HandlePLEvent(PLEvent *) [nsAsyncStreamListener.cpp:144] PL_HandleEvent [plevent.c:509] PL_ProcessPendingEvents [plevent.c:470] md_EventReceiverProc [plevent.c:938] TranslateMessageEx [user32.dll]