search

parallaxsecond / rust-tss-esapi

TSS 2.0 Enhanced System API (ESAPI) Rust wrapper
https://docs.rs/tss-esapi/
Apache License 2.0
85 stars 51 forks source link

Migrate from mbox to malloced #490

Closed kcking closed 6 months ago

kcking commented 7 months ago

Fixes #371.

Superhepper commented 7 months ago

Nice! This is something I have been looking at my self a couple of times.

Seems like the malloced crate have not been updated in 11 months. mbox have not been updated in 3 years.

I wonder what other options are out there. I would like to see a library that gets maintained and could provide the functionality that we have implemented here. Which is basically from FFI to owned with a possibility of adding a cleanup function.

kcking commented 7 months ago

The last commit and release of malloced was just under 1 year ago. As there are no outstanding issues, there is no evidence the crate is unmaintained. Unfortunately mbox has a couple of potential soundness issues and the author is unresponsive. Perhaps malloced could be considered as a short-term alternative, leaving open the possibility later for a crate that supports cleanup natively.

Superhepper commented 7 months ago

Yes, I would rather like a dependency that has no known issues and was updated recently then have a dependency to something that seams abandoned and with a maintainer that does not seem unresponsive. Maybe it would even be possible to add feature request to the malloced crate.

ionut-arm commented 7 months ago

I'm definitely for this move, seems like a great alternative for MBox

Superhepper commented 6 months ago

Could you please rebase this on the latest changes from main?

kcking commented 6 months ago

Could you please rebase this on the latest changes from main?

Done :)