Skip to content

fix(napi): wrong drop impl for TypedArray::with_external_data - #2724

Merged
Brooooooklyn merged 1 commit into
mainfrom
06-23-fix_napi_wrong_drop_impl_for_typedarray_with_external_data
Jun 23, 2025
Merged

Brooooooklyn merged 1 commit into
mainfrom
06-23-fix_napi_wrong_drop_impl_for_typedarray_with_external_data

Conversation

@Brooooooklyn

@Brooooooklyn Brooooooklyn commented Jun 23, 2025

Copy link
Copy Markdown
Member

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Brooooooklyn
Brooooooklyn merged commit 8df3f94 into main Jun 23, 2025
@Brooooooklyn
Brooooooklyn deleted the 06-23-fix_napi_wrong_drop_impl_for_typedarray_with_external_data branch June 23, 2025 09:48
@overlookmotel

Copy link
Copy Markdown
Contributor

Amazing! Thank you. When you have time, would it be possible to do a release?

@Brooooooklyn

Copy link
Copy Markdown
Member Author

@overlookmotel will be released after I address this issue: #2727

@overlookmotel

overlookmotel commented Jun 24, 2025

Copy link
Copy Markdown
Contributor

Confirmed this fixed the issue.

Thank you for swift fix and for doing a release.

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.

Uint8Array::with_external_data results in allocation being freed on wrong side (napi-rs v3)

2 participants