add zero address check in all initializers

src/BrickedStrategy.sol

@@ -10,6 +10,8 @@ contract BrickedStrategy is BaseStrategy {
     /// @notice Proxies will set any non constant variable you declare as default value
     /// @dev add any extra changeable variable at end of initializer as shown
     function initialize(address _vault, address _want) public initializer {
+        require(_vault != address(0), "address 0 invalid");
+        require(_want != address(0), "address 0 invalid");
         __BaseStrategy_init(_vault);
         want = _want;
     }

src/CitadelToken.sol

@@ -24,6 +24,7 @@ contract CitadelToken is GlobalAccessControlManaged, ERC20Upgradeable {
         string memory _symbol,
         address _gac
     ) public initializer {
+        require(_gac != address(0), "address 0 invalid");
         __ERC20_init(_name, _symbol);
         __GlobalAccessControlManaged_init(_gac);
     }

src/Funding.sol

@@ -110,6 +110,12 @@ contract Funding is GlobalAccessControlManaged, ReentrancyGuardUpgradeable {
         address _citadelPerAssetOracle,
         uint256 _assetCap
     ) external initializer {
+        require(_gac != address(0), "address 0 invalid");
+        require(_citadel != address(0), "address 0 invalid");
+        require(_asset != address(0), "address 0 invalid");
+        require(_xCitadel != address(0), "address 0 invalid");
+        require(_saleRecipient != address(0), "address 0 invalid");
+        require(_citadelPerAssetOracle != address(0), "address 0 invalid");
         require(_saleRecipient != address(0), "Funding: 0 sale");
         require(_citadelPerAssetOracle != address(0), "Funding: 0 oracle");
 

src/GlobalAccessControl.sol

@@ -58,6 +58,7 @@ contract GlobalAccessControl is
         external
         initializer
     {
+        require(_initialContractGovernance != address(0), "address 0 invalid");
         __AccessControlEnumerable_init();
         __Pausable_init();
 

src/KnightingRound.sol

@@ -120,6 +120,7 @@ contract KnightingRound is
         address _guestlist,
         uint256 _tokenInLimit
     ) external initializer {
+        require(_globalAccessControl != address(0), "address 0 invalid");
         require(
             _saleStart >= block.timestamp,
             "KnightingRound: start date may not be in the past"

src/KnightingRoundGuestlist.sol

@@ -35,6 +35,7 @@ contract KnightingRoundGuestlist is GlobalAccessControlManaged {
      * `owner`.
      */
     function initialize(address _globalAccessControl) public initializer {
+        require(_globalAccessControl != address(0), "address 0 invalid");
         __GlobalAccessControlManaged_init(_globalAccessControl);
     }
 

src/SupplySchedule.sol

@@ -41,6 +41,7 @@ contract SupplySchedule is GlobalAccessControlManaged, DSTest {
     /// =======================
 
     function initialize(address _gac) public initializer {
+        require(_gac != address(0), "address 0 invalid");
         __GlobalAccessControlManaged_init(_gac);
         _setEpochRates();
     }