You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-commits@jackrabbit.apache.org by md...@apache.org on 2016/12/14 15:29:03 UTC

svn commit: r1774269 - /jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/MapEntry.java

Author: mduerig
Date: Wed Dec 14 15:29:03 2016
New Revision: 1774269

URL: http://svn.apache.org/viewvc?rev=1774269&view=rev
Log:
OAK-5301: Possible null dereference in MapRecord
FIXME tag

Modified:
    jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/MapEntry.java

Modified: jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/MapEntry.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/MapEntry.java?rev=1774269&r1=1774268&r2=1774269&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/MapEntry.java (original)
+++ jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/MapEntry.java Wed Dec 14 15:29:03 2016
@@ -100,7 +100,7 @@ class MapEntry extends AbstractChildNode
         return ComparisonChain.start()
                 .compare(getHash() & HASH_MASK, that.getHash() & HASH_MASK)
                 .compare(name, that.name)
-                .compare(value, that.value)
+                .compare(value, that.value)  // FIXME OAK-5301: Possible null dereference in MapRecord
                 .result();
     }