Merge pull request #37 from zeppelinos/fix/28_check_transfers_return_…

…value Check transfers return value
zeppelinos · Apr 24, 2018 · 6327d83 · 6327d83
2 parents c128069 + 6d503e1
commit 6327d83
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/contracts/Kernel.sol b/contracts/Kernel.sol
@@ -105,7 +105,7 @@ contract Kernel is Migratable {
    */
   function unvouch(Release release, uint256 amount, bytes data) public whenRegistered(release) {
     vouches.unvouch(msg.sender, release, amount, data);
-    token.transfer(msg.sender, amount);
+    require(token.transfer(msg.sender, amount));
   }
 
   /**
@@ -134,6 +134,6 @@ contract Kernel is Migratable {
 
     uint256 vouchedAmount = amount.sub(developerPayout);
     vouches.vouch(voucher, release, vouchedAmount, data);
-    token.transfer(release.developer(), developerPayout);
+    require(token.transfer(release.developer(), developerPayout));
   }
 }
diff --git a/contracts/test/MockKernelV2.sol b/contracts/test/MockKernelV2.sol
@@ -15,7 +15,7 @@ contract MockKernelV2 is Kernel {
 
     uint256 vouchedAmount = amount.sub(developerPayout);
     vouches.vouch(voucher, release, vouchedAmount, data);
-    token.transfer(release.developer(), developerPayout);
+    require(token.transfer(release.developer(), developerPayout));
   }
 
 }