FuelLabs/fuels-rs

Add clippy check for overflowing arithmetic

Open

Aperta il 14 dic 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (1 assegnatario)Rust (43.211 star) (1360 fork)batch import
Securitygood first issuetech-debtv1.0

Descrizione

Our fuzzing efforts have mainly discovered unchecked arithmetic. We should add the following clippy check to our CI to disallow unchecked arithmetic in the future: https://rust-lang.github.io/rust-clippy/master/index.html#/arithmetic_side_effects

Guida contributor

Tech stack
rust
Dominio
ci cddeveloper experience
Tipo issue
chore
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
under 1 hour
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
blocked
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
RustCI configurationclippy
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50
Direzione di ricerca
The CI configuration is likely in a GitHub Actions workflow file (e.g., .github/workflows/ci.yml). The assignee may already be working on this issue. To proceed, review the current workflow and add the clippy flag ` D clippy::arithmetic side effects`. See the clippy documentation linked in the issue for details.