Skip to content

readme: Update test instructions#7

Merged
ananas-block merged 1 commit intodev_v4from
michal-dev_v4-update-readme
Jan 13, 2023
Merged

readme: Update test instructions#7
ananas-block merged 1 commit intodev_v4from
michal-dev_v4-update-readme

Conversation

@vadorovsky
Copy link
Contributor

  • Point to the master (and the last reliable commit) of Solana, instead of pointing to the fork.
  • Use relative paths for running the validator and programs.

* Point to the master (and the last reliable commit) of Solana, instead
  of pointing to the fork.
* Use relative paths for running the validator and programs.
@ananas-block ananas-block merged commit 4bc3cba into dev_v4 Jan 13, 2023
@vadorovsky vadorovsky deleted the michal-dev_v4-update-readme branch January 13, 2023 16:10
vadorovsky referenced this pull request in vadorovsky/light-protocol May 1, 2024
ananas-block added a commit that referenced this pull request Feb 6, 2026
Audit issue #7 (MEDIUM): is_valid_mint was only called inside
configure_compression_info, so non-compressible token accounts
could be initialized with an invalid mint. Move validation to
initialize_ctoken_account so it runs for all account types.
ananas-block added a commit that referenced this pull request Feb 12, 2026
Audit issue #7 (MEDIUM): is_valid_mint was only called inside
configure_compression_info, so non-compressible token accounts
could be initialized with an invalid mint. Move validation to
initialize_ctoken_account so it runs for all account types.
ananas-block added a commit that referenced this pull request Feb 12, 2026
* fix: validate mint for all token accounts, not just compressible

Audit issue #7 (MEDIUM): is_valid_mint was only called inside
configure_compression_info, so non-compressible token accounts
could be initialized with an invalid mint. Move validation to
initialize_ctoken_account so it runs for all account types.

* fix: read mint data once, pass decimals to configure_compression_info

* fix: tests

* fix tests

* fix tests

* fix tests

* fix js tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants