From e62361f530d5fc2bbec75193a59cb709501fb3a1 Mon Sep 17 00:00:00 2001
From: Henning Glawe <glaweh@debian.org>
Date: Sat, 15 Mar 2025 14:09:44 +0100
Subject: [PATCH] use explicit list instead of implicit split
---
src/mpsd_hpc_tools/quota.py | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mpsd_hpc_tools/quota.py b/src/mpsd_hpc_tools/quota.py
index cbdeb91..c561b9b 100644
--- a/src/mpsd_hpc_tools/quota.py
+++ b/src/mpsd_hpc_tools/quota.py
@@ -40,8 +40,8 @@ def get_ceph_attribute(path: str, attribute: str) -> int:
value returned by getfattrs command (only integers support)
"""
# getfattr command should return a single integer number
- cmd = f"getfattr -n {attribute} --only-values --absolute-names {path}"
- output = subprocess.check_output(cmd.split()).decode()
+ cmd = ["getfattr", "-n", attribute, "--only-values", "--absolute-names", path]
+ output = subprocess.check_output(cmd).decode()
return int(output)
--
GitLab