Skip to content

Add initialization of clevel to static cdict (#3525)#3527

Merged
yoniko merged 1 commit intofacebook:devfrom
yoniko:fix-3525-static-cdict-clevel-init
Mar 7, 2023
Merged

Add initialization of clevel to static cdict (#3525)#3527
yoniko merged 1 commit intofacebook:devfrom
yoniko:fix-3525-static-cdict-clevel-init

Conversation

@yoniko
Copy link
Contributor

@yoniko yoniko commented Mar 7, 2023

  • Initializes clevel in ZSTD_CCtxParams_init
  • Adds CI workflow for msan fuzzers runs without optimization (-O0)
  • Fixes Makefile to correctly pass on user defined MOREFLAGS and FUZZER_FLAGS in cases they have been overwritten

UPDATE: CI failed without the fix, fix is now enabled

@yoniko
Copy link
Contributor Author

yoniko commented Mar 7, 2023

clang-msan-fuzz-unoptimized has failed as expected, verifying that the new CI flow does catch the issue. Going to amend the fix to make sure all CI tests pass now.

- Initializes clevel in `ZSTD_CCtxParams_init`
- Adds CI workflow for msan fuzzers runs without optimization (`-O0`)
- Fixes Makefile to correctly pass on user defined `MOREFLAGS` and `FUZZER_FLAGS` in cases they have been overwritten
@yoniko yoniko force-pushed the fix-3525-static-cdict-clevel-init branch from 2473df2 to 6ef471f Compare March 7, 2023 01:13
@yoniko yoniko merged commit 988ce61 into facebook:dev Mar 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants