Summary

The function IBridge(bridge).withdraw(msg.sender, ethAmount) at line 112 returns a value that is not stored or used in the BridgeRouterFacet.sol contract.

Vulnerability Details

The withdrawal function returns a value that is not captured or tracked in the code.

Impact

This could lead to a potential loss of important information or state changes that are not monitored.

Tools Used

Manul reviwe and writing report with help of chatgpt

Recommendations

It is recommended to store the return value of the function call in a local or state variable. This ensures that any information returned from the function call is not lost and can be used or checked later in the code if necessary. This is particularly important when interacting with external contracts, as return values may contain vital status codes or results that need handling.