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

[GHSA-rv9v-r4vm-gj8x] Miniscript allows stack consumption #4719

Conversation

apoelstra
Copy link

Updates

  • Affected products
  • Description
  • Severity

Comments
Affected versions are wrong.

@apoelstra
Copy link
Author

I don't know why the form keeps changing the "severity" and I don't know how to properly specify the affected versions.

But right now this is a false positive. Can somebody please fix it?

@github-actions github-actions bot changed the base branch from main to apoelstra/advisory-improvement-4719 August 23, 2024 13:42
@darakian
Copy link
Contributor

Hey @apoelstra, ya I can fix that on our end. Any chance you have release notes or commits or something to support this change of versions?

@apoelstra
Copy link
Author

@darakian the CHANGELOG.md on each of the fixed versions mentions it, but I realize that there isn't anywhere that has all the versions at once. (We are not very good at forward-porting changelog entries. I will file an issue to do this if it's a prerequisite to fixing the advisory here.)

But meanwhile you can see the changelog entries invidiually for

Each of those is a short identical fix which updates the CHANGELOG, Cargo.toml version, and the actual code.

Copy link

github-actions bot commented Sep 9, 2024

👋 This pull request has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the Keep label to hold stale off permanently, or do nothing. If you do nothing this pull request will be closed eventually by the stale bot. Please see CONTRIBUTING.md for more policy details.

@github-actions github-actions bot added the Stale label Sep 9, 2024
@apoelstra
Copy link
Author

The "affected versions" on GHSA-rv9v-r4vm-gj8x are still incorrect.

@advisory-database advisory-database bot merged commit e384e72 into apoelstra/advisory-improvement-4719 Sep 9, 2024
2 checks passed
@advisory-database
Copy link
Contributor

Hi @apoelstra! Thank you so much for contributing to the GitHub Advisory Database. This database is free, open, and accessible to all, and it's people like you who make it great. Thanks for choosing to help others. We hope you send in more contributions in the future!

@advisory-database advisory-database bot deleted the apoelstra-GHSA-rv9v-r4vm-gj8x branch September 9, 2024 18:20
@darakian
Copy link
Contributor

darakian commented Sep 9, 2024

@apoelstra sorry about the delay this got away from me. Should be good now 👍

@apoelstra
Copy link
Author

@darakian great, thank you so much!

And I will remember this PR to crib syntax from, so that if this happens again I should be able to fix it myself :).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants