Author: fanningpj
Date: Sat Jan 28 20:44:26 2023
New Revision: 1907066

URL: http://svn.apache.org/viewvc?rev=1907066&view=rev
Log:
edge case (avoid short overflow)

Modified:
    
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java

Modified: 
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java?rev=1907066&r1=1907065&r2=1907066&view=diff
==============================================================================
--- 
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java 
(original)
+++ 
poi/trunk/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFSheet.java 
Sat Jan 28 20:44:26 2023
@@ -1706,9 +1706,7 @@ public class XSSFSheet extends POIXMLDoc
             index = Math.toIntExact(col.getMax());
         }
         worksheet.setColsArray(0, ctCols);
-        if (maxLevelCol > getSheetFormatPrOutlineLevelCol()) {
-            increaseSheetFormatPrOutlineLevelColIfNecessary((short) 
maxLevelCol);
-        }
+        increaseSheetFormatPrOutlineLevelColIfNecessary((short) 
Math.min(Short.MAX_VALUE, maxLevelCol));
     }
 
     /**
@@ -1743,9 +1741,7 @@ public class XSSFSheet extends POIXMLDoc
             maxOutlineLevel = Math.max(maxOutlineLevel, newOutlineLevel);
             ctrow.setOutlineLevel((short) newOutlineLevel);
         }
-        if (maxOutlineLevel >= 0) {
-            increaseSheetFormatPrOutlineLevelRowIfNecessary((short) 
maxOutlineLevel);
-        }
+        increaseSheetFormatPrOutlineLevelRowIfNecessary((short) 
Math.min(Short.MAX_VALUE, maxOutlineLevel));
     }
 
     private short getMaxOutlineLevelRows(){



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to