Fjord Token Staking

Summary

A redundancy in the inheritance structure, where both ERC20 and ERC20Burnable were being inherited, despite ERC20Burnable already inheriting from ERC20.

Vulnerability Details

The following contract was defined as contract FjordPoints is ERC20, ERC20Burnable {}, leading to unnecessary duplication of inherited properties and methods from ERC20.

• FjordPoints.sol#L14

Impact

This redundant inheritance increases code complexity without adding any functional benefit, potentially leading to confusion and harder maintenance.

Tools Used

• Manual review

Recommendations

Remove the redundant inheritance. Modify FjordPoints to inherit only from ERC20Burnable, as it already includes ERC20 functionalities. The correct declaration should be: contract FjordPoints is ERC20Burnable {}.