L-05 [Oval] PermissionProxy.execute Will Fail if Non-Zero value Is Provided

[Reinis-FRP]

Addresses audit issue: L-05 [Oval] PermissionProxy.execute Will Fail if Non-Zero value Is Provided

The PermissionProxy contract is intended to be used as a proxy for calls that unlock new
prices reported as a part of the MEV-share auction flow. It allows multiple different accounts to
unlock new prices by performing calls to the controllers on their behalf. Calls may only be
performed by authorized accounts and may be executed by invoking the execute function.
Apart from the call target and the call data, it is possible to also specify the call value. However,
the PermissionProxy contract has no function allowing it to receive the native value. This
means that it is not possible for users to execute any call involving non-zero native value as all
such calls would revert.

Consider making the execute function payable or removing the value argument if the
PermissionProxy contract is not expected to make any calls with non-zero native value.

This implements the recommended fix by removing the value argument from PermissionProxy.

[linear]

UMA-2650 L-05 [Oval] PermissionProxy.execute Will Fail if Non-Zero value Is Provided

[chrismaree]

nice. I think this is easier & safer

[md0x]

LGTM