What does this pull request do? Explain your changes. (required)
This is to disallow the *For functions to be called with the zero address
as the owner, which is effectively a buggy way of burning tokens. This also
fixes the BondingVotes behavior to not allow the zero address to ever have
any voting power, as specified in the corresponding EIP.
Specific updates (required)
Make bondForWithHint disallow bonding for address(0)
Make transferBond disallow transferring bond to address(0)
How did you test each of these updates (required)yarn test
What does this pull request do? Explain your changes. (required) This is to disallow the
*Forfunctions to be called with the zero address as the owner, which is effectively a buggy way of burning tokens. This also fixes theBondingVotesbehavior to not allow the zero address to ever have any voting power, as specified in the corresponding EIP.Specific updates (required)
bondForWithHintdisallow bonding foraddress(0)transferBonddisallow transferring bond toaddress(0)How did you test each of these updates (required)
yarn testDoes this pull request close any open issues? Fixes https://github.com/code-423n4/2023-08-livepeer-findings/issues/190
Checklist:
yarn testpass