Delete target directory before running odrefresh
This allows it to re-create the directory and files when it runs.
Bug: 210460516
Test: composd_cmd async-odrefresh
Change-Id: I1e5a100f0d68d0266c9af9d681e243dd5bdb0556
diff --git a/compos/src/compilation.rs b/compos/src/compilation.rs
index cf6f30a..7eaae5d 100644
--- a/compos/src/compilation.rs
+++ b/compos/src/compilation.rs
@@ -20,7 +20,7 @@
use std::env;
use std::fs::{read_dir, File};
use std::os::unix::io::{AsRawFd, RawFd};
-use std::path::{Path, PathBuf};
+use std::path::{self, Path, PathBuf};
use crate::artifact_signer::ArtifactSigner;
use crate::compos_key_service::Signer;
@@ -83,6 +83,11 @@
if zygote_arch != "zygote64" && zygote_arch != "zygote64_32" {
bail!("Invalid zygote arch");
}
+ // Disallow any sort of path traversal
+ if target_dir_name.contains(path::MAIN_SEPARATOR) {
+ bail!("Invalid target directory {}", target_dir_name);
+ }
+
Ok(Self { system_dir_fd, output_dir_fd, staging_dir_fd, target_dir_name, zygote_arch })
}
}