NFTBridge
60,000 USDC
View results
Submission Details
Severity: high
Invalid

Inconsistent Offset Handling at cairoStringPack in Cairo library

Hello ArkProject,

  • The offset is initially set to 0x20, which skips the first 32 bytes (usually the length prefix of a dynamic array in Solidity). However, this offset is used throughout the function without further checks. This could lead to incorrect memory access if the string does not conform to the expected format.

Mitigation Recommendations:

  • Add Boundary Checks: Ensure that offset does not exceed the bounds of strBytes and that memory accesses are safe.

Updates

Lead Judging Commences

n0kto Lead Judge 12 months ago
Submission Judgement Published
Invalidated
Reason: Lack of quality

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.