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

[KGA-31] [KGA-130] [KGA-138] fix: felt_to_bytes_little loop stop condition #1573

Merged

Conversation

obatirou
Copy link
Collaborator

@obatirou obatirou commented Nov 4, 2024

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

What is the current behavior?

Resolves #1563

What is the new behavior?

The loop will stop at 31 bytes


This change is Reviewable

@ClementWalter ClementWalter changed the title fix: felt_to_bytes_little loop stop condition [KGA-31] fix: felt_to_bytes_little loop stop condition Nov 4, 2024
@ClementWalter ClementWalter merged commit f599f71 into kkrt-labs:main Nov 4, 2024
13 checks passed
@obatirou obatirou changed the title [KGA-31] fix: felt_to_bytes_little loop stop condition [KGA-31] [KGA-130] [KGA-138] fix: felt_to_bytes_little loop stop condition Nov 5, 2024
Copy link

Copy link

QA Report

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: Underconstrained output in felt_to_bytes_little() allows contract deployments to incorrect addresses[1]
2 participants