diff options
author | Eike Rathke <erack@redhat.com> | 2013-04-05 14:41:46 +0200 |
---|---|---|
committer | Michael Meeks <michael.meeks@suse.com> | 2013-04-05 13:24:13 +0000 |
commit | ec1c30d34d9c9445ab6e22e280a9868cf910a3bc (patch) | |
tree | fc6dbe83948bebbfe90faea5156744b6e1f68240 /svl | |
parent | 9779054cc12fa8025f85b5d7d36c13cd25982db1 (diff) |
resolved fdo#63161 out of bounds string access
Introduced with c7709b460394283fd5b1d2779b6af3585f80a991
String to OUString where previous String::GetChar() handled this.
Change-Id: I9253df1af7498e2d7a8ea8077ec6b369697ad44e
(cherry picked from commit d526e48912deeb44061ff570d715c31ca45f77b8)
Reviewed-on: https://gerrit.libreoffice.org/3211
Reviewed-by: Michael Meeks <michael.meeks@suse.com>
Tested-by: Michael Meeks <michael.meeks@suse.com>
Diffstat (limited to 'svl')
-rw-r--r-- | svl/source/numbers/zforfind.hxx | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/svl/source/numbers/zforfind.hxx b/svl/source/numbers/zforfind.hxx index 412e189f5028..dadde5cc1a46 100644 --- a/svl/source/numbers/zforfind.hxx +++ b/svl/source/numbers/zforfind.hxx @@ -218,6 +218,10 @@ private: const OUString& rString, sal_Int32 nPos ) { + if (rWhat.isEmpty() || rString.getLength() <= nPos) + { + return false; + } // mostly used with one character if ( rWhat[ 0 ] != rString[ nPos ] ) { |