From 9d610374d7016f21e1271b8d78b2a89f8746b243 Mon Sep 17 00:00:00 2001
From: Guillaume Dallenne <guillaume.dallenne@stormshield.eu>
Date: Thu, 10 Nov 2022 10:57:07 +0100
Subject: [PATCH] Rename run to __libfuzzer_sys_run

This reduces chances of function collision. When using the `fuzz_target` macro, calling `run` inside the block will call the `run` function declared inside the macro definition instead of calling the `run` function defined in the module using `fuzz_target`. Using the `run` function of the macro is probably not the intended goal because it leads to a recursive call. Renaming makes it less likely to call `__libfuzzer_sys_run` by accident.
---
 src/lib.rs | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/lib.rs b/src/lib.rs
index ce1eb98..376feb9 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -221,7 +221,7 @@ macro_rules! fuzz_target {
                     return 0;
                 }
 
-                run(bytes);
+                __libfuzzer_sys_run(bytes);
                 0
             }
 
@@ -234,11 +234,11 @@ macro_rules! fuzz_target {
             // panics in separate fuzzers can accidentally appear the same
             // because each fuzzer will have a function called
             // `rust_fuzzer_test_input`. By using a normal Rust function here
-            // it's named something like `the_fuzzer_name::_::run` which should
+            // it's named something like `the_fuzzer_name::_::__libfuzzer_sys_run` which should
             // ideally help prevent oss-fuzz from deduplicate fuzz bugs across
             // distinct targets accidentally.
             #[inline(never)]
-            fn run($bytes: &[u8]) {
+            fn __libfuzzer_sys_run($bytes: &[u8]) {
                 $body
             }
         };
@@ -294,13 +294,13 @@ macro_rules! fuzz_target {
                     Err(_) => return -1,
                 };
 
-                let result = ::libfuzzer_sys::Corpus::from(run(data));
+                let result = ::libfuzzer_sys::Corpus::from(__libfuzzer_sys_run(data));
                 result.to_libfuzzer_code()
             }
 
             // See above for why this is split to a separate function.
             #[inline(never)]
-            fn run($data: $dty) -> $rty {
+            fn __libfuzzer_sys_run($data: $dty) -> $rty {
                 $body
             }
         };