[Libreoffice-commits] core.git: l10ntools/source
Zolnai Tamás
zolnaitamas2000 at gmail.com
Tue Apr 2 07:23:28 PDT 2013
l10ntools/source/xmlparse.cxx | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
New commits:
commit 68c72dfb9db5c452bad13203699b17c4f4fcf6cc
Author: Zolnai Tamás <zolnaitamas2000 at gmail.com>
Date: Tue Apr 2 16:16:12 2013 +0200
Fix assertion in xmlparse and call this function with more care
Plus change range to [nStart,nEnd).
Change-Id: I1570d07fdc90a6b2bdf3eef7914958212cbbaa87
diff --git a/l10ntools/source/xmlparse.cxx b/l10ntools/source/xmlparse.cxx
index 651a27a..954aed0 100644
--- a/l10ntools/source/xmlparse.cxx
+++ b/l10ntools/source/xmlparse.cxx
@@ -1180,9 +1180,10 @@ static icu::UnicodeString lcl_QuotRange(
const sal_Int32 nEnd, bool bInsideTag = false )
{
icu::UnicodeString sReturn;
+ assert( nStart < nEnd );
assert( nStart >= 0 );
- assert( nEnd < rString.length() );
- for (sal_Int32 i = nStart; i <= nEnd; ++i)
+ assert( nEnd <= rString.length() );
+ for (sal_Int32 i = nStart; i < nEnd; ++i)
{
switch (rString[i])
{
@@ -1258,20 +1259,22 @@ OUString XMLUtil::QuotHTML( const OUString &rString )
while( aRegexMatcher.find(nStartPos, nIcuErr) && nIcuErr == U_ZERO_ERROR )
{
nStartPos = aRegexMatcher.start(nIcuErr);
- sReturn.append(lcl_QuotRange(sSource, nEndPos, nStartPos-1));
+ if ( nEndPos < nStartPos )
+ sReturn.append(lcl_QuotRange(sSource, nEndPos, nStartPos));
nEndPos = aRegexMatcher.end(nIcuErr);
icu::UnicodeString sMatch = aRegexMatcher.group(nIcuErr);
if( lcl_isTag(sMatch) )
{
sReturn.append("<");
- sReturn.append(lcl_QuotRange(sSource, nStartPos+1, nEndPos-2, true));
+ sReturn.append(lcl_QuotRange(sSource, nStartPos+1, nEndPos-1, true));
sReturn.append(">");
}
else
- sReturn.append(lcl_QuotRange(sSource, nStartPos, nEndPos-1));
+ sReturn.append(lcl_QuotRange(sSource, nStartPos, nEndPos));
++nStartPos;
}
- sReturn.append(lcl_QuotRange(sSource, nEndPos, sSource.length()-1));
+ if( nEndPos < sSource.length() )
+ sReturn.append(lcl_QuotRange(sSource, nEndPos, sSource.length()));
sReturn.append('\0');
return OUString(reinterpret_cast<const sal_Unicode*>(sReturn.getBuffer()));
}
More information about the Libreoffice-commits
mailing list