Merge pull request #33 from canterberry/security/ucac-dos-vulnerability

🔒 Fix DoS vulnerability in Credit Protocol contract
blockmason · Apr 19, 2018 · d91ba95 · d91ba95
2 parents e742d19 + 082e01f
commit d91ba95
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/contracts/CreditProtocol.sol b/contracts/CreditProtocol.sol
@@ -105,7 +105,7 @@ contract CreditProtocol is Ownable {
  return adjustedTxLevel;
  }
 
- function executeUcacTx(address _ucacContractAddr) public {
+ function executeUcacTx(address _ucacContractAddr) private {
  uint256 txLevelBeforeCurrentTx = currentTxLevel(_ucacContractAddr);
  uint256 txLevelAfterCurrentTx = txLevelBeforeCurrentTx + 10 ** 27 / txPerGigaTokenPerHour;
  require(ucacs[_ucacContractAddr].totalStakedTokens >= txLevelAfterCurrentTx);