Audit found that the transferFrom function never deducts the allowance value

[michael7245]

eattheblocks/screencast/308-create-bep20-token-bsc/Token.sol

Line 30 in e749714

function transferFrom(address from, address to, uint value) public returns(bool) {

Hello,

We used this code for our BEP-20 token project as none of us really had any smart contract experience. Our project ended up getting a lot of traction and so we took the next step and got an audit from Certik. They raised a critical error with the TransferFrom function because the allowance value never gets deducted.

Obviously we don't blame you because this contract was used for learning purposes but I wanted to let you know just in case you had thought about this before and felt there was another way this error was accounted for. Thanks!

[michael7245]

@jklepatch