diff --git a/poi/src/main/java/org/apache/poi/hpsf/TypedPropertyValue.java b/poi/src/main/java/org/apache/poi/hpsf/TypedPropertyValue.java
index 1b4f5c64890..2be08b3b09d 100644
--- a/poi/src/main/java/org/apache/poi/hpsf/TypedPropertyValue.java
+++ b/poi/src/main/java/org/apache/poi/hpsf/TypedPropertyValue.java
@@ -255,7 +255,7 @@ static void skipPadding( LittleEndianByteArrayInputStream lei ) {
         for (int i=0; i<skipBytes; i++) {
             lei.mark(1);
             int b = lei.read();
-            if (b == -1 || b != 0) {
+            if (b != 0) {
                 lei.reset();
                 break;
             }
diff --git a/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFSheet.java b/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
index bf879598124..7fb8d3a5e39 100644
--- a/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
+++ b/poi/src/main/java/org/apache/poi/hssf/usermodel/HSSFSheet.java
@@ -2552,7 +2552,7 @@ private void setRepeatingRowsAndColumns(
         if (rowDef != null) {
             row1 = rowDef.getFirstRow();
             row2 = rowDef.getLastRow();
-            if ((row1 == -1 && row2 != -1) || (row1 > row2)
+            if ((row1 > row2)
                     || (row1 < 0 || row1 > maxRowIndex)
                     || (row2 < 0 || row2 > maxRowIndex)) {
                 throw new IllegalArgumentException("Invalid row range specification");
@@ -2561,7 +2561,7 @@ private void setRepeatingRowsAndColumns(
         if (colDef != null) {
             col1 = colDef.getFirstColumn();
             col2 = colDef.getLastColumn();
-            if ((col1 == -1 && col2 != -1) || (col1 > col2)
+            if ((col1 > col2)
                     || (col1 < 0 || col1 > maxColIndex)
                     || (col2 < 0 || col2 > maxColIndex)) {
                 throw new IllegalArgumentException("Invalid column range specification");