Owner can drain bridge via emergencyWithdrawBNB
Centralization Risk. The owner can unilaterally withdraw all native BNB from the bridge, including BNB escrowed to back wrapped tokens, with no solvency check, timelock, or multisig.
Description
Centralization Risk. The owner can unilaterally withdraw all native
BNB from the bridge via emergencyWithdrawBNB, including BNB escrowed to
back wrapped tokens. No solvency check, timelock, or multisig exists.
Recommendation
Consider adding a timelock, e.g. 48-hour delay publicly visible on-chain,
giving users time to unwrap and exit if they disagree with the withdrawal.
Alternatively, add a solvency check
(address(this).balance - wrappedBNBToken.totalSupply()) to limit
withdrawals to excess BNB only.
Resolution
YadaCoin, Confirmed.
Zealynx, Fixed.

