capstone-rust / capstone-rs

high-level Capstone system bindings for Rust
220 stars 78 forks source link

Valgrind failures in CI #133

Closed tmfink closed 2 years ago

tmfink commented 2 years ago

Regular Valgrind tests fail in master branch. This could be a false positive.

https://github.com/capstone-rust/capstone-rs/runs/6753889756?check_suite_focus=true

clubby789 commented 2 years ago

Seems to be caused by https://stdrs.dev/nightly/x86_64-unknown-linux-gnu/src/std/sys/unix/fs.rs.html#157-160. The stdlib calls statx will a NULL pointer to check if the syscall is available. It seems to have been fixed in upstream Valgrind https://sourceware.org/git/?p=valgrind.git;a=commit;h=2a7d3ae768f9e5b29acd5cb743c3fb13640a391c