Skip to content

Commit 83c145f

Browse files
authored
fix: use a valid 20-byte signer in the gasless module-sender test (#331)
The hardcoded literal decoded to 42 bytes, which F-2026-18200's signer-length guard rejects in GetAddressPair before ExecutePayload runs. Red on audit-fixes since #317.
1 parent 115951e commit 83c145f

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

test/integration/uexecutor/gasless_module_sender_test.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,8 +120,13 @@ func TestGaslessExecutePayloadWithModuleSender(t *testing.T) {
120120

121121
// The gasless message itself: signer is a relayer, the EVM caller is the
122122
// uexecutor module. This must still succeed after the x/vm change.
123+
// testSigner (execute_payload_test.go) is a valid 20-byte account. The
124+
// literal this test originally carried decoded to 42 bytes, which
125+
// F-2026-18200's signer-length guard rejects in GetAddressPair before
126+
// ExecutePayload does any work - so the test failed on an address that was
127+
// never the thing under test.
123128
_, err = ms.ExecutePayload(ctx, &uexecutortypes.MsgExecutePayload{
124-
Signer: "cosmos1xpurwdecvsenyvpkxvmnge3cv93nyd34xuersef38pjnxen9xfsk2dnz8yek2drrv56qmn2ak9",
129+
Signer: testSigner,
125130
UniversalAccountId: universalAccount,
126131
UniversalPayload: payload,
127132
VerificationData: "0x91987784d56359fa91c3e3e0332f4f0cffedf9c081eb12874a63b41d5b5e5c660dc827947c2ae26e658d0551ad4b2d2aa073d62691429a0ae239d2cc58055bf11c",

0 commit comments

Comments
 (0)