getStorageAt from empty cell now returns 0x0

rsksmart · Apr 7, 2021 · 9b36a73 · 9b36a73
1 parent dc17c4e
commit 9b36a73
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/rskj-core/src/main/java/org/ethereum/rpc/Web3Impl.java b/rskj-core/src/main/java/org/ethereum/rpc/Web3Impl.java
@@ -409,7 +409,7 @@ public String eth_getStorageAt(String address, String storageIdx, String blockId
                     .getStorageValue(addr, DataWord.valueOf(stringHexToByteArray(storageIdx)));
 
             if (sv == null) {
-                s = null;
+                s = "0x0";
             } else {
                 s = toUnformattedJsonHex(sv.getData());
             }

diff --git a/rskj-core/src/test/java/org/ethereum/rpc/Web3ImplUnitTest.java b/rskj-core/src/test/java/org/ethereum/rpc/Web3ImplUnitTest.java
@@ -143,6 +143,25 @@ public void eth_getStorageAt() {
                 result);
     }
 
+
+    @Test
+    public void eth_getStorageAtEmptyCell() {
+        String id = "id";
+        String addr = "0x0011223344556677880011223344556677889900";
+        RskAddress expectedAddress = new RskAddress(addr);
+        String storageIdx = "0x01";
+        DataWord expectedIdx = DataWord.valueOf(stringHexToByteArray(storageIdx));
+
+        AccountInformationProvider aip = mock(AccountInformationProvider.class);
+        when(retriever.getInformationProvider(eq(id))).thenReturn(aip);
+        when(aip.getStorageValue(eq(expectedAddress), eq(expectedIdx)))
+                .thenReturn(null);
+
+        String result = target.eth_getStorageAt(addr, storageIdx, id);
+        assertEquals("0x0",
+                result);
+    }
+
     @Test
     public void eth_getBlockTransactionCountByNumber_blockNotFound() {
         String id = "id";