bug 297584 : MS IE profile migrator casts LWPSTR to |char *| and treats the result as UTF-8. fixes memory leaks as well (r=cbie, sr=darin, a=asa)

This commit is contained in:
jshin%mailaps.org 2005-06-18 01:33:50 +00:00
Родитель 84d4890b64
Коммит acc702ca90
1 изменённых файлов: 12 добавлений и 6 удалений

Просмотреть файл

@ -556,9 +556,14 @@ nsIEProfileMigrator::CopyHistory(PRBool aReplace)
// 3 - URL
url = statURL.pwcsUrl;
nsDependentCString urlStr((const char *) url);
if (NS_FAILED(ios->ExtractScheme(urlStr, scheme)))
NS_ConvertUTF16toUTF8 urlStr(url);
if (NS_FAILED(ios->ExtractScheme(urlStr, scheme))) {
::CoTaskMemFree(statURL.pwcsUrl);
if (statURL.pwcsTitle)
::CoTaskMemFree(statURL.pwcsTitle);
continue;
}
ToLowerCase(scheme);
// XXXben -
@ -578,13 +583,14 @@ nsIEProfileMigrator::CopyHistory(PRBool aReplace)
if (uri) {
if (tempTitle)
hist->AddPageWithDetails(uri, tempTitle, lastVisited);
else {
NS_ConvertUTF8toUTF16 urlTitle(urlStr);
hist->AddPageWithDetails(uri, urlTitle.get(), lastVisited);
}
else
hist->AddPageWithDetails(uri, url, lastVisited);
}
}
::CoTaskMemFree(statURL.pwcsUrl);
}
if (statURL.pwcsTitle)
::CoTaskMemFree(statURL.pwcsTitle);
}
nsCOMPtr<nsIRDFRemoteDataSource> ds(do_QueryInterface(hist));
if (ds)