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

RUSTSEC-2022-0054: wee_alloc is Unmaintained #33

Closed
github-actions bot opened this issue Mar 4, 2023 · 4 comments · Fixed by #37
Closed

RUSTSEC-2022-0054: wee_alloc is Unmaintained #33

github-actions bot opened this issue Mar 4, 2023 · 4 comments · Fixed by #37

Comments

@github-actions
Copy link

github-actions bot commented Mar 4, 2023

wee_alloc is Unmaintained

Details
Status unmaintained
Package wee_alloc
Version 0.4.5
URL rustwasm/wee_alloc#107
Date 2022-05-11

Two of the maintainers have indicated that the crate may not be maintained.

The crate has open issues including memory leaks and may not be suitable for production use.

It may be best to switch to the default Rust standard allocator on wasm32 targets.

Last release seems to have been three years ago.

Possible Alternative(s)

The below list has not been vetted in any way and may or may not contain alternatives;

Honorable Mention(s)

The below may serve to educate on potential future alternatives:

See advisory page for additional details.

@mpalmer
Copy link
Owner

mpalmer commented Mar 5, 2023

This seems relevant to your interests, @bcheidemann. I'm inclined to just remove wee_alloc entirely and rely on the default allocator, unless you've got an alternate plan.

@bcheidemann
Copy link
Contributor

@mpalmer ouch! I didn't realise it was un-maintained 😬 The wasm-pack docs make reference to it so I assumed it would be ok. It's not really needed anyway so I can raise a PR to remove it.

@corneliusroemer
Copy link

@bcheidemann could you maybe open an issue on wasm-pack to indicate that wee_alloc should be removed from the docs so that others don't fall into the same trap? You know best where you found it mentioned there!

@bcheidemann
Copy link
Contributor

@corneliusroemer Thanks for reminding me :)

See rustwasm/wasm-pack#1258

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 a pull request may close this issue.

3 participants