blockworks-foundation
/
solana
mirror of https://github.com/blockworks-foundation/solana.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Removes erroneous Copy on AsRef<Path> params in snapshot_utils (#31382)

This commit is contained in:
Brooks 2023-04-27 22:45:59 -04:00 committed by GitHub
parent 18baafbb48
commit 6393c7d98d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
runtime/src/snapshot_utils.rs
View File

@ -427,8 +427,8 @@ pub enum VerifySlotDeltasError {
/// This is useful if the process does not have permission
/// to delete the top level directory it might be able to
/// delete the contents of that directory.
fn delete_contents_of_path(path: impl AsRef<Path> + Copy) {
if let Ok(dir_entries) = std::fs::read_dir(path) {
fn delete_contents_of_path(path: impl AsRef<Path>) {
if let Ok(dir_entries) = std::fs::read_dir(&path) {
for entry in dir_entries.flatten() {
let sub_path = entry.path();
let metadata = match entry.metadata() {
@ -474,9 +474,9 @@ fn delete_contents_of_path(path: impl AsRef<Path> + Copy) {
/// If the process is killed and the deleting process is not done,
/// the leftover path will be deleted in the next process life, so
/// there is no file space leaking.
pub fn move_and_async_delete_path(path: impl AsRef<Path> + Copy) {
pub fn move_and_async_delete_path(path: impl AsRef<Path>) {
let mut path_delete = PathBuf::new();
path_delete.push(path);
path_delete.push(&path);
path_delete.set_file_name(format!(
"{}{}",
path_delete.file_name().unwrap().to_str().unwrap(),
@ -491,7 +491,7 @@ pub fn move_and_async_delete_path(path: impl AsRef<Path> + Copy) {
return;
}
if let Err(err) = std::fs::rename(path, &path_delete) {
if let Err(err) = std::fs::rename(&path, &path_delete) {
warn!(
"Path renaming failed: {}. Falling back to rm_dir in sync mode",
err.to_string()