tyler
/
proxmox-backup
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

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 <w.bumiller@proxmox.com>
This commit is contained in:
Wolfgang Bumiller 2021-05-04 09:45:27 +02:00
parent 686173dc2a
commit 95f3692545
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/tools.rs
View File

@ -16,6 +16,7 @@ use openssl::hash::{hash, DigestBytes, MessageDigest};
use percent_encoding::{utf8_percent_encode, AsciiSet}; use percent_encoding::{utf8_percent_encode, AsciiSet};
pub use proxmox::tools::fd::Fd; pub use proxmox::tools::fd::Fd;
use proxmox::tools::fs::{create_path, CreateOptions};
pub mod acl; pub mod acl;
pub mod apt; 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 /// This exists to fixate the permissions for the run *base* directory while allowing intermediate
/// directories after it to have different permissions. /// directories after it to have different permissions.
pub fn create_run_dir() -> Result<(), Error> { 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(()) Ok(())
} }