Stream: wasmtime

Topic: wmemcheck improvements


view this post on Zulip cobbal (Nov 20 2024 at 23:20):

I was recently tracking down a memory corruption bug in swift and found the wmemcheck tool to be very helpful. Since the runtime of swift is fairly large, it required some extra features to remove false positives. I was able to modify wmemcheck to be useful enough for the job.

I'd like to contribute the changes, many of which overlap with parts of issue https://github.com/bytecodealliance/wasmtime/issues/7037 . In particular, I needed to add the following:

The last 2 possibly should be hidden behind options, but I'm not familiar enough with the code base or how others use this tool to know what the right approach would be.

I also wasn't quite sure if this should be an issue, a PR, or a comment somewhere, so I'm asking here instead. My current WIP branch is at https://github.com/cobbal/wasmtime/tree/cobbal/wmemcheck-updates

This issue is meant to track what remains to make wmemcheck feature-complete and robust enough for practical use: Handle SIMD loads and stores (#6829) Hook calloc as well as existing malloc and fre...
A fast and secure runtime for WebAssembly. Contribute to cobbal/wasmtime development by creating an account on GitHub.

view this post on Zulip Chris Fallin (Nov 20 2024 at 23:23):

Please do submit a PR, possibly in draft status (but ping/request reviews at least from me)! This is very useful, thanks for pushing this forward and I'm glad it helped you debug some issues!

view this post on Zulip cobbal (Nov 20 2024 at 23:57):

Chris Fallin said:

Please do submit a PR, possibly in draft status (but ping/request reviews at least from me)! This is very useful, thanks for pushing this forward and I'm glad it helped you debug some issues!

Done! I don't seem to be able to request a review though. https://github.com/bytecodealliance/wasmtime/pull/9641

I was recently tracking down a memory corruption bug in swift and found the wmemcheck tool to be very helpful. Since the runtime of swift is fairly large, it required some extra features to remove ...

view this post on Zulip Chris Fallin (Nov 21 2024 at 00:00):

ah that's likely because you're not a project member, sorry; I'll assign myself and take a look


Last updated: Dec 23 2024 at 13:07 UTC