sc/source/ui/docshell/tablink.cxx | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
New commits:
commit b1caf176a44b6979d2e0ea47f495a3dacf86e197
Author: Eike Rathke <er...@redhat.com>
Date: Thu Oct 10 18:02:25 2013 +0200
fixed another "Out of bounds substring access"
Introduced with 3c2a26c84e70fd996855181d319970e2230b2459
Didn't have consequences here because the result was used to check for a
"startsWith" condition which it wouldn't had matched anyway. However,
use the real startsWith() now.
Change-Id: Ia1d558fc106d9a2010ad41c30f7eb2a2a480c59b
diff --git a/sc/source/ui/docshell/tablink.cxx
b/sc/source/ui/docshell/tablink.cxx
index 5a4cfcd..b6255d1 100644
--- a/sc/source/ui/docshell/tablink.cxx
+++ b/sc/source/ui/docshell/tablink.cxx
@@ -501,12 +501,11 @@ bool ScDocumentLoader::GetFilterName( const OUString&
rFileName,
void ScDocumentLoader::RemoveAppPrefix( OUString& rFilterName )
{
- OUStringBuffer aAppPrefix;
- aAppPrefix.appendAscii(STRING_SCAPP);
- aAppPrefix.appendAscii(": ");
- sal_Int32 nPreLen = aAppPrefix.getLength();
- if (rFilterName.copy(0, nPreLen).equals(aAppPrefix.makeStringAndClear()))
- rFilterName = rFilterName.copy(nPreLen);
+ OUStringBuffer aBuf;
+ aBuf.appendAscii( STRING_SCAPP).appendAscii( ": ");
+ OUString aAppPrefix( aBuf.makeStringAndClear());
+ if (rFilterName.startsWith( aAppPrefix))
+ rFilterName = rFilterName.copy( aAppPrefix.getLength());
}
ScDocumentLoader::ScDocumentLoader( const OUString& rFileName,
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
