From 38b755363e200cbd551cec78da4ecf5c0b620c61 Mon Sep 17 00:00:00 2001 From: josedahlquist Date: Fri, 28 Oct 2022 15:12:00 -0300 Subject: [PATCH] Update RSKIP176 Fixed incorrect formatting definition for derivation hash --- IPs/RSKIP176.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/IPs/RSKIP176.md b/IPs/RSKIP176.md index 617c5fcf..013e23e2 100644 --- a/IPs/RSKIP176.md +++ b/IPs/RSKIP176.md @@ -61,9 +61,9 @@ A new method called **registerFastBridgeBtcTransaction**() is added to the Bridg - *bytes* **liquidityProviderBtcAddress**: Binary encoding representing the Liquidity Provider BTC refund address. - *bool* **shouldTransferToContract**: Status provided by the caller contract to know if should transfer value to the contract or not. -To compute the UPI, the IDAs are serialized as a concatenation of derivationArgumentsHash (32 bytes), userRefundBtcAddress (21 bytes), liquidityProviderBtcAddress (21 bytes), liquidityBridgeContractAddress (20 bytes). Each argument occupies a fixed number of bytes shown in brackets. Formally, this is: +To compute the UPI, the IDAs are serialized as a concatenation of derivationArgumentsHash (32 bytes), userRefundBtcAddress (21 bytes), liquidityBridgeContractAddress (20 bytes), liquidityProviderBtcAddress (21 bytes). Each argument occupies a fixed number of bytes shown in brackets. Formally, this is: -`UPI = Keccak256(derivationArgumentsHash, userRefundBtcAddress, liquidityProviderBtcAddress , liquidityBridgeContractAddress)` +`UPI = Keccak256(derivationArgumentsHash, userRefundBtcAddress, liquidityBridgeContractAddress , liquidityProviderBtcAddress)` The total number of bytes hashed is 94. Note that the concatenation order is different from the argument passing order.