Skip to content

fix: audit issue 1#867

Closed
rflechtner wants to merge 18 commits intodevelopfrom
audit-issue-1
Closed

fix: audit issue 1#867
rflechtner wants to merge 18 commits intodevelopfrom
audit-issue-1

Conversation

@rflechtner
Copy link
Contributor

fixes https://github.com/KILTprotocol/ticket/issues/3796

POC for restricting curve parameters on pool creation. Makes a dry run to check that a mint to max supply would not overflow.

Checklist:

  • I have verified that the code works
    • No panics! (checked arithmetic ops, no indexing array[3] use get(3), ...)
  • I have verified that the code is easy to understand
    • If not, I have left a well-balanced amount of inline comments
  • I have left the code in a better state
  • I have documented the changes (where applicable)
    • Either PR or Ticket to update the Docs
    • Link the PR/Ticket here

@rflechtner rflechtner changed the base branch from develop to bonded-coins-audit-fixes March 3, 2025 17:24
rflechtner and others added 11 commits March 6, 2025 09:20
…mum (#866)

Adopts the recommended resolution of audit issue #2.
## fixes adit issue 8

Disallows re-using the same token symbol or name. Empty names and
symbols are always allowed.
## fixes audit issue 13

Disallows manager changes after a pool has been moved to refunding or
destroying.
rflechtner and others added 4 commits March 14, 2025 10:14
## fixes KILTProtocol/ticket#3813

Following recommendations by OAK, this adds another event to be emitted on calling `reset_team`. This is emitted in addition to the events emitted by the assets pallet itself.
## re/ KILTProtocol/ticket#3800

Addresses potential centralisation concerns around excessive manager permissions.
If set to `false` the reset_team() transaction cannot be used on this pool, 
in which case the manager has much more limited privileges (locking/unlocking and
initiating refunding). This flag can only be set on pool creation and not changed.
Base automatically changed from bonded-coins-audit-fixes to develop April 30, 2025 10:46
@Ad96el
Copy link
Contributor

Ad96el commented May 6, 2025

@rflechtner I think we can close this?

@rflechtner
Copy link
Contributor Author

@rflechtner I think we can close this?

Oh, yes

@rflechtner rflechtner closed this May 7, 2025
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