From: <sv...@op...> - 2024-06-02 13:04:30
|
Author: sagamusix Date: Sun Jun 2 15:04:17 2024 New Revision: 20903 URL: https://source.openmpt.org/browse/openmpt/?op=revision&rev=20903 Log: [Fix] Fuzzer: Don't return from the AFL loop if we weren't able to load a file. Modified: trunk/OpenMPT/contrib/fuzzing/fuzz.cpp Modified: trunk/OpenMPT/contrib/fuzzing/fuzz.cpp ============================================================================== --- trunk/OpenMPT/contrib/fuzzing/fuzz.cpp Sun Jun 2 01:12:52 2024 (r20902) +++ trunk/OpenMPT/contrib/fuzzing/fuzz.cpp Sun Jun 2 15:04:17 2024 (r20903) @@ -60,7 +60,7 @@ OpenMPT::mpt::reinit_global_random(); openmpt_module * mod = openmpt_module_create_from_memory2( fileBuffer, fileSize, nullptr, nullptr, ErrFunc, nullptr, nullptr, nullptr, nullptr); if ( mod == NULL ) - return 1; + break; // verify API contract: If the file can be loaded, header probing must be successful too. if ( openmpt_probe_file_header( OPENMPT_PROBE_FILE_HEADER_FLAGS_DEFAULT, fileBuffer, fileSize, fileSize, nullptr, nullptr, ErrFunc, nullptr, nullptr, nullptr ) == OPENMPT_PROBE_FILE_HEADER_RESULT_FAILURE ) |