[pbs-devel] [PATCH proxmox-backup 3/5] fix #4382: api2: remove permissions and tokens of user on deletion

Hannes Laimer h.laimer at proxmox.com
Tue Dec 20 15:57:12 CET 2022


Signed-off-by: Hannes Laimer <h.laimer at proxmox.com>
---
 src/api2/access/user.rs | 24 +++++++++++++++++++++++-
 1 file changed, 23 insertions(+), 1 deletion(-)

diff --git a/src/api2/access/user.rs b/src/api2/access/user.rs
index ce676252..40177c8d 100644
--- a/src/api2/access/user.rs
+++ b/src/api2/access/user.rs
@@ -346,7 +346,7 @@ pub fn update_user(
 )]
 /// Remove a user from the configuration file.
 pub fn delete_user(userid: Userid, digest: Option<String>) -> Result<(), Error> {
-    let _lock = pbs_config::user::lock_config()?;
+    let _user_lock = pbs_config::user::lock_config()?;
     let _tfa_lock = crate::config::tfa::write_lock()?;
 
     let (mut config, expected_digest) = pbs_config::user::config()?;
@@ -390,6 +390,28 @@ pub fn delete_user(userid: Userid, digest: Option<String>) -> Result<(), Error>
         }
     }
 
+    // delete_token needs the user config lock, therefore we have to drop it here
+    drop(_user_lock);
+
+    let user_tokens: Vec<ApiToken> = config
+        .convert_to_typed_array::<ApiToken>("token")?
+        .into_iter()
+        .filter(|token| token.tokenid.user().eq(&userid))
+        .collect();
+    for token in user_tokens {
+        if let Some(name) = token.tokenid.tokenname() {
+            let user = token.tokenid.user();
+            delete_token(user.clone(), name.to_owned(), None)?
+        }
+    }
+
+    // delete_token needs the acl config lock, that's why it is not aquired at the beginning
+    let _acl_lock = pbs_config::acl::lock_config()?;
+
+    let (mut tree, _digest) = pbs_config::acl::config()?;
+    tree.delete_authid(&Authid::from(userid));
+    pbs_config::acl::save_config(&tree)?;
+
     Ok(())
 }
 
-- 
2.30.2






More information about the pbs-devel mailing list