fix: slither warnings

Author: Luisfc68

contracts/interfaces/ICollateralManagement.sol

@@ -30,14 +30,14 @@ interface ICollateralManagement {
         address punisher,
         Quotes.PegInQuote calldata quote,
         bytes32 quoteHash
-    ) external returns (uint256);
+    ) external;
     function addPegOutCollateralTo(address addr) external payable;
     function addPegOutCollateral() external payable;
     function slashPegOutCollateral(
         address punisher,
         Quotes.PegOutQuote calldata quote,
         bytes32 quoteHash
-    ) external returns (uint256);
+    ) external;
 
     function getPegInCollateral(address addr) external view returns (uint256);
     function getPegOutCollateral(address addr) external view returns (uint256);

contracts/split/CollateralManagement.sol

@@ -95,7 +95,7 @@ contract CollateralManagementContract is
         address punisher,
         Quotes.PegInQuote calldata quote,
         bytes32 quoteHash
-    ) external onlyRole(COLLATERAL_SLASHER) returns (uint256) {
+    ) external onlyRole(COLLATERAL_SLASHER) {
         uint penalty = _min(
             quote.penaltyFee,
             _pegInCollateral[quote.liquidityProviderRskAddress]
@@ -104,7 +104,6 @@ contract CollateralManagementContract is
         uint256 punisherReward = (penalty * rewardPercentage) / 100;
         _rewards[punisher] += punisherReward;
         emit Penalized(quote.liquidityProviderRskAddress, quoteHash, Flyover.ProviderType.PegIn, penalty, punisherReward);
-        return penalty;
     }
 
     function addPegOutCollateralTo(address addr) external onlyRole(COLLATERAL_ADDER) payable {
@@ -119,7 +118,7 @@ contract CollateralManagementContract is
         address punisher,
         Quotes.PegOutQuote calldata quote,
         bytes32 quoteHash
-    ) external onlyRole(COLLATERAL_SLASHER) returns (uint256) {
+    ) external onlyRole(COLLATERAL_SLASHER) {
         uint penalty = _min(
             quote.penaltyFee,
             _pegOutCollateral[quote.lpRskAddress]
@@ -128,7 +127,6 @@ contract CollateralManagementContract is
         uint256 punisherReward = (penalty * rewardPercentage) / 100;
         _rewards[punisher] += punisherReward;
         emit Penalized(quote.lpRskAddress, quoteHash, Flyover.ProviderType.PegOut, penalty, punisherReward);
-        return penalty;
     }
 
     function getMinCollateral() external view returns (uint) {

contracts/split/FlyoverDiscoveryFull.sol

@@ -64,12 +64,14 @@ contract FlyoverDiscoveryFull is
         address owner,
         uint48 initialDelay,
         uint minCollateral,
-        uint resignDelayInBlocks
+        uint resignDelayInBlocks,
+        uint rewardPercentage_
     ) public initializer {
         __AccessControlDefaultAdminRules_init(initialDelay, owner);
         __ReentrancyGuard_init();
         _minCollateral = minCollateral;
         _resignDelayInBlocks = resignDelayInBlocks;
+        rewardPercentage = rewardPercentage_;
     }
 
     function register(
@@ -336,7 +338,7 @@ contract FlyoverDiscoveryFull is
         address punisher,
         Quotes.PegInQuote calldata quote,
         bytes32 quoteHash
-    ) external onlyRole(COLLATERAL_SLASHER) returns(uint256) {
+    ) external onlyRole(COLLATERAL_SLASHER) {
         uint penalty = _min(
             quote.penaltyFee,
             _pegInCollateral[quote.liquidityProviderRskAddress]
@@ -345,14 +347,13 @@ contract FlyoverDiscoveryFull is
         uint256 punisherReward = (penalty * rewardPercentage) / 100;
         _rewards[punisher] += punisherReward;
         emit Penalized(quote.liquidityProviderRskAddress, quoteHash, Flyover.ProviderType.PegIn, penalty, punisherReward);
-        return penalty;
     }
 
     function slashPegOutCollateral(
         address punisher,
         Quotes.PegOutQuote calldata quote,
         bytes32 quoteHash
-    ) external onlyRole(COLLATERAL_SLASHER) returns (uint256) {
+    ) external onlyRole(COLLATERAL_SLASHER) {
         uint penalty = _min(
             quote.penaltyFee,
             _pegOutCollateral[quote.lpRskAddress]
@@ -361,7 +362,6 @@ contract FlyoverDiscoveryFull is
         uint256 punisherReward = (penalty * rewardPercentage) / 100;
         _rewards[punisher] += punisherReward;
         emit Penalized(quote.lpRskAddress, quoteHash, Flyover.ProviderType.PegOut, penalty, punisherReward);
-        return penalty;
     }
 
     function _min(uint a, uint b) private pure returns (uint) {