First Flight #2: Puppy Raffle

Summary

changeFeeAddress should check for zero address

Vulnerability Details

there is no validation of the address being sent to changeFeeAddress. it could be set to the zero address accidentally

Impact

if changeFeeAddress changed the fee address to the zero address, fees would be lost instead of being sent to the owner's fee address.

Tools Used

VSCode

Recommendations

revert if the zero address is sent to the function. only the owner can call it but this extra check will prevent accidental loss of fees.