facebook/zstd
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix unused variable warnings in fuzzing build mode without asserts

Browse Source 
Fix unused vairable warnings when `FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION` is defined but asserts are disabled.

Fixes #2210.
This commit is contained in:
Nick Terrell 2020-06-22 12:56:57 -07:00
parent f03192c122
commit cce0edfdbe
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
lib/decompress/zstd_decompress_block.c
View File

@ -948,7 +948,7 @@ ZSTD_decodeSequence(seqState_t* seqState, const ZSTD_longOffset_e longOffsets, c
} }
#ifdef FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION #ifdef FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION
static int ZSTD_dictionaryIsActive(ZSTD_DCtx const* dctx, BYTE const* prefixStart, BYTE const* oLitEnd) MEM_STATIC int ZSTD_dictionaryIsActive(ZSTD_DCtx const* dctx, BYTE const* prefixStart, BYTE const* oLitEnd)
{ {
size_t const windowSize = dctx->fParams.windowSize; size_t const windowSize = dctx->fParams.windowSize;
/* No dictionary used. */ /* No dictionary used. */
@ -969,6 +969,7 @@ MEM_STATIC void ZSTD_assertValidSequence(
seq_t const seq, seq_t const seq,
BYTE const* prefixStart, BYTE const* virtualStart) BYTE const* prefixStart, BYTE const* virtualStart)
{ {
#if DEBUGLEVEL >= 1
size_t const windowSize = dctx->fParams.windowSize; size_t const windowSize = dctx->fParams.windowSize;
size_t const sequenceSize = seq.litLength + seq.matchLength; size_t const sequenceSize = seq.litLength + seq.matchLength;
BYTE const* const oLitEnd = op + seq.litLength; BYTE const* const oLitEnd = op + seq.litLength;
@ -986,6 +987,9 @@ MEM_STATIC void ZSTD_assertValidSequence(
/* Offset must be within our window. */ /* Offset must be within our window. */
assert(seq.offset <= windowSize); assert(seq.offset <= windowSize);
} }
#else
(void)dctx, (void)op, (void)oend, (void)seq, (void)prefixStart, (void)virtualStart;
#endif
} }
#endif #endif