rename

remove useless collect
aptos-labs · Jan 16, 2025 · 4f6bcc2 · 4f6bcc2
1 parent e251e49
commit 4f6bcc2
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/testsuite/fuzzer/fuzz.sh b/testsuite/fuzzer/fuzz.sh
@@ -284,7 +284,7 @@ function run() {
         fi
     fi
     info "Running $fuzz_target"
-    cargo_fuzz run --sanitizer none -O $fuzz_target $testcase -- -fork=15 #-ignore_crashes=1
+    cargo_fuzz run --sanitizer address -O $fuzz_target $testcase -- -fork=15 #-ignore_crashes=1
 }
 
 function test() {

diff --git a/third_party/move/move-binary-format/src/errors.rs b/third_party/move/move-binary-format/src/errors.rs
@@ -20,12 +20,11 @@ pub type PartialVMResult<T> = ::std::result::Result<T, PartialVMError>;
 /// e.g. DEBUG_VM_STATUS=ABORTED,UNKNOWN_INVARIANT_VIOLATION_ERROR ./fuzz.sh run move_aptosvm_publish_and_run <testcase>
 /// third_party/move/move-core/types/src/vm_status.rs:506 for the list of status codes.
 #[cfg(feature = "fuzzing")]
-macro_rules! maybe_debug_panic {
+macro_rules! fuzzing_maybe_panic {
     ($major_status:expr, $message:expr) => {{
         if let Ok(debug_statuses) = std::env::var("DEBUG_VM_STATUS") {
-            let status_strings: Vec<&str> = debug_statuses.split(',').collect();
-            if status_strings
-                .iter()
+            if debug_statuses
+                .split(',')
                 .any(|s| s.trim() == format!("{:?}", $major_status))
             {
                 panic!("PartialVMError: {:?} {:?}", $major_status, $message);
@@ -458,7 +457,7 @@ impl PartialVMError {
         };
 
         #[cfg(feature = "fuzzing")]
-        maybe_debug_panic!(major_status, message);
+        fuzzing_maybe_panic!(major_status, message);
 
         Self(Box::new(PartialVMError_ {
             major_status,