fix bug in VirtualPaymentApp

Bob's address was hardcoded and only correct for a single intermediary case
statechannels · Oct 6, 2023 · 3c19430 · 3c19430
1 parent 5e3ac38
commit 3c19430
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/packages/nitro-protocol/contracts/VirtualPaymentApp.sol b/packages/nitro-protocol/contracts/VirtualPaymentApp.sol
@@ -60,8 +60,9 @@ contract VirtualPaymentApp is IForceMoveApp {
         if (proof.length == 1) {
             requireProofOfUnanimousConsensusOnPostFund(proof[0], fixedPart.participants.length);
             require(candidate.variablePart.turnNum == 2, 'bad candidate turnNum; |proof|=1');
+            uint8 bobIndex = uint8(fixedPart.participants.length - 1);
             require(
-                NitroUtils.isClaimedSignedBy(candidate.signedBy, 2),
+                NitroUtils.isClaimedSignedBy(candidate.signedBy, bobIndex),
                 'redemption not signed by Bob'
             );
             uint256 voucherAmount = requireValidVoucher(candidate.variablePart.appData, fixedPart);