diff options
author | Jia Tan <jiat0218@gmail.com> | 2023-12-06 18:30:25 +0800 |
---|---|---|
committer | Jia Tan <jiat0218@gmail.com> | 2023-12-07 20:06:57 +0800 |
commit | ee2f48350099201694a7586e41d7aa2f09fc74da (patch) | |
tree | 35ce97c6b55eeda46cb843195c176a400f6b63a5 /tests/ossfuzz/fuzz_common.h | |
parent | Tests: Add fuzz_encode_stream ossfuzz target. (diff) | |
download | xz-ee2f48350099201694a7586e41d7aa2f09fc74da.tar.xz |
Tests: Minor cleanups to OSS-Fuzz files.
Most of these fixes are small typos and tweaks. A few were caused by bad
advice from me. Here is the summary of what is changed:
- Author line edits
- Small comment changes/additions
- Using the return value in the error messages in the fuzz targets'
coder initialization code
- Removed fuzz_encode_stream.options. This set a max length, which may
prevent some worthwhile code paths from being properly exercised.
- Removed the max_len option from fuzz_decode_stream.options for the
same reason as fuzz_encode_stream. The alone decoder fuzz target still
has this restriction.
- Altered the dictionary contents for fuzz_lzma.dict. Instead of keeping
the properties static and varying the dictionary size, the properties
are varied and the dictionary size is kept small. The dictionary size
doesn't have much impact on the code paths but the properties do.
Closes: https://github.com/tukaani-project/xz/pull/73
Diffstat (limited to '')
-rw-r--r-- | tests/ossfuzz/fuzz_common.h | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/tests/ossfuzz/fuzz_common.h b/tests/ossfuzz/fuzz_common.h index ce3f9345..14742f2e 100644 --- a/tests/ossfuzz/fuzz_common.h +++ b/tests/ossfuzz/fuzz_common.h @@ -1,9 +1,10 @@ /////////////////////////////////////////////////////////////////////////////// // -/// \file fuzz_decode_auto.c -/// \brief Fuzz test program for liblzma lzma_auto_decoder() +/// \file fuzz_common.h +/// \brief Common macros and functions needed by the fuzz targets // -// Author: Maksym Vatsyk +// Authors: Maksym Vatsyk +// Lasse Collin // // This file has been put into the public domain. // You can do whatever you want with this file. @@ -21,13 +22,12 @@ #define MEM_LIMIT (300 << 20) // 300 MiB -// Output buffer for decompressed data. This is write only; nothing cares -// about the actual data written here. -static uint8_t outbuf[4096]; - - static void fuzz_code(lzma_stream *stream, const uint8_t *inbuf, size_t inbuf_size) { + // Output buffer for decompressed data. This is write only; nothing + // cares about the actual data written here. + uint8_t outbuf[4096]; + // Give the whole input buffer at once to liblzma. // Output buffer isn't initialized as liblzma only writes to it. stream->next_in = inbuf; |