Memory leak of 41 bytes from 1 block allocated in nsAbCardProperty::SetMailListURI

VERIFIED FIXED

Status

SeaMonkey
MailNews: Address Book & Contacts
VERIFIED FIXED
16 years ago
13 years ago

People

(Reporter: stephend@netscape.com (gone - use stephen.donner@gmail.com instead), Assigned: Bienvenu)

Tracking

({mlk})

Trunk
x86
Windows 2000

Firefox Tracking Flags

(Not tracked)

Details

MLK: Memory leak of 41 bytes from 1 block allocated in PL_strdup
        Distribution of leaked blocks
                41 bytes from 1 block of 41 bytes (0x0ecf9e28) 
        Allocation location
            malloc+0xc           [dbgheap.c:129 ip=0x002e3c3c]
            PL_strdup+0x75       [.\../../../../lib/libc/src/strdup.c:46
ip=0x004c15a5]
            nsCRT::strdup(char const*)+0x24 [.\..\..\dist\include\nsCRT.h:167
ip=0x1000e8c4]
            nsAbCardProperty::SetMailListURI(char const*)+0x81
[c:\moz_src\mozilla\mailnews\addrbook\src\nsAbCardProperty.cpp:144 ip=0x0fb89591]
            nsAddrDatabase::CreateABListCard(nsIMdbRow *,nsIAbCard * *)+0x6f7
[c:\moz_src\mozilla\mailnews\addrbook\src\nsAddrDatabase.cpp:3792 ip=0x0fbcd017]
            nsAddrDBEnumerator::CurrentItem(nsISupports * *)+0xe4
[c:\moz_src\mozilla\mailnews\addrbook\src\nsAddrDatabase.cpp:3484 ip=0x0fbca5c4]
            nsAbAutoCompleteSession::SearchCards(nsIAbDirectory
*,nsAbAutoCompleteSearchString *,nsIAutoCompleteResults *)+0x2d3
[c:\moz_src\mozilla\mailnews\addrbook\src\nsAbAutoCompleteSession.cpp:355
ip=0x0fbd9e73]
           
nsAbAutoCompleteSession::SearchDirectory(nsString&,nsAbAutoCompleteSearchString
*,nsIAutoCompleteResults *,int)+0x644
[c:\moz_src\mozilla\mailnews\addrbook\src\nsAbAutoCompleteSession.cpp:456
ip=0x0fbdba24]
           
nsAbAutoCompleteSession::SearchDirectory(nsString&,nsAbAutoCompleteSearchString
*,nsIAutoCompleteResults *,int)+0xc31
[c:\moz_src\mozilla\mailnews\addrbook\src\nsAbAutoCompleteSession.cpp:481
ip=0x0fbdc011]
            nsAbAutoCompleteSession::OnStartLookup(WORD
const*,nsIAutoCompleteResults *,nsIAutoCompleteListener *)+0x6c6
[c:\moz_src\mozilla\mailnews\addrbook\src\nsAbAutoCompleteSession.cpp:598
ip=0x0fbdd586]
Keywords: mlk
changing qa over to myself.
QA Contact: nbaca → stephend
(Assignee)

Comment 2

16 years ago
taking.
Assignee: chuang → bienvenu
(Assignee)

Comment 3

16 years ago
fixed, here's patch:

C:\six5\mozilla\mailnews\addrbook\src>cvs diff nsAbCardProperty.cpp
Index: nsAbCardProperty.cpp
===================================================================
RCS file: /cvsroot/mozilla/mailnews/addrbook/src/nsAbCardProperty.cpp,v
retrieving revision 1.44
diff -r1.44 nsAbCardProperty.cpp
72a73
>   PR_FREEIF(m_MailListURI);
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
Verified FIXED using the latest tip build on Windows 2000.

I ran mozilla.exe -addressbook, scrolled and opened the new card window.
Status: RESOLVED → VERIFIED
Wait, I need to verify that using -compose.  I'll re-verify it. 
This is fixed, using -compose, too.
Product: Browser → Seamonkey
You need to log in before you can comment on or make changes to this bug.