Sablier

Summary

L-5: Event is missing indexed fields

Vulnerability Details

• Found in src/interfaces/ISablierV2Lockup.sol Line: 48

  event SetNFTDescriptor()

Impact

Low

Tools Used

Aderyn

Recommendations

Index event fields make the field more quickly accessible to off-chain tools that parse events. However, note that each index field costs extra gas during emission, so it's not necessarily best to index the maximum allowed per event (three fields). Each event should use three indexed fields if there are three or more fields, and gas usage is not particularly of concern for the events in question. If there are fewer than three fields, all of the fields should be indexed.