2023-04-21 21:58:55 +00:00
|
|
|
#![allow(unused)]
|
|
|
|
#![warn(clippy::missing_asserts_for_indexing)]
|
|
|
|
|
|
|
|
// ok
|
|
|
|
fn sum_with_assert(v: &[u8]) -> u8 {
|
|
|
|
assert!(v.len() > 4);
|
|
|
|
v[0] + v[1] + v[2] + v[3] + v[4]
|
|
|
|
}
|
|
|
|
|
|
|
|
// ok
|
|
|
|
fn sum_with_assert_other_way(v: &[u8]) -> u8 {
|
|
|
|
assert!(5 <= v.len());
|
|
|
|
v[0] + v[1] + v[2] + v[3] + v[4]
|
|
|
|
}
|
|
|
|
|
|
|
|
// ok
|
|
|
|
fn sum_with_assert_ge(v: &[u8]) -> u8 {
|
|
|
|
assert!(v.len() >= 5);
|
|
|
|
v[0] + v[1] + v[2] + v[3] + v[4]
|
|
|
|
}
|
|
|
|
|
|
|
|
// ok
|
|
|
|
fn sum_with_assert_ge_other_way(v: &[u8]) -> u8 {
|
|
|
|
assert!(4 < v.len());
|
|
|
|
v[0] + v[1] + v[2] + v[3] + v[4]
|
|
|
|
}
|
|
|
|
|
|
|
|
fn sum_with_assert_lt(v: &[u8]) -> u8 {
|
|
|
|
assert!(v.len() > 4);
|
|
|
|
v[0] + v[1] + v[2] + v[3] + v[4]
|
2023-08-31 16:42:27 +00:00
|
|
|
//~^ ERROR: indexing into a slice multiple times with an `assert` that does not cover the
|
2023-04-21 21:58:55 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
fn sum_with_assert_le(v: &[u8]) -> u8 {
|
|
|
|
assert!(v.len() > 4);
|
|
|
|
v[0] + v[1] + v[2] + v[3] + v[4]
|
2023-08-31 16:42:27 +00:00
|
|
|
//~^ ERROR: indexing into a slice multiple times with an `assert` that does not cover the
|
2023-04-21 21:58:55 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
fn sum_with_incorrect_assert_len(v: &[u8]) -> u8 {
|
|
|
|
assert!(v.len() > 4);
|
|
|
|
v[0] + v[1] + v[2] + v[3] + v[4]
|
2023-08-31 16:42:27 +00:00
|
|
|
//~^ ERROR: indexing into a slice multiple times with an `assert` that does not cover the
|
2023-04-21 21:58:55 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
fn sum_with_incorrect_assert_len2(v: &[u8]) -> u8 {
|
|
|
|
assert!(v.len() > 4);
|
|
|
|
v[0] + v[1] + v[2] + v[3] + v[4]
|
2023-08-31 16:42:27 +00:00
|
|
|
//~^ ERROR: indexing into a slice multiple times with an `assert` that does not cover the
|
2023-04-21 21:58:55 +00:00
|
|
|
}
|
|
|
|
|
|
|
|
// ok, don't lint for single array access
|
|
|
|
fn single_access(v: &[u8]) -> u8 {
|
|
|
|
v[0]
|
|
|
|
}
|
|
|
|
|
|
|
|
// ok
|
|
|
|
fn subslice_ok(v: &[u8]) {
|
|
|
|
assert!(v.len() > 3);
|
|
|
|
let _ = v[0];
|
|
|
|
let _ = v[1..4];
|
|
|
|
}
|
|
|
|
|
|
|
|
fn subslice_bad(v: &[u8]) {
|
|
|
|
assert!(v.len() > 3);
|
|
|
|
let _ = v[0];
|
2023-08-31 16:42:27 +00:00
|
|
|
//~^ ERROR: indexing into a slice multiple times with an `assert` that does not cover the
|
2023-04-21 21:58:55 +00:00
|
|
|
let _ = v[1..4];
|
|
|
|
}
|
|
|
|
|
|
|
|
// ok
|
|
|
|
fn subslice_inclusive_ok(v: &[u8]) {
|
|
|
|
assert!(v.len() > 4);
|
|
|
|
let _ = v[0];
|
|
|
|
let _ = v[1..=4];
|
|
|
|
}
|
|
|
|
|
|
|
|
fn subslice_inclusive_bad(v: &[u8]) {
|
|
|
|
assert!(v.len() > 4);
|
|
|
|
let _ = v[0];
|
2023-08-31 16:42:27 +00:00
|
|
|
//~^ ERROR: indexing into a slice multiple times with an `assert` that does not cover the
|
2023-04-21 21:58:55 +00:00
|
|
|
let _ = v[1..=4];
|
|
|
|
}
|
|
|
|
|
|
|
|
fn index_different_slices_ok(v1: &[u8], v2: &[u8]) {
|
|
|
|
assert!(v1.len() > 12);
|
|
|
|
assert!(v2.len() > 15);
|
|
|
|
let _ = v1[0] + v1[12];
|
|
|
|
let _ = v2[5] + v2[15];
|
|
|
|
}
|
|
|
|
|
|
|
|
fn index_different_slices_wrong_len(v1: &[u8], v2: &[u8]) {
|
|
|
|
assert!(v1.len() > 12);
|
|
|
|
assert!(v2.len() > 15);
|
|
|
|
let _ = v1[0] + v1[12];
|
2023-08-31 16:42:27 +00:00
|
|
|
//~^ ERROR: indexing into a slice multiple times with an `assert` that does not cover the
|
2023-04-21 21:58:55 +00:00
|
|
|
let _ = v2[5] + v2[15];
|
2023-08-31 16:42:27 +00:00
|
|
|
//~^ ERROR: indexing into a slice multiple times with an `assert` that does not cover the
|
2023-04-21 21:58:55 +00:00
|
|
|
}
|
|
|
|
fn index_different_slices_one_wrong_len(v1: &[u8], v2: &[u8]) {
|
|
|
|
assert!(v1.len() > 12);
|
|
|
|
assert!(v2.len() > 15);
|
|
|
|
let _ = v1[0] + v1[12];
|
2023-08-31 16:42:27 +00:00
|
|
|
//~^ ERROR: indexing into a slice multiple times with an `assert` that does not cover the
|
2023-04-21 21:58:55 +00:00
|
|
|
let _ = v2[5] + v2[15];
|
|
|
|
}
|
|
|
|
|
|
|
|
fn side_effect() -> &'static [u8] {
|
|
|
|
&[]
|
|
|
|
}
|
|
|
|
|
|
|
|
fn index_side_effect_expr() {
|
|
|
|
let _ = side_effect()[0] + side_effect()[1];
|
|
|
|
}
|
|
|
|
|
|
|
|
// ok, single access for different slices
|
|
|
|
fn index_different_slice_in_same_expr(v1: &[u8], v2: &[u8]) {
|
|
|
|
let _ = v1[0] + v2[1];
|
|
|
|
}
|
|
|
|
|
|
|
|
fn main() {}
|