diff options
author | Yi Kong <yikong@google.com> | 2021-06-11 03:28:09 +0000 |
---|---|---|
committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2021-06-11 03:28:09 +0000 |
commit | 3387c68feb8567aaeb1a33ebec5c9dfbafea5737 (patch) | |
tree | bb2d12fcb00260a05a06fd9dde7c12b7e21c53b9 | |
parent | c52e999c37df4f54d1b0f6a16d2f8cc406c239f3 (diff) | |
parent | 2a8f28f46094672eec33ae234ed97a8b63c483c4 (diff) | |
download | extras-3387c68feb8567aaeb1a33ebec5c9dfbafea5737.tar.gz |
profcollectd: fix setting ACL bits am: 2a8f28f460
Original change: https://googleplex-android-review.googlesource.com/c/platform/system/extras/+/14934458
Change-Id: Icac7747ad07b2f54902b3881d4bcfba593496746
-rw-r--r-- | profcollectd/libprofcollectd/report.rs | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/profcollectd/libprofcollectd/report.rs b/profcollectd/libprofcollectd/report.rs index f3b8fe12..c37993b7 100644 --- a/profcollectd/libprofcollectd/report.rs +++ b/profcollectd/libprofcollectd/report.rs @@ -19,9 +19,9 @@ use anyhow::{anyhow, Result}; use lazy_static::lazy_static; use macaddr::MacAddr6; -use std::fs::{self, File}; +use std::fs::{self, File, Permissions}; use std::io::{Read, Write}; -use std::os::unix::fs::OpenOptionsExt; +use std::os::unix::fs::PermissionsExt; use std::path::{Path, PathBuf}; use std::time::SystemTime; use uuid::v1::{Context, Timestamp}; @@ -45,11 +45,14 @@ pub fn pack_report(profile: &Path, report: &Path, config: &Config) -> Result<Str // Remove the current report file if exists. fs::remove_file(&report).ok(); + let report_file = fs::OpenOptions::new().create_new(true).write(true).open(&report)?; + // Set report file ACL bits to 644, so that this can be shared to uploaders. // Who has permission to actually read the file is protected by SELinux policy. - let report = fs::OpenOptions::new().create_new(true).write(true).mode(0o644).open(&report)?; + fs::set_permissions(&report, Permissions::from_mode(0o644))?; + let options = FileOptions::default().compression_method(Deflated); - let mut zip = ZipWriter::new(report); + let mut zip = ZipWriter::new(report_file); fs::read_dir(profile)? .filter_map(|e| e.ok()) |