TIKA-3336 -- new zip bombs detect in 1.26-SNAPSHOT compared with 1.25 -- bug, don't advance twice per call to chars/whitespace

Author: tballison

tika-core/src/main/java/org/apache/tika/parser/RecursiveParserWrapper.java

@@ -488,12 +488,10 @@ public void endElement(String uri, String localName, String name) throws SAXExce
         @Override
         public void characters(char[] ch, int start, int length) throws SAXException {
             if (totalWriteLimit < 0) {
-                advance(length);
                 super.characters(ch, start, length);
                 return;
             }
             int availableLength = Math.min(totalWriteLimit - totalChars, length);
-            advance(availableLength);
             super.characters(ch, start, availableLength);
             if (availableLength < length) {
                 throw new WriteLimitReached();
@@ -503,12 +501,10 @@ public void characters(char[] ch, int start, int length) throws SAXException {
         @Override
         public void ignorableWhitespace(char[] ch, int start, int length) throws SAXException {
             if (totalWriteLimit < 0) {
-                advance(length);
                 super.ignorableWhitespace(ch, start, length);
                 return;
             }
             int availableLength = Math.min(totalWriteLimit - totalChars, length);
-            advance(availableLength);
             super.ignorableWhitespace(ch, start, availableLength);
             if (availableLength < length) {
                 throw new WriteLimitReached();