You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tika.apache.org by ta...@apache.org on 2016/10/20 19:13:24 UTC
tika git commit: TIKA-2133
Repository: tika
Updated Branches:
refs/heads/master a6b8e0451 -> bc7216ff7
TIKA-2133
Project: http://git-wip-us.apache.org/repos/asf/tika/repo
Commit: http://git-wip-us.apache.org/repos/asf/tika/commit/bc7216ff
Tree: http://git-wip-us.apache.org/repos/asf/tika/tree/bc7216ff
Diff: http://git-wip-us.apache.org/repos/asf/tika/diff/bc7216ff
Branch: refs/heads/master
Commit: bc7216ff70d87c71c5817cda89d990d4f7075ccf
Parents: a6b8e04
Author: tballison <ta...@mitre.org>
Authored: Thu Oct 20 15:13:17 2016 -0400
Committer: tballison <ta...@mitre.org>
Committed: Thu Oct 20 15:13:17 2016 -0400
----------------------------------------------------------------------
.../main/java/org/apache/tika/parser/microsoft/ListManager.java | 5 +++++
1 file changed, 5 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/tika/blob/bc7216ff/tika-parsers/src/main/java/org/apache/tika/parser/microsoft/ListManager.java
----------------------------------------------------------------------
diff --git a/tika-parsers/src/main/java/org/apache/tika/parser/microsoft/ListManager.java b/tika-parsers/src/main/java/org/apache/tika/parser/microsoft/ListManager.java
index a211de5..a0f19e1 100644
--- a/tika-parsers/src/main/java/org/apache/tika/parser/microsoft/ListManager.java
+++ b/tika-parsers/src/main/java/org/apache/tika/parser/microsoft/ListManager.java
@@ -145,6 +145,11 @@ public class ListManager extends AbstractListManager {
if (offset == 0) {
break;
}
+ if (offset-1 < last || offset > numberText.length()) {
+ //something went wrong.
+ //silently stop
+ break;
+ }
sb.append(numberText.substring(last, offset - 1));
//need to add one because newer format
//adds one. In .doc, this was the array index;