Giveth / vaultcontroller

Vault Architecture
GNU General Public License v3.0
0 stars 3 forks source link

Unreachable condition in checkMainTransfer #39

Open adria0 opened 7 years ago

adria0 commented 7 years ago

in checkMainTransfer() the check

if (canceled) return false;

is never reached, since checkMainTransfer is only called from topUpVault() and sendToAuthorizedRecipient() that checks for this condition using notCanceled modifier.