The parameter i lacks of sufficient checks and used directly in get_unchecked under safe public function, which might cause memory risks.
|
pub fn length_of_array(&self, i: usize) -> usize { |
|
let current = unsafe { *self.offsets.get_unchecked(i) }; |
|
let next = unsafe { *self.offsets.get_unchecked(i + 1) }; |
|
next - current |
|
} |
In Rust, we shoud not face memory issues if merely use safe function API.
Suggestion:
- add sufficient check
- add unsafe to the function declaration to notify the developer
The parameter i lacks of sufficient checks and used directly in get_unchecked under safe public function, which might cause memory risks.
trk-io/src/array_sequence.rs
Lines 162 to 166 in 6f78936
In Rust, we shoud not face memory issues if merely use safe function API.
Suggestion: