fix unsafe string handling in wxGetTranslation

git-svn-id: https://bitcoin.svn.sourceforge.net/svnroot/bitcoin/trunk@73 1a98c847-1fd6-4fd8-948a-caf3550aa51b
This commit is contained in:
s_nakamoto 2010-02-28 15:00:32 +00:00
parent 6557910ccf
commit d7d80a74d5
3 changed files with 14 additions and 14 deletions

View File

@ -20,7 +20,7 @@ class CDataStream;
class CAutoFile; class CAutoFile;
static const int VERSION = 207; static const int VERSION = 207;
static const char* pszSubVer = ".0"; static const char* pszSubVer = ".1";

View File

@ -445,17 +445,17 @@ const char* wxGetTranslation(const char* pszEnglish)
return (*mi).second; return (*mi).second;
// wxWidgets translation // wxWidgets translation
const char* pszTranslated = wxGetTranslation(wxString(pszEnglish, wxConvUTF8)).utf8_str(); wxString strTranslated = wxGetTranslation(wxString(pszEnglish, wxConvUTF8));
// We don't cache unknown strings because caller might be passing in a // We don't cache unknown strings because caller might be passing in a
// dynamic string and we would keep allocating memory for each variation. // dynamic string and we would keep allocating memory for each variation.
if (strcmp(pszEnglish, pszTranslated) == 0) if (strcmp(pszEnglish, strTranslated.utf8_str()) == 0)
return pszEnglish; return pszEnglish;
// Add to cache, memory doesn't need to be freed. We only cache because // Add to cache, memory doesn't need to be freed. We only cache because
// we must pass back a pointer to permanently allocated memory. // we must pass back a pointer to permanently allocated memory.
char* pszCached = new char[strlen(pszTranslated)+1]; char* pszCached = new char[strlen(strTranslated.utf8_str())+1];
strcpy(pszCached, pszTranslated); strcpy(pszCached, strTranslated.utf8_str());
mapCache[pszEnglish] = pszCached; mapCache[pszEnglish] = pszCached;
return pszCached; return pszCached;
} }