Use local variable in fundStream()

[code423n4]

Handle

gpersoon

Vulnerability details

Impact

The function fundStream() stores a calculated value in amount.
It saves some gas to use the already present variable "amt"

Proof of Concept

https://github.com/code-423n4/2021-11-streaming/blob/56d81204a00fc949d29ddd277169690318b36821/Streaming/src/Locke.sol#L417-L479

 function fundStream(uint112 amount) public lock {
   ...
        uint112 amt;
  ...
        amount = uint112(newBal - prevBal);   // cheaper to store in amt

Tools Used

Recommended Mitigation Steps

Use amt instead of amount and adapt the rest of the code accordingly.