Fix getString in COSString

MaximPlusov · MaximPlusov · commit cc574e1387ab · 2024-12-19T00:19:38.000+03:00
diff --git a/src/main/java/org/verapdf/cos/COSString.java b/src/main/java/org/verapdf/cos/COSString.java
@@ -106,15 +106,15 @@ public Double getReal() {
 
     @Override
     public String getString() {
-        if (value.length > 2) {
+        if (value.length >= 2) {
             if ((value[0] & 0xFF) == 0xFE && (value[1] & 0xFF) == 0xFF) {
                 return new String(value, 2, value.length - 2, StandardCharsets.UTF_16BE);
             }
             if ((value[0] & 0xFF) == 0xFF && (value[1] & 0xFF) == 0xFE) {
                 LOGGER.log(Level.WARNING, "String object uses encoding UTF16-LE not supported by PDF");
             }
         }
-        if (value.length > 3) {
+        if (value.length >= 3) {
             if ((value[0] & 0xFF) == 0xEF && (value[1] & 0xFF) == 0xBB && (value[2] & 0xFF) == 0xBF) {
                 return new String(value, 3, value.length - 3, StandardCharsets.UTF_8);
             }

Original file line number	Diff line number	Diff line change
`@@ -106,15 +106,15 @@ public Double getReal() {`
`106`	`106`
`107`	`107`	`@Override`
`108`	`108`	`public String getString() {`
`109`		`- if (value.length > 2) {`
	`109`	`+ if (value.length >= 2) {`
`110`	`110`	`if ((value[0] & 0xFF) == 0xFE && (value[1] & 0xFF) == 0xFF) {`
`111`	`111`	`return new String(value, 2, value.length - 2, StandardCharsets.UTF_16BE);`
`112`	`112`	`}`
`113`	`113`	`if ((value[0] & 0xFF) == 0xFF && (value[1] & 0xFF) == 0xFE) {`
`114`	`114`	`LOGGER.log(Level.WARNING, "String object uses encoding UTF16-LE not supported by PDF");`
`115`	`115`	`}`
`116`	`116`	`}`
`117`		`- if (value.length > 3) {`
	`117`	`+ if (value.length >= 3) {`
`118`	`118`	`if ((value[0] & 0xFF) == 0xEF && (value[1] & 0xFF) == 0xBB && (value[2] & 0xFF) == 0xBF) {`
`119`	`119`	`return new String(value, 3, value.length - 3, StandardCharsets.UTF_8);`
`120`	`120`	`}`