You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2014/10/08 19:14:32 UTC
svn commit: r1630186 -
/pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java
Author: tilman
Date: Wed Oct 8 17:14:31 2014
New Revision: 1630186
URL: http://svn.apache.org/r1630186
Log:
PDFBOX-2401: fix COSString equals so that String values with 00 and FF are considered different
Modified:
pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java
Modified: pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java?rev=1630186&r1=1630185&r2=1630186&view=diff
==============================================================================
--- pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java (original)
+++ pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java Wed Oct 8 17:14:31 2014
@@ -22,6 +22,7 @@ import java.io.OutputStream;
import java.nio.ByteBuffer;
import java.nio.CharBuffer;
import java.nio.charset.Charset;
+import java.util.Arrays;
import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory;
@@ -481,7 +482,9 @@ public class COSString extends COSBase
if (obj instanceof COSString)
{
COSString strObj = (COSString) obj;
- return this.getString().equals(strObj.getString()) && forceHexForm == strObj.forceHexForm;
+ // Workaround because different String objects with
+ // 0x00 and 0xFF content are equal in java?!
+ return Arrays.equals(this.getBytes(), strObj.getBytes()) && this.forceHexForm == strObj.forceHexForm;
}
return false;
}