Skip to content

Commit 261b9ff

Browse files
authored
add semgrep rule to ensure no usage of vm.ffi or vm.tryFfi outside the process library (#15447)
1 parent c5df555 commit 261b9ff

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

.semgrep/rules/sol-rules.yaml

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -351,6 +351,19 @@ rules:
351351
- packages/contracts-bedrock/src/universal/OptimismMintableERC20.sol
352352
- packages/contracts-bedrock/src/universal/ReinitializableBase.sol
353353

354+
- id: sol-style-use-process-run
355+
languages: [solidity]
356+
severity: ERROR
357+
message: Use Process.run instead of vm.ffi or vm.tryFfi
358+
pattern-either:
359+
- pattern: |
360+
vm.ffi(...);
361+
- pattern: |
362+
vm.tryFfi(...);
363+
paths:
364+
exclude:
365+
- packages/contracts-bedrock/scripts/libraries/Process.sol
366+
354367
- id: sol-style-vm-env-only-in-config-sol
355368
languages: [solidity]
356369
severity: ERROR

0 commit comments

Comments
 (0)