Summary

No fees sent to the fee address in the PuppyRaffle::selectWinner() function

Vulnerability Details

The selectWinner function doesn't call the PuppyRaffle::withdrawFees() function to send eth to fee address, this will lead to accumulation of funds in the PuppyRaffle contract and it will remain locked in to the contract.

Impact

Locking of funds in the contract.

Tools Used

Manual Review

Recommendations

To call the withdrawFees() function in the selectWinner() function after sending eth to winner.