Steadefi

Vulnerability Details

In emergencyPause and emergencyResume the executionFee is set to msg.value when the function is not payable (which sets the value to 0 each time these functions are executed)

To fix the functions, the execution fee should be passed as an explicit parameter to the emergencyPause and emergencyResume functions. This will ensure that the execution fee is always set to the correct value, regardless of the value of msg.value.

Impact

executionFee is always set to 0.

Tools Used

Manual

Recommendations

Set an explicit fee parameter to the emergencyPause and emergencyResume functions