From 2cfd9db918fc40ebf4cbb5fde47c67024c9bfb4d Mon Sep 17 00:00:00 2001 From: Charles Giessen Date: Mon, 23 Dec 2024 17:33:31 -0600 Subject: [PATCH] vulkan-loader: Use correct variable in destroy function Commit e17999e "fixed" the settings_fuzzer.c test by cleaning up memory, but did not pass in the correct variable. It would reinterpret a pointer to memory as a `loader_layer_list` struct, which is all sorts of wrong. It doesn't help that I forcibly cast the memory pointer to the loader_layer_list struct type, causing any warnings to be surpressed. --- projects/vulkan-loader/fuzzers/settings_fuzzer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/projects/vulkan-loader/fuzzers/settings_fuzzer.c b/projects/vulkan-loader/fuzzers/settings_fuzzer.c index 9d2430f81f79..63b1a2115171 100644 --- a/projects/vulkan-loader/fuzzers/settings_fuzzer.c +++ b/projects/vulkan-loader/fuzzers/settings_fuzzer.c @@ -72,7 +72,7 @@ int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size) { bool should_search_for_other_layers = true; get_settings_layers(NULL, &settings_layers, &should_search_for_other_layers); // Free allocated memory - loader_delete_layer_list_and_properties(NULL, (struct loader_layer_list *)settings_layers.list); + loader_delete_layer_list_and_properties(NULL, &settings_layers); should_skip_logging_global_messages(0); update_global_loader_settings(); teardown_global_loader_settings();