From 95f36925455c952ae7a2bbd3da43c2ad6f7e64a3 Mon Sep 17 00:00:00 2001 From: Wolfgang Bumiller Date: Tue, 4 May 2021 09:45:27 +0200 Subject: [PATCH] fix permissions set in create_run_dir This directory needs to be owned by the backup user instead of root. Signed-off-by: Wolfgang Bumiller --- src/tools.rs | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/tools.rs b/src/tools.rs index 25323881..c58569a6 100644 --- a/src/tools.rs +++ b/src/tools.rs @@ -16,6 +16,7 @@ use openssl::hash::{hash, DigestBytes, MessageDigest}; use percent_encoding::{utf8_percent_encode, AsciiSet}; pub use proxmox::tools::fd::Fd; +use proxmox::tools::fs::{create_path, CreateOptions}; pub mod acl; pub mod apt; @@ -569,7 +570,11 @@ pub fn compute_file_csum(file: &mut File) -> Result<([u8; 32], u64), Error> { /// This exists to fixate the permissions for the run *base* directory while allowing intermediate /// directories after it to have different permissions. pub fn create_run_dir() -> Result<(), Error> { - let _: bool = proxmox::tools::fs::create_path(PROXMOX_BACKUP_RUN_DIR_M!(), None, None)?; + let backup_user = crate::backup::backup_user()?; + let opts = CreateOptions::new() + .owner(backup_user.uid) + .group(backup_user.gid); + let _: bool = create_path(PROXMOX_BACKUP_RUN_DIR_M!(), None, Some(opts))?; Ok(()) }