safe-global / safe-smart-account

Safe allows secure management of blockchain assets.
https://safe.global
GNU Lesser General Public License v3.0
1.84k stars 907 forks source link

LZ's `sendFrom()` produces insufficient gas error in tx simulation #743

Closed RajaMBZ closed 5 months ago

RajaMBZ commented 6 months ago

The gas amount required by the sendfrom function is paid by the tx executor, not from the Safe itself. Hence, the check to see if there are enough funds in Safe to cover the LayerZero's fee in sendfrom shouldn't produce an error.

nlordell commented 6 months ago

Do you have a particular simulation we could look at to better understand what is going on?