search

Cyfrin / aderyn

Aderyn 🦜 Rust-based Solidity AST analyzer.
https://docs.cyfrin.io/
GNU General Public License v2.0
356 stars 51 forks source link

Wrongly assigned values to sarif report #558

Closed TilakMaddy closed 2 months ago

TilakMaddy commented 2 months ago

https://github.com/Cyfrin/aderyn/blob/96b423789ef36ae970d6ba975165a5daa2fcef5c/aderyn_core/src/report/sarif_printer.rs#L224-L228

The values Some(offset.parse().unwrap()) and Some(len.parse().unwrap()) are byte offset and byte length respectively

alexroan commented 2 months ago

This fix will be released in v0.1.4