diff options
author | Corentin Noël <corentin.noel@collabora.com> | 2023-06-15 15:38:06 +0200 |
---|---|---|
committer | Corentin Noël <corentin.noel@collabora.com> | 2023-06-15 15:43:53 +0200 |
commit | 5c15c7f5a828aeeeaca8a5eeb4e428e32ebb2b0c (patch) | |
tree | d3f10e0e707c46e43596443fbaa9cc9b2c2da45d | |
parent | 1c206818afb6b8c31dc32a301a6b8bda26993ee9 (diff) |
fuzzer: Ensure strict prototypes of all functions
Allows to turn -Wstrict-prototypes by default in virglrenderer.
Part-of: <https://gitlab.freedesktop.org/virgl/virglrenderer/-/merge_requests/1159>
-rw-r--r-- | tests/fuzzer/virgl_fuzzer.c | 4 | ||||
-rw-r--r-- | tests/test_fuzzer_formats.c | 38 |
2 files changed, 21 insertions, 21 deletions
diff --git a/tests/fuzzer/virgl_fuzzer.c b/tests/fuzzer/virgl_fuzzer.c index af966bb..a381ea9 100644 --- a/tests/fuzzer/virgl_fuzzer.c +++ b/tests/fuzzer/virgl_fuzzer.c @@ -46,7 +46,7 @@ int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size); // with CLEANUP_EACH_INPUT), so suppress leak detection on everything // allocated by it. const char* __lsan_default_suppressions(void); -const char* __lsan_default_suppressions() { +const char* __lsan_default_suppressions(void) { return "leak:eglInitialize\n"; } @@ -108,7 +108,7 @@ static struct virgl_renderer_callbacks fuzzer_cbs = { static bool initialized = false; -static int initialize_environment() +static int initialize_environment(void) { if (!initialized) { EGLBoolean ok; diff --git a/tests/test_fuzzer_formats.c b/tests/test_fuzzer_formats.c index 727cd93..c977184 100644 --- a/tests/test_fuzzer_formats.c +++ b/tests/test_fuzzer_formats.c @@ -97,7 +97,7 @@ static int setenv(const char *name, const char *value, int overwrite) } #endif -static void initialize_environment() +static void initialize_environment(void) { setenv("LIBGL_ALWAYS_SOFTWARE", "true", 0); setenv("GALLIUM_DRIVER", "softpipe", 0); @@ -111,7 +111,7 @@ static void initialize_environment() virgl_renderer_context_create(ctx_id, (unsigned)strlen(name), name); } -static void test_format_wrong_size() +static void test_format_wrong_size(void) { struct virgl_renderer_resource_create_args args; args.handle = 10; @@ -158,7 +158,7 @@ static void test_format_wrong_size() virgl_renderer_submit_cmd((void *) cmd, ctx_id, VIRGL_CMD_BLIT_SIZE + 1); } -static void test_format_fail_and_double_free() +static void test_format_fail_and_double_free(void) { struct virgl_renderer_resource_create_args args; @@ -182,7 +182,7 @@ static void test_format_fail_and_double_free() /* Issue #141 */ -static void test_blit_info_format_check() +static void test_blit_info_format_check(void) { struct virgl_renderer_resource_create_args args; args.handle = 10; @@ -229,7 +229,7 @@ static void test_blit_info_format_check() virgl_renderer_submit_cmd((void *) cmd, ctx_id, VIRGL_CMD_BLIT_SIZE + 1); } -static void test_blit_info_format_check_null_format() +static void test_blit_info_format_check_null_format(void) { struct virgl_renderer_resource_create_args args; args.handle = 10; @@ -277,7 +277,7 @@ static void test_blit_info_format_check_null_format() } /* #142 */ -static void test_format_is_plain_nullptr_deref_trigger() +static void test_format_is_plain_nullptr_deref_trigger(void) { struct virgl_renderer_resource_create_args args; args.handle = 10; @@ -325,7 +325,7 @@ static void test_format_is_plain_nullptr_deref_trigger() } /* Issue #143 */ -static void test_format_util_format_is_rgb_nullptr_deref_trigger_illegal_resource() +static void test_format_util_format_is_rgb_nullptr_deref_trigger_illegal_resource(void) { struct virgl_renderer_resource_create_args args; args.handle = 8; @@ -357,7 +357,7 @@ static void test_format_util_format_is_rgb_nullptr_deref_trigger_illegal_resourc virgl_renderer_submit_cmd((void *) cmd, ctx_id, VIRGL_OBJ_SAMPLER_VIEW_SIZE + 1); } -static void test_format_util_format_is_rgb_nullptr_deref_trigger() +static void test_format_util_format_is_rgb_nullptr_deref_trigger(void) { struct virgl_renderer_resource_create_args args; args.handle = 8; @@ -390,7 +390,7 @@ static void test_format_util_format_is_rgb_nullptr_deref_trigger() } /* Test as reported in #139 */ -static void test_double_free_in_vrend_renderer_blit_int_trigger_invalid_formats() +static void test_double_free_in_vrend_renderer_blit_int_trigger_invalid_formats(void) { struct virgl_renderer_resource_create_args args; args.handle = 1; @@ -467,7 +467,7 @@ static void test_double_free_in_vrend_renderer_blit_int_trigger_invalid_formats( virgl_renderer_submit_cmd((void *) cmd, ctx_id, VIRGL_CMD_BLIT_SIZE + 1); } -static void test_double_free_in_vrend_renderer_blit_int_trigger() +static void test_double_free_in_vrend_renderer_blit_int_trigger(void) { struct virgl_renderer_resource_create_args args; args.handle = 1; @@ -545,7 +545,7 @@ static void test_double_free_in_vrend_renderer_blit_int_trigger() } -static void test_format_is_has_alpha_nullptr_deref_trigger_original() +static void test_format_is_has_alpha_nullptr_deref_trigger_original(void) { struct virgl_renderer_resource_create_args args; args.handle = 8; @@ -577,7 +577,7 @@ static void test_format_is_has_alpha_nullptr_deref_trigger_original() } -static void test_format_is_has_alpha_nullptr_deref_trigger_legal_resource() +static void test_format_is_has_alpha_nullptr_deref_trigger_legal_resource(void) { struct virgl_renderer_resource_create_args args; args.handle = 8; @@ -608,7 +608,7 @@ static void test_format_is_has_alpha_nullptr_deref_trigger_legal_resource() virgl_renderer_submit_cmd((void *) cmd, ctx_id, VIRGL_OBJ_SAMPLER_VIEW_SIZE + 1); } -static void test_heap_overflow_vrend_renderer_transfer_write_iov() +static void test_heap_overflow_vrend_renderer_transfer_write_iov(void) { struct virgl_renderer_resource_create_args args; args.handle = 4; @@ -648,7 +648,7 @@ static void test_heap_overflow_vrend_renderer_transfer_write_iov() virgl_renderer_submit_cmd((void *) cmd, ctx_id, 11 + 4 + 1); } -static void test_heap_overflow_vrend_renderer_transfer_write_iov_compressed_tex() +static void test_heap_overflow_vrend_renderer_transfer_write_iov_compressed_tex(void) { struct virgl_renderer_resource_create_args args; args.handle = 1; @@ -689,7 +689,7 @@ static void test_heap_overflow_vrend_renderer_transfer_write_iov_compressed_tex( } -static void test_cs_nullpointer_deference() +static void test_cs_nullpointer_deference(void) { struct virgl_renderer_resource_create_args args; @@ -723,7 +723,7 @@ static void test_cs_nullpointer_deference() virgl_renderer_submit_cmd((void *) cmd, ctx_id, 9); } -static void test_vrend_set_signle_abo_heap_overflow() { +static void test_vrend_set_signle_abo_heap_overflow(void) { struct virgl_renderer_resource_create_args args; args.handle = 0x4c474572; @@ -970,7 +970,7 @@ static void test_vrend_set_signle_abo_heap_overflow() { virgl_renderer_submit_cmd((void *) cmd, ctx_id, 0xde); } -static void test_vrend_set_shader_images_overflow() +static void test_vrend_set_shader_images_overflow(void) { uint32_t num_shaders = PIPE_MAX_SHADER_IMAGES + 1; uint32_t size = num_shaders * VIRGL_SET_SHADER_IMAGE_ELEMENT_SIZE + 3; @@ -986,7 +986,7 @@ static void test_vrend_set_shader_images_overflow() /* Test adapted from yaojun8558363@gmail.com: * https://gitlab.freedesktop.org/virgl/virglrenderer/-/issues/250 */ -static void test_vrend_3d_resource_overflow() { +static void test_vrend_3d_resource_overflow(void) { struct virgl_renderer_resource_create_args resource; resource.handle = 0x4c474572; @@ -1025,7 +1025,7 @@ static void test_vrend_3d_resource_overflow() { } -int main() +int main(void) { initialize_environment(); |