Skip to content

Conversation

@nicholaspai
Copy link
Member

From Audit

Consider correcting the following typographical errors:
In BondToken.sol :
Line 11, "rootBundleProposer()" should be "rootBundleProposal()".
Line 21: "permissiong" should be "permissioning".
In EIP712CrossChainUpgradeable.sol :
Line 83: "its always" should be "it's always".
In MultiCallerUpgradeable.sol :
Line 5: "@title MockSpokePool" should be "@title MultiCallerUpgradeable".
In Optimism_Adapter.sol :
Line 23: "its only" should be "it's only".
In Polygon_SpokePool.sol :
Line 51: the sentence "See" is incomplete.
Line 139: the sentence is malformed.
In SpokePool.sol :
Line 567: "speedUpRelay()" should be "speedUpDeposit()".
Line 1177: "its always" should be "it's always".
In Succinct_Adapter.sol :
Line 27: "destinatipn" should be "destination".
Line 27: "the message.." should be "the message.".
In Succinct_SpokePool.sol :
Line 22: "callValidated" should be "adminCallValidated".
Line 24: "callValidated" should be "adminCallValidated".
Line 35: "callValidated" should be "adminCallValidated".

## From Audit

Consider correcting the following typographical errors:
In BondToken.sol :
Line 11, "rootBundleProposer()" should be "rootBundleProposal()".
Line 21: "permissiong" should be "permissioning".
In EIP712CrossChainUpgradeable.sol :
Line 83: "its always" should be "it's always".
In MultiCallerUpgradeable.sol :
Line 5: "@title MockSpokePool" should be "@title MultiCallerUpgradeable".
In Optimism_Adapter.sol :
Line 23: "its only" should be "it's only".
In Polygon_SpokePool.sol :
Line 51: the sentence "See" is incomplete.
Line 139: the sentence is malformed.
In SpokePool.sol :
Line 567: "speedUpRelay()" should be "speedUpDeposit()".
Line 1177: "its always" should be "it's always".
In Succinct_Adapter.sol :
Line 27: "destinatipn" should be "destination".
Line 27: "the message.." should be "the message.".
In Succinct_SpokePool.sol :
Line 22: "callValidated" should be "adminCallValidated".
Line 24: "callValidated" should be "adminCallValidated".
Line 35: "callValidated" should be "adminCallValidated".
@nicholaspai nicholaspai added the OZ Audit - March 2023 Resolves issue discovered in March 2023 OZ Audit label Mar 15, 2023
Copy link
Contributor

@mrice32 mrice32 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@nicholaspai nicholaspai merged commit 43a0ea7 into master Mar 17, 2023
@nicholaspai nicholaspai deleted the audit2/n04 branch March 17, 2023 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OZ Audit - March 2023 Resolves issue discovered in March 2023 OZ Audit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants