40,000 USDC
View results
Submission Details
Severity: gas

Use `bytes.concat()` on bytes instead of `abi.encodePacked()` for clearer semantic meaning

Summary

Use bytes.concat() on bytes instead of abi.encodePacked() for clearer semantic meaning

Vulnerability Details

Starting with version 0.8.4, Solidity has the bytes.concat() function, which allows one to concatenate a list of bytes/strings, without extra padding. Using this function rather than abi.encodePacked() makes the intended operation more clear, leading to less reviewer confusion.

File: /src/EscrowFactory.sol
76: abi.encodePacked(

Impact

Informational

Tools Used

Manual

Recommendations

Use bytes.concat() on bytes instead of abi.encodePacked() for clearer semantic meaning

Support

FAQs

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