fix(FeeVault): add zero address check in constructor

tac0turtle · claude · tac0turtle · commit cc05d655e9cc · 2025-12-08T13:59:10.000+01:00
Prevents deploying the contract with a zero address owner, which would permanently lock all administrative functions. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
diff --git a/contracts/src/FeeVault.sol b/contracts/src/FeeVault.sol
@@ -37,6 +37,7 @@ contract FeeVault {
     }
 
     constructor(address _owner) {
+        require(_owner != address(0), "FeeVault: owner is the zero address");
         owner = _owner;
         bridgeShareBps = 10000; // Default to 100% bridge
         emit OwnershipTransferred(address(0), _owner);

Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,7 @@ contract FeeVault {`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`constructor(address _owner) {`
	`40`	`+ require(_owner != address(0), "FeeVault: owner is the zero address");`
`40`	`41`	`owner = _owner;`
`41`	`42`	`bridgeShareBps = 10000; // Default to 100% bridge`
`42`	`43`	`emit OwnershipTransferred(address(0), _owner);`