Skip to content
This repository was archived by the owner on Mar 16, 2026. It is now read-only.

fuzzer: refactor logging to dump state of loaded config before validation#38

Merged
Wenzel merged 4 commits intoIntelLabs:masterfrom
Wenzel:add_debug_config_load
Nov 30, 2022
Merged

fuzzer: refactor logging to dump state of loaded config before validation#38
Wenzel merged 4 commits intoIntelLabs:masterfrom
Wenzel:add_debug_config_load

Conversation

@Wenzel
Copy link
Copy Markdown
Contributor

@Wenzel Wenzel commented Nov 30, 2022

I refactored the logger.py to split the main setup_logging() function into 2 functions

  • load_logging_config(): simply loads the logging.yaml config file, and store it in a global variable
  • setup_basic_logging(): setup console, useful to log loaded configuration before validation happens
  • add_file_logging(): add $WORKDIR/kafl_fuzzer.log if --log has been specified

Finally I dump all possible info I could find about the loaded kAFL configuration, in order to simplify detecting config issues.
First: Loaded configuration files
image

Second: Command line configuration
image

Third Loaded configuration values (Dynaconf loaded values, either from env vars (matching KAFL_) or config files)
image

And finally the final config after validation:
image

@Wenzel Wenzel force-pushed the add_debug_config_load branch from ae84f75 to d708414 Compare November 30, 2022 16:53
@il-steffen il-steffen self-requested a review November 30, 2022 17:14
@Wenzel Wenzel merged commit 9a90e85 into IntelLabs:master Nov 30, 2022
@Wenzel Wenzel deleted the add_debug_config_load branch November 30, 2022 17:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants