Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update some system files #89

Merged
merged 1 commit into from
Sep 29, 2022
Merged

Update some system files #89

merged 1 commit into from
Sep 29, 2022

Conversation

MiguelDD1
Copy link
Contributor

@MiguelDD1 MiguelDD1 commented Sep 29, 2022

It still not compiles, but it merges these files from polkadot-v0.9.29

@MiguelDD1 MiguelDD1 requested a review from rtyshko September 29, 2022 08:53
@MiguelDD1 MiguelDD1 self-assigned this Sep 29, 2022
@linear
Copy link

linear bot commented Sep 29, 2022

AVA-350 Add `CheckNonZeroSender` Signed Extension

Description

In some blockchains, the null address (a.k.a 0x0000…0) is used to burn/destroy tokens because nobody holds the associate private key.

However that is not true in substrate, where it is a sort of public/well known private key.

It means that someone sends some tokens/funds into the null address instead of burn them (natively), then anyone could use them, like transfer them into real wallet or use to spam the network with transactions.

Solution

Add CheckNonZeroSender to our Signed Extension in the Runtime. It will requires an substrate upgrade to branch > polkadot-v0.9.16 at least. The latest branch is polkadot-v0.9.29 ATM.

Reference:

@rtyshko rtyshko merged commit ceeaf27 into rtyshko/ava-350 Sep 29, 2022
@Leouarz Leouarz deleted the miguel/ava-350 branch September 13, 2023 13:30
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