Skip to content

Commit 937fd4a

Browse files
committed
Fix wrong wallet RPC context set after #21366
This bug doesn't have any effects currently because it only affects external signer RPCs which aren't currently using the wallet context, but it does cause an appveyor failure in a upcoming PR: https://ci.appveyor.com/project/DrahtBot/bitcoin/builds/38512882 This bug is subtle and could have been avoided if JSONRPCRequest didn't have constructors that were so loose with type checking. Suggested change bitcoin/bitcoin#21366 (comment) eliminates these and would be a good followup for a future PR.
1 parent e08f319 commit 937fd4a

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/wallet/interfaces.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -522,7 +522,7 @@ class WalletClientImpl : public WalletClient
522522
#ifdef ENABLE_EXTERNAL_SIGNER
523523
for (const CRPCCommand& command : GetSignerRPCCommands()) {
524524
m_rpc_commands.emplace_back(command.category, command.name, [this, &command](const JSONRPCRequest& request, UniValue& result, bool last_handler) {
525-
return command.actor({request, m_context}, result, last_handler);
525+
return command.actor({request, &m_context}, result, last_handler);
526526
}, command.argNames, command.unique_id);
527527
m_rpc_handlers.emplace_back(m_context.chain->handleRpc(m_rpc_commands.back()));
528528
}

0 commit comments

Comments
 (0)