Skip to content

Commit

Permalink
Fix audittens I04 (#39)
Browse files Browse the repository at this point in the history
  • Loading branch information
StanislavBreadless authored Nov 29, 2024
1 parent 9016bd3 commit 6bd1fb6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions l1-contracts/contracts/bridge/L1Nullifier.sol
Original file line number Diff line number Diff line change
Expand Up @@ -613,8 +613,8 @@ contract L1Nullifier is IL1Nullifier, ReentrancyGuard, Ownable2StepUpgradeable,
_erc20Metadata: new bytes(0)
});
} else if (bytes4(functionSignature) == IAssetRouterBase.finalizeDeposit.selector) {
// The data is expected to be at least 36 bytes long to contain assetId.
require(_l2ToL1message.length >= 36, "L1N: wrong msg len"); // wrong message length
// The data is expected to be at least 68 bytes long to contain assetId.
require(_l2ToL1message.length >= 68, "L1N: wrong msg len"); // wrong message length
// slither-disable-next-line unused-return
(, offset) = UnsafeBytes.readUint256(_l2ToL1message, offset); // originChainId, not used for L2->L1 txs
(assetId, offset) = UnsafeBytes.readBytes32(_l2ToL1message, offset);
Expand Down

0 comments on commit 6bd1fb6

Please sign in to comment.