tuxmain/qoi
1
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

Error::BadDecodingDataSize -> InputBufferTooSmall

Browse source
This commit is contained in:
Ivan Smirnov 2021-12-01 16:01:17 +00:00
parent d35fa000fa
commit d7c8346e39
2 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/decode.rs
View file

@ -36,7 +36,10 @@ where
Pixel<N>: SupportedChannels,
{
if data.len() < QOI_HEADER_SIZE + QOI_PADDING {
return Err(Error::BadDecodingDataSize { size: data.len() });
return Err(Error::InputBufferTooSmall {
size: data.len(),
required: QOI_HEADER_SIZE + QOI_PADDING,
});
}
let header = Header::from_bytes(unsafe {
// Safety: Header is a POD type and we have just checked that the data fits it

9
src/error.rs
View file

@ -2,14 +2,14 @@ use std::error::Error as StdError;
use std::fmt::{self, Display};
use std::result::Result as StdResult;
use crate::consts::{QOI_HEADER_SIZE, QOI_MAGIC, QOI_PADDING};
use crate::consts::QOI_MAGIC;
#[derive(Debug, Copy, Clone, PartialEq, Eq)]
pub enum Error {
InvalidChannels { channels: u8 },
EmptyImage { width: u32, height: u32 },
BadEncodingDataSize { size: usize, expected: usize },
BadDecodingDataSize { size: usize },
InputBufferTooSmall { size: usize, required: usize },
OutputBufferTooSmall { size: usize, required: usize },
InvalidMagic { magic: u32 },
// TODO: invalid colorspace
@ -29,9 +29,8 @@ impl Display for Error {
Self::BadEncodingDataSize { size, expected } => {
write!(f, "bad data size when encoding: {} (expected: {})", size, expected)
}
Self::BadDecodingDataSize { size } => {
let min_size = QOI_HEADER_SIZE + QOI_PADDING;
write!(f, "bad data size when decoding: {} (minimum required: {})", size, min_size)
Self::InputBufferTooSmall { size, required } => {
write!(f, "input buffer size too small: {} (minimum required: {})", size, required)
}
Self::OutputBufferTooSmall { size, required } => {
write!(f, "output buffer size too small: {} (minimum required: {})", size, required)