Skip to content

Commit 6b57656

Browse files
authored
Update fma-operator-fee.md
1 parent f8ec0a4 commit 6b57656

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

security/fma-operator-fee.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -185,11 +185,11 @@ Below is what needs to be done before launch to reduce the chances of the above
185185

186186
## Audit Requirements
187187

188-
An audit has not been deemed necessary for the relatively simple changes to the SystemConfig contract, which has been reviewed by Base internal security team.
188+
An audit has not been deemed necessary on **L1** for the relatively simple changes to the SystemConfig contract, which has been reviewed by Base internal security team.
189189

190190
Indeed, the only addition is the `setOperatorFeeScalars` function, which is a setter function that updates the operator fee parameters and trigger an event. This function is callable by the SystemConfig owner only.
191191

192-
However, for the infrastructure code (op-geth/op-reth) and not the smart contracts: Since this is modifiying **the state transition**, I recommend an audit of this feature, however, this need to see with other member of evm-safety to know what they think etc).
192+
However, for the **clients code** (op-geth/op-reth) and not the smart contracts: Since this is modifiying **the state transition**, I recommend an audit of this feature, however, this need to see with other member of evm-safety to know what they think etc).
193193

194194

195195
Additionally, we are performing multi-client testing (op-geth and op-reth) and will run a multi-client devnet, so bugs in either will be quickly detected by consensus disagreements among them. So, we don't think the operator fee feature falls into the [Audit Framework](https://gov.optimism.io/t/op-labs-audit-framework-when-to-get-external-security-review-and-how-to-prepare-for-it/6864) Existential + Safety category.

0 commit comments

Comments
 (0)