Skip to content
This repository was archived by the owner on Jan 9, 2025. It is now read-only.

[KGA-14] [KGA-28] [KGA-74] [KGA-132] [KGA-92] fix: finalize dictionary in RIPEMD160 #1577

Merged

Conversation

obatirou
Copy link
Collaborator

@obatirou obatirou commented Nov 6, 2024

code-423n4/2024-09-kakarot-findings#54

code-423n4/2024-09-kakarot-findings#120

What is the current behavior?

Resolves #1561

What is the new behavior?

Finalize the dict and reassign start after reseting the dict.


This change is Reviewable

Copy link

Copy link

Copy link

Copy link

@obatirou obatirou changed the title [KGA-14] [KGA-28] [KGA-74] [KGA-132] fix: finalize dictionary in RIPEMD160 [KGA-14] [KGA-28] [KGA-74] [KGA-132] [KGA-92] fix: finalize dictionary in RIPEMD160 Nov 6, 2024
Copy link

@ClementWalter ClementWalter merged commit b8853b4 into kkrt-labs:main Nov 6, 2024
13 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: non-finalized dictionary in RIPEMD160 allows forging of output[1]
3 participants