[pve-devel] [PATCH cluster 3/5] pmxcfs: status: call g_strdup instead of g_strdup_printf when no formatting is needed
Lukas Wagner
l.wagner at proxmox.com
Fri Aug 1 11:54:29 CEST 2025
This should be a tiny bit more efficient.
Signed-off-by: Lukas Wagner <l.wagner at proxmox.com>
---
src/pmxcfs/status.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/pmxcfs/status.c b/src/pmxcfs/status.c
index e438d9f..c8e072f 100644
--- a/src/pmxcfs/status.c
+++ b/src/pmxcfs/status.c
@@ -1371,7 +1371,7 @@ static void update_rrd_data(const char *key, gconstpointer data, size_t len) {
use_pve2_file = 1;
g_free(filename);
- filename = g_strdup_printf("%s", filename_pve2);
+ filename = g_strdup(filename_pve2);
} else {
// neither file exists, check for directories to decide and create file
@@ -1383,7 +1383,7 @@ static void update_rrd_data(const char *key, gconstpointer data, size_t len) {
use_pve2_file = 1;
g_free(filename);
- filename = g_strdup_printf("%s", filename_pve2);
+ filename = g_strdup(filename_pve2);
char *dir = g_path_get_dirname(filename);
checked_mkdir(dir, 0755);
@@ -1443,7 +1443,7 @@ static void update_rrd_data(const char *key, gconstpointer data, size_t len) {
// old file exists, use it
use_pve2_file = 1;
g_free(filename);
- filename = g_strdup_printf("%s", filename_pve2);
+ filename = g_strdup(filename_pve2);
} else {
// neither file exists, check for directories to decide and create file
@@ -1455,7 +1455,7 @@ static void update_rrd_data(const char *key, gconstpointer data, size_t len) {
use_pve2_file = 1;
g_free(filename);
- filename = g_strdup_printf("%s", filename_pve2);
+ filename = g_strdup(filename_pve2);
int argcount = sizeof(rrd_def_vm) / sizeof(void *) - 1;
create_rrd_file(filename, argcount, rrd_def_vm);
@@ -1509,7 +1509,7 @@ static void update_rrd_data(const char *key, gconstpointer data, size_t len) {
} else if (g_file_test(filename_pve2, G_FILE_TEST_EXISTS)) {
// old file exists, use it
g_free(filename);
- filename = g_strdup_printf("%s", filename_pve2);
+ filename = g_strdup(filename_pve2);
} else {
// neither file exists, check for directories to decide and create file
@@ -1522,7 +1522,7 @@ static void update_rrd_data(const char *key, gconstpointer data, size_t len) {
create_rrd_file(filename, argcount, rrd_def_storage_pve9_0);
} else if (g_file_test(RRDDIR "/pve2-storage", G_FILE_TEST_IS_DIR)) {
g_free(filename);
- filename = g_strdup_printf("%s", filename_pve2);
+ filename = g_strdup(filename_pve2);
char *dir = g_path_get_dirname(filename);
checked_mkdir(dir, 0755);
--
2.47.2
More information about the pve-devel
mailing list