Skip to content
This repository was archived by the owner on Oct 20, 2024. It is now read-only.

Commit 4e71418

Browse files
authored
Add AA41 to validation OOG case (#286)
1 parent 66a3b89 commit 4e71418

File tree

2 files changed

+10
-5
lines changed

2 files changed

+10
-5
lines changed

pkg/entrypoint/execution/simulate.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,9 @@ func SimulateHandleOp(in *SimulateInput) (*reverts.ExecutionResultRevert, error)
4242
if simErr != nil {
4343
fo, foErr := reverts.NewFailedOp(err)
4444
if foErr != nil {
45+
if err != nil {
46+
return nil, err
47+
}
4548
return nil, fmt.Errorf("%s, %s", simErr, foErr)
4649
}
4750
return nil, errors.NewRPCError(errors.REJECTED_BY_EP_OR_ACCOUNT, fo.Reason, fo)

pkg/gas/estimate.go

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,14 +17,16 @@ var (
1717
)
1818

1919
func isPrefundNotPaid(err error) bool {
20-
return strings.HasPrefix(err.Error(), "AA21") || strings.HasPrefix(err.Error(), "AA31")
20+
return strings.Contains(err.Error(), "AA21 didn't pay prefund") ||
21+
strings.Contains(err.Error(), "AA31 paymaster deposit too low")
2122
}
2223

2324
func isValidationOOG(err error) bool {
24-
return strings.HasPrefix(err.Error(), "AA13") ||
25-
strings.HasPrefix(err.Error(), "AA40") ||
26-
err.Error() == "AA23 reverted (or OOG)" ||
27-
err.Error() == "AA33 reverted (or OOG)" ||
25+
return strings.Contains(err.Error(), "AA40 over verificationGasLimit") ||
26+
strings.Contains(err.Error(), "AA41 too little verificationGas") ||
27+
strings.Contains(err.Error(), "AA13 initCode failed or OOG") ||
28+
strings.Contains(err.Error(), "AA23 reverted (or OOG)") ||
29+
strings.Contains(err.Error(), "AA33 reverted (or OOG)") ||
2830
strings.Contains(err.Error(), "return data out of bounds") ||
2931
strings.Contains(err.Error(), "validation OOG")
3032
}

0 commit comments

Comments
 (0)