mirrors/sprout
1
0
Fork 0
mirror of https://github.com/edera-dev/sprout.git synced 2025-12-19 12:30:17 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(framebuffer): add proper bounds checking for accessing a pixel

Browse Source
This commit is contained in:
Alex Zenla
2025-10-24 19:54:28 -07:00
parent 6cd502ef18
commit 0c2303d789
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/utils/framebuffer.rs
View File

@@ -31,6 +31,11 @@ impl Framebuffer {
/// Mutably acquires a pixel of the framebuffer at the specified `x` and `y` coordinate. /// Mutably acquires a pixel of the framebuffer at the specified `x` and `y` coordinate.
pub fn pixel(&mut self, x: usize, y: usize) -> Option<&mut BltPixel> { pub fn pixel(&mut self, x: usize, y: usize) -> Option<&mut BltPixel> {
// Verify that the coordinates are within the bounds of the framebuffer.
if x >= self.width || y >= self.height {
return None;
}
// Calculate the index of the pixel safely, returning None if it overflows. // Calculate the index of the pixel safely, returning None if it overflows.
let index = y.checked_mul(self.width)?.checked_add(x)?; let index = y.checked_mul(self.width)?.checked_add(x)?;
// Return the pixel at the index. If the index is out of bounds, this will return None. // Return the pixel at the index. If the index is out of bounds, this will return None.