Fixed bug: For extension node condition should use current path point…

…er instead of updated path pointer
0xsarvesh · Feb 15, 2019 · 97714b6 · 97714b6
1 parent c48747a
commit 97714b6
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/contracts/lib/MerklePatriciaProof.sol b/contracts/lib/MerklePatriciaProof.sol
@@ -60,6 +60,7 @@ library MerklePatriciaProof {
                 nodeKey = RLP.toBytes32(currentNodeList[nextPathNibble]);
                 pathPtr += 1;
             } else if(currentNodeList.length == 2) {
+                uint currentpathPtr = pathPtr;
                 pathPtr += _nibblesToTraverse(RLP.toData(currentNodeList[0]), path, pathPtr);
 
                 if(pathPtr == path.length) {//leaf node
@@ -70,7 +71,7 @@ library MerklePatriciaProof {
                     }
                 }
                 //extension node ... test if means that it is empty value
-                if(_nibblesToTraverse(RLP.toData(currentNodeList[0]), path, pathPtr) == 0) {
+                if(_nibblesToTraverse(RLP.toData(currentNodeList[0]), path, currentpathPtr) == 0) {
                     return (keccak256(abi.encodePacked()) == value);
                 }