Skip to content

Permission bugs corrected in cw1-subkeys#354

Merged
ethanfrey merged 1 commit intomainfrom
subkeys-permission-bug-fixes
Jul 26, 2021
Merged

Permission bugs corrected in cw1-subkeys#354
ethanfrey merged 1 commit intomainfrom
subkeys-permission-bug-fixes

Conversation

@hashedone
Copy link
Copy Markdown
Contributor

No description provided.

@hashedone hashedone requested a review from maurolacy July 26, 2021 09:52
Copy link
Copy Markdown
Contributor

@ethanfrey ethanfrey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix.

The test for this is in #343 I assume?

}
}
s => panic!("Unsupported distribution message: {:?}", s),
_ => return Err(ContractError::UnsupportedMessage {}),
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@ethanfrey ethanfrey merged commit 7a6d61b into main Jul 26, 2021
@ethanfrey ethanfrey deleted the subkeys-permission-bug-fixes branch July 26, 2021 10:21
@hashedone
Copy link
Copy Markdown
Contributor Author

Yup, it is basically how I found the bug.

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