Cleanup impl for remove_tmp_snapshot_archives() (solana-labs#32297)

wen-coding · Aug 15, 2023 · aa9ad25 · aa9ad25
1 parent eb4a21d
commit aa9ad25
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/runtime/src/snapshot_utils.rs b/runtime/src/snapshot_utils.rs
@@ -623,11 +623,12 @@ fn is_bank_snapshot_complete(bank_snapshot_dir: impl AsRef<Path>) -> bool {
 pub fn remove_tmp_snapshot_archives(snapshot_archives_dir: impl AsRef<Path>) {
     if let Ok(entries) = std::fs::read_dir(snapshot_archives_dir) {
         for entry in entries.flatten() {
-            let file_name = entry
+            if entry
                 .file_name()
-                .into_string()
-                .unwrap_or_else(|_| String::new());
-            if file_name.starts_with(TMP_SNAPSHOT_ARCHIVE_PREFIX) {
+                .to_str()
+                .map(|file_name| file_name.starts_with(TMP_SNAPSHOT_ARCHIVE_PREFIX))
+                .unwrap_or(false)
+            {
                 let path = entry.path();
                 let result = if path.is_dir() {
                     fs_err::remove_dir_all(path)