Merge pull request #243 from Nadahar/fix_signed_byte_bug

drewnoakes · web-flow · commit bb05e9e59578 · 2017-02-15T22:52:00.000Z
Fix another "signed byte" bug introduced in #234
diff --git a/Source/com/drew/metadata/jpeg/JpegDhtReader.java b/Source/com/drew/metadata/jpeg/JpegDhtReader.java
@@ -72,7 +72,7 @@ public void extract(@NotNull final SequentialReader reader, @NotNull final Metad
                 byte[] lBytes = getBytes(reader, 16);
                 int vCount = 0;
                 for (byte b : lBytes) {
-                    vCount += b;
+                    vCount += (b & 0xFF);
                 }
                 byte[] vBytes = getBytes(reader, vCount);
                 directory.getTables().add(new HuffmanTable(tableClass, tableDestinationId, lBytes, vBytes));

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ public void extract(@NotNull final SequentialReader reader, @NotNull final Metad`
`72`	`72`	`byte[] lBytes = getBytes(reader, 16);`
`73`	`73`	`int vCount = 0;`
`74`	`74`	`for (byte b : lBytes) {`
`75`		`- vCount += b;`
	`75`	`+ vCount += (b & 0xFF);`
`76`	`76`	`}`
`77`	`77`	`byte[] vBytes = getBytes(reader, vCount);`
`78`	`78`	`directory.getTables().add(new HuffmanTable(tableClass, tableDestinationId, lBytes, vBytes));`