Skip to content

Commit

Permalink
Review feedback.
Browse files Browse the repository at this point in the history
  • Loading branch information
Lichtso committed Aug 15, 2024
1 parent 440bae1 commit 55578ff
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions proposals/0163-lift-cpi-caller-restriction.md
Original file line number Diff line number Diff line change
Expand Up @@ -79,18 +79,25 @@ dispatch
- Use the instruction **data**, not instruction **accounts**, to receive the
callee address as a parameter, in case they want a dynamic dispatch

Transaction building should append the required program accounts, which are not
passed as instruction accounts, at the end of the transaction accounts list.
How the dapps describe which callee prorgams they require to be present in the
transaction is explicitly left unspecified.

## Security Considerations

None.

## Backwards Compatibility

Programs remain unaffected the way they are, unless they want to profit from
this change. In that case they can do one of the following:

Existing programs, which have hard-coded the callee statically and only need it
as any instruction account to satisfy the constraint imposed by the runtime,
can be fed a placeholder like `NativeLoader1111111111111111111111111111111`, in
order not to shift the indices of the other instruction accounts.

All other existing programs, which dynamically call whatever is passed in a
specific instruction account will have to be updated and redeployed to benefit
from the lifting of the restriction. Otherwise they remain unaffected the way
they are.
from the lifting of the restriction.

0 comments on commit 55578ff

Please sign in to comment.