rawloader icon indicating copy to clipboard operation
rawloader copied to clipboard

various adjustments for migrating to Rust 2021

Open marcograss opened this issue 3 years ago • 2 comments

  • Fixes all relevant clippy warnings so it's easier to keep using it in the future
  • code format standard
  • migrate to rust 2021

marcograss avatar Aug 05 '22 06:08 marcograss

Thanks for the interest and the work!

Migrating to rust 2021 and fixing clippy warnings both sound great but please don't reformat all the code.

pedrocr avatar Aug 08 '22 00:08 pedrocr

I also got a travis notification for another branch where you were fuzzing rawloader although that's not in a PR. Please note that rawloader is meant to be used with panic = unwind. If you're finding fuzzing issues with parsers that are panics that the library catches and handles gracefully that was meant to be that way. I've gone down the path of trying to add error catching for all eventualities in parsing and it's maddening. Since the rust compiler already does that automatically for all possible cases, I'd love it if I could just use it to turn panics into errors at compile time but unfortunately right now that's only possible dynamically.

The library has been pretty extensively fuzzed so new error cases should be hard to find but it's perfectly possible there are still cases left where there's either an infinite loop or a memory growing unbounded kind of situation. I need to get some better hardware to be able to run the fuzzing continuously.

pedrocr avatar Aug 08 '22 00:08 pedrocr