Cryptographic Issues in Matrix’s Rust Library Vodozemac

32 points by dzwdz


kitkat

Interesting finds, but it would have been an easier read if he wasn't going out of his way to trash talk the Matrix team every other sentence (deserved or not).

edit: the vulnerability also seems much less severe than he's making it out to be.

val

Minor part of the article, but:

  1. #[cfg(fuzzing)] Bypasses MAC and Signature Verification

If you ever accidentally compile vodozemac with the fuzzing Cargo feature flag enabled, you’ve just disabled all security in your client.

is wrong. #[cfg(fuzzing)] is not conditional on enabling feature (which would be #[cfg(feature = "fuzzing")], it's conditional in running the code with cargo fuzz. Which is fine.

The Matrix team could have [...] used a Cargo feature flag in Cargo.toml instead of what they did here, but alas.

using a Cargo feature flag would introduce the vulnerability the article claims already exists.

Aks

Meanwhile I am quite interested in these type of posts, the constant shittalking is rather tiresome. Their blog, they can do as they wish of course, but as a casual observer this seems just excessive.

Having been a target of this kind of shittalking many times is probably why I feel this way.