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

[Merged by Bors] - fix: misalignment crash #259

Closed
wants to merge 9 commits into from

Conversation

sehz
Copy link
Collaborator

@sehz sehz commented Sep 26, 2023

resolves #255

The underlying issue was that the reference to wrap was removed just after the constructor was called. The code was there probably due to a workaround bug in the previous version of node.

May need to clean up reference to wrap when finalizer is invoked

@sehz
Copy link
Collaborator Author

sehz commented Sep 26, 2023

bors r+

bors bot pushed a commit that referenced this pull request Sep 26, 2023
resolves #255 

The underlying issue was that the reference to wrap was removed just after the constructor was called.  The code was there probably due to a workaround bug in the previous version of node. 

May need to clean up reference to wrap when finalizer is invoked
@bors
Copy link

bors bot commented Sep 26, 2023

Build failed:

@sehz
Copy link
Collaborator Author

sehz commented Sep 26, 2023

bors r+

bors bot pushed a commit that referenced this pull request Sep 26, 2023
resolves #255 

The underlying issue was that the reference to wrap was removed just after the constructor was called.  The code was there probably due to a workaround bug in the previous version of node. 

May need to clean up reference to wrap when finalizer is invoked
@bors
Copy link

bors bot commented Sep 26, 2023

Pull request successfully merged into master.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title fix: misalignment crash [Merged by Bors] - fix: misalignment crash Sep 26, 2023
@bors bors bot closed this Sep 26, 2023
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.

misaligned pointer ref with node version 18.14 or later
1 participant