NestedRecords: Unnecessary variable in the Holding struct

[code423n4]

Handle

GreyArt

Vulnerability details

Impact

It is unnecessary to store the token variable in the Holding struct because the token is used as the key to access the Holding struct.

Recommended Mitigation Steps

Remove the token variable in the Holding struct.

/// @dev Info about assets stored in reserves
struct Holding {
  uint256 amount;
  bool isActive;
}