From 7f59516611b65dee598966034d526ff1e6e00cf5 Mon Sep 17 00:00:00 2001 From: Joel Klinghed Date: Mon, 15 Sep 2025 20:56:50 +0200 Subject: decompress: Return better io error for BUF_ERROR Use new MaxTooSmall. As the comment notes tho, it might be that we are lacking input as well, but until I figure out how to test for that case and determine the cause, lets at least return a more specific error. --- src/decompress_lzma.cc | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) (limited to 'src/decompress_lzma.cc') diff --git a/src/decompress_lzma.cc b/src/decompress_lzma.cc index 6baea18..299803d 100644 --- a/src/decompress_lzma.cc +++ b/src/decompress_lzma.cc @@ -63,9 +63,16 @@ class XzReader : public io::Reader { if (!in_eof_) buffer_->consume(stream_.next_in - rptr); } else { - return std::unexpected( - ret == LZMA_DATA_ERROR - ? io::ReadError::InvalidData : io::ReadError::Error); + switch (ret) { + case LZMA_DATA_ERROR: + return std::unexpected(io::ReadError::InvalidData); + case LZMA_BUF_ERROR: + // Tricky, could also be because of too little input. + // TODO: Improve logic here if we can + return std::unexpected(io::ReadError::MaxTooSmall); + default: + return std::unexpected(io::ReadError::Error); + } } return got; } -- cgit v1.3