mir/libreoffice
2
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

coverity#1019431: use of freed memory

Browse Source
This commit is contained in:
Markus Mohrhard 2013-05-08 20:48:14 +02:00
parent 9210f1a50a
commit 228659f0a6
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
unotools/source/i18n/localedatawrapper.cxx
View File

@ -20,6 +20,7 @@
#include <string.h> // memcpy() #include <string.h> // memcpy()
#include <stdio.h> // fprintf(), stderr #include <stdio.h> // fprintf(), stderr
#include <string>
#include <unotools/localedatawrapper.hxx> #include <unotools/localedatawrapper.hxx>
#include <unotools/numberformatcodewrapper.hxx> #include <unotools/numberformatcodewrapper.hxx>
@ -1168,7 +1169,7 @@ static sal_Unicode* ImplAdd2UNum( sal_Unicode* pBuf, sal_uInt16 nNumber, bool bL
static sal_Unicode* ImplAdd9UNum( sal_Unicode* pBuf, sal_uInt32 nNumber, bool bLeading ) static sal_Unicode* ImplAdd9UNum( sal_Unicode* pBuf, sal_uInt32 nNumber, bool bLeading )
{ {
DBG_ASSERT( nNumber < 1000000000, "ImplAdd2UNum() - Number >= 1000000000" ); DBG_ASSERT( nNumber < 1000000000, "ImplAdd9UNum() - Number >= 1000000000" );
std::ostringstream ostr; std::ostringstream ostr;
if (bLeading) if (bLeading)
@ -1177,7 +1178,8 @@ static sal_Unicode* ImplAdd9UNum( sal_Unicode* pBuf, sal_uInt32 nNumber, bool bL
ostr.width(9); ostr.width(9);
} }
ostr << nNumber; ostr << nNumber;
for(const char *pAB=ostr.str().c_str(); *pAB != '\0'; ++pAB, ++pBuf) std::string aStr = ostr.str();
for(const char *pAB= aStr.c_str(); *pAB != '\0'; ++pAB, ++pBuf)
{ {
*pBuf = *pAB; *pBuf = *pAB;
} }