Carlos López comments

Results 6 comments of


                                            Carlos López

ISISd (IPv4) LeakSanitizer: detected memory leaks

Fixed in #11656

Consider using reliable crash-resistant xml parser instead of xml-rs.

I tested this with the latest `xml-rs` version (v0.8.13) and the crash does not reproduce anymore. Consider updating the dependencies for this repository.

Consider using reliable crash-resistant xml parser instead of xml-rs.

Just FYI, I have tested this again and found 2 more panics in `xml-rs` through the `svg-hush` harness: - https://github.com/netvl/xml-rs/issues/225 (only with debug assertions enabled) - https://github.com/netvl/xml-rs/pull/226 (fix got merged)

Add fuzzing and a target for `DynamicAvx2Searcher`

No bugs found so far, no :)

Improvements to generated docs

Hi, could you review the changes? I believe the test failures are unrelated.

Arc::from(Box<T>) allocates and can panic in FallibleArc::try_new()

Hi, doesn't this apply to `Rc` as well? The code path looks very similar when doing `Rc::from(Box)` in this crate's `Rc::try_new`.