Skip to content

fix: Fix out-of-bounds panic in some macros due to unhandled self_ref #16221

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

Merged
merged 1 commit into from
Dec 31, 2023
Merged

fix: Fix out-of-bounds panic in some macros due to unhandled self_ref #16221

merged 1 commit into from
Dec 31, 2023

Conversation

holly-hacker
Copy link
Contributor

Fixes #16200

I don't fully understand these changes, I just applied and tested the changes suggested in #16200 and they seem to fix the issue on both the repro and my original project.

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 31, 2023
@lnicola
Copy link
Member

lnicola commented Dec 31, 2023

@bors r+

@bors
Copy link
Contributor

bors commented Dec 31, 2023

📌 Commit c133309 has been approved by lnicola

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Dec 31, 2023

⌛ Testing commit c133309 with merge cf52c4b...

@bors
Copy link
Contributor

bors commented Dec 31, 2023

☀️ Test successful - checks-actions
Approved by: lnicola
Pushing cf52c4b to master...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Out-of-bounds panics in Worker thread
4 participants