Skip to content

Conversation

@songgaoye
Copy link
Contributor

Description

The pull request references are PR #332 and PR #274.

I believe PR #274 also requires a modification in another location.

rBz, sBz, err := parseSignature(sig)
s.Require().NoError(err)
input := make([]byte, p256.VerifyInputLength)
copy(input[0:32], hash)
copy(input[32:64], rBz)
copy(input[64:96], sBz)

Referring to issue #273, I think rBz, rBz(when shorter than 32 bytes) has the same issue.

Closes: #XXXX


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • tackled an existing issue or discussed with a team member
  • left instructions on how to review the changes
  • targeted the main branch

Reviewers Checklist

All items are required.
Please add a note if the item is not applicable
and please add your handle next to the items reviewed
if you only reviewed selected items.

I have...

  • added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • confirmed all author checklist items have been addressed
  • confirmed that this PR does not change production code
  • reviewed content
  • tested instructions (if applicable)
  • confirmed all CI checks have passed

@songgaoye songgaoye requested review from a team as code owners July 24, 2025 10:08
@aljo242 aljo242 enabled auto-merge July 24, 2025 14:09
@aljo242 aljo242 added this pull request to the merge queue Jul 24, 2025
Merged via the queue into cosmos:main with commit 6085578 Jul 24, 2025
16 checks passed
@songgaoye songgaoye deleted the fix_p256_flakey_test branch July 24, 2025 15:01
zsystm pushed a commit to zsystm/evm that referenced this pull request Nov 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants