diff options
| author | David Oberhollenzer <david.oberhollenzer@sigma-star.at> | 2019-11-24 16:36:20 +0100 |
|---|---|---|
| committer | David Oberhollenzer <david.oberhollenzer@sigma-star.at> | 2019-11-24 16:47:49 +0100 |
| commit | 162142b7930248d83a77e8d8f4eba5896d5b267a (patch) | |
| tree | f65796333f80b4f710a19f5fcb6770d6aedbafe6 /lib/sqfs/comp/xz.c | |
| parent | dea3b64dd11a3a3391bd9d8f8c26f20392e7eb3c (diff) | |
Cleanup libsquashfs invalid argument error codes
- Add an explicit "you're holding it wrong" error code.
- Consistently return error codes and not have some special places
where -1 is returned.
Signed-off-by: David Oberhollenzer <david.oberhollenzer@sigma-star.at>
Diffstat (limited to 'lib/sqfs/comp/xz.c')
| -rw-r--r-- | lib/sqfs/comp/xz.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/sqfs/comp/xz.c b/lib/sqfs/comp/xz.c index fd0f70b..91d48da 100644 --- a/lib/sqfs/comp/xz.c +++ b/lib/sqfs/comp/xz.c @@ -144,7 +144,7 @@ static sqfs_s32 xz_comp_block(sqfs_compressor_t *base, const sqfs_u8 *in, size_t i; if (size >= 0x7FFFFFFF) - return 0; + return SQFS_ERROR_ARG_INVALID; ret = compress(xz, LZMA_VLI_UNKNOWN, in, size, out, outsize); if (ret < 0 || xz->flags == 0) @@ -184,7 +184,7 @@ static sqfs_s32 xz_uncomp_block(sqfs_compressor_t *base, const sqfs_u8 *in, (void)base; if (outsize >= 0x7FFFFFFF) - return 0; + return SQFS_ERROR_ARG_INVALID; ret = lzma_stream_buffer_decode(&memlimit, 0, NULL, in, &src_pos, size, |