From 004de2f422b6827da8984305a07f65fbc832ccad Mon Sep 17 00:00:00 2001 From: "bienvenu%netscape.com" Date: Wed, 20 Oct 1999 20:41:27 +0000 Subject: [PATCH] fix memory leaks in address book r=mscott git-svn-id: svn://10.0.0.236/trunk@51284 18797224-902f-48f8-a5cc-f745e15eee43 --- mozilla/mailnews/addrbook/src/nsAbCardProperty.cpp | 2 +- mozilla/mailnews/addrbook/src/nsCardDataSource.cpp | 1 + mozilla/mailnews/addrbook/src/nsDirectoryDataSource.cpp | 1 + 3 files changed, 3 insertions(+), 1 deletion(-) diff --git a/mozilla/mailnews/addrbook/src/nsAbCardProperty.cpp b/mozilla/mailnews/addrbook/src/nsAbCardProperty.cpp index 2eb118ab657..ad66ff0ed4c 100644 --- a/mozilla/mailnews/addrbook/src/nsAbCardProperty.cpp +++ b/mozilla/mailnews/addrbook/src/nsAbCardProperty.cpp @@ -410,7 +410,7 @@ NS_IMETHODIMP nsAbCardProperty::SetCardValue(const char *attrname, const PRUnich nsAutoString cardValue(value); char* valueStr = cardValue.ToNewCString(); rv = SetAnonymousStringAttribute(attrname, valueStr); - delete[] valueStr; + nsAllocator::Free(valueStr); } return rv; } diff --git a/mozilla/mailnews/addrbook/src/nsCardDataSource.cpp b/mozilla/mailnews/addrbook/src/nsCardDataSource.cpp index 247d18c1ae6..2d147345bf8 100644 --- a/mozilla/mailnews/addrbook/src/nsCardDataSource.cpp +++ b/mozilla/mailnews/addrbook/src/nsCardDataSource.cpp @@ -454,6 +454,7 @@ nsresult nsAbCardDataSource::DoNewCard(nsIAbCard *card, nsISupportsArray *argume PRUnichar *name; literal->GetValue(&name); nsString tempStr = name; + nsAllocator::Free(name); nsAutoCString nameStr(tempStr); // rv = card->CreateNewCard(nameStr); diff --git a/mozilla/mailnews/addrbook/src/nsDirectoryDataSource.cpp b/mozilla/mailnews/addrbook/src/nsDirectoryDataSource.cpp index 115eb159734..f0317414279 100644 --- a/mozilla/mailnews/addrbook/src/nsDirectoryDataSource.cpp +++ b/mozilla/mailnews/addrbook/src/nsDirectoryDataSource.cpp @@ -626,6 +626,7 @@ nsresult nsAbDirectoryDataSource::DoNewDirectory(nsIAbDirectory *directory, nsIS literal->GetValue(&name); rv = directory->CreateNewDirectory(name, nsnull); + nsAllocator::Free(name); } return rv; }