Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix for bug 26236. Unescaping can change the length of the string. We now use the length of the unescaped string. Removed unnecessary null check. r=ftang

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@59575 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
vidur%netscape.com
2000-02-03 00:11:16 +00:00
parent 6553dfb4d1
commit 96827d8003
1 changed files with 1 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
mozilla/dom/src/base/nsGlobalWindow.cpp
View File

@@ -1627,13 +1627,9 @@ GlobalWindowImpl::Unescape(const nsString& aStr, nsString& aReturn)
// Unescape the string
char* src = nsUnescape(inBuf);
// nsAllocator::Free(inBuf);
if (nsnull == src) {
return NS_ERROR_OUT_OF_MEMORY;
}
PRInt32 maxLength, srcLen;
srcLen = aStr.Length();
srcLen = nsCRT::strlen(src);
// Get the expected length of the result string
result = decoder->GetMaxLength(src, srcLen, &maxLength);