Skip to content

Commit 7afd5d8

Browse files
authored
Merge pull request #2218 from terrelln/assert-seq
Fix unused variable warnings in fuzzing build mode without asserts
2 parents 081691a + cce0edf commit 7afd5d8

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

lib/decompress/zstd_decompress_block.c

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -948,7 +948,7 @@ ZSTD_decodeSequence(seqState_t* seqState, const ZSTD_longOffset_e longOffsets, c
948948
}
949949

950950
#ifdef FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION
951-
static int ZSTD_dictionaryIsActive(ZSTD_DCtx const* dctx, BYTE const* prefixStart, BYTE const* oLitEnd)
951+
MEM_STATIC int ZSTD_dictionaryIsActive(ZSTD_DCtx const* dctx, BYTE const* prefixStart, BYTE const* oLitEnd)
952952
{
953953
size_t const windowSize = dctx->fParams.windowSize;
954954
/* No dictionary used. */
@@ -969,6 +969,7 @@ MEM_STATIC void ZSTD_assertValidSequence(
969969
seq_t const seq,
970970
BYTE const* prefixStart, BYTE const* virtualStart)
971971
{
972+
#if DEBUGLEVEL >= 1
972973
size_t const windowSize = dctx->fParams.windowSize;
973974
size_t const sequenceSize = seq.litLength + seq.matchLength;
974975
BYTE const* const oLitEnd = op + seq.litLength;
@@ -986,6 +987,9 @@ MEM_STATIC void ZSTD_assertValidSequence(
986987
/* Offset must be within our window. */
987988
assert(seq.offset <= windowSize);
988989
}
990+
#else
991+
(void)dctx, (void)op, (void)oend, (void)seq, (void)prefixStart, (void)virtualStart;
992+
#endif
989993
}
990994
#endif
991995

0 commit comments

Comments
 (0)