Skip to content

Superfluous require guard on WithdrawOwnerEth() #84

Description

@maurelian

What is it?

CrowdSaleLib makes a check on the token balance, to determine eligibility to withdraw Ether.

https://github.com/Modular-Network/ethereum-libraries/blob/audits/CrowdsaleLib/IICOLib/truffle/contracts/CrowdsaleLib.sol#L204

Problem

I can't see a reason that contract's token balance would need to be checked in order to allow a withdrawal, and I can think of some problem's it might cause.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions