|
function refundContributors() public ended() onlyOwner() { |
I checked the code again and realized there is glitch.
When you do a transfer(), execution flow is transfered to an external code. For example, If the address that you are refunding is a smart contract and it hangs or revert when receiving funds, you won´t be able to refund your users using this function.
You are safe because you have refund() and each user can go and get its funds back.
I am writing this issue just to let you know what can happen when using a loop and transfer.
eos/contracts/Auction.sol
Line 88 in 4a4929d
I checked the code again and realized there is glitch.
When you do a transfer(), execution flow is transfered to an external code. For example, If the address that you are refunding is a smart contract and it hangs or revert when receiving funds, you won´t be able to refund your users using this function.
You are safe because you have refund() and each user can go and get its funds back.
I am writing this issue just to let you know what can happen when using a loop and transfer.