[1.0.rc-1] Lockdrop Fix Needed for Match Message in Execution
Closed this issue · 0 comments
daniel-wehbe commented
Describe the bug
In the lockdrop contract.rs, we have the following match message for handling execution:
match msg {
ExecuteMsg::Receive(msg) => receive_cw20(ctx, msg),
ExecuteMsg::DepositNative {} => execute_deposit_native(ctx),
ExecuteMsg::WithdrawNative { amount } => execute_withdraw_native(ctx, amount),
ExecuteMsg::EnableClaims {} => execute_enable_claims(ctx),
ExecuteMsg::ClaimRewards {} => execute_claim_rewards(ctx),
// ExecuteMsg::WithdrawProceeds { recipient } => execute_withdraw_proceeds(ctx, recipient),
_ => handle_execute(ctx, msg),
}
The last line is incorrect and will cause infinite looping:
_ => handle_execute(ctx, msg),
Should be replaced with going to the default executes:
_ => ADOContract::default().execute(ctx, msg),