From: Michael Tremer <michael.tremer@ipfire.org>
Date: Wed, 8 Oct 2025 20:33:15 +0000 (+0000)
Subject: file: Try to make the static analyzer happy
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1edca80eaf8ef4b25b836637ca91cbc80efc41dd;p=telemetry.git

file: Try to make the static analyzer happy

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
---

diff --git a/src/daemon/file.c b/src/daemon/file.c
index d18b80f..b48ddd6 100644
--- a/src/daemon/file.c
+++ b/src/daemon/file.c
@@ -64,56 +64,50 @@ static int collecty_file_create(collecty_file** file, collecty_ctx* ctx) {
 }
 
 int collecty_file_open_path(collecty_file** file, collecty_ctx* ctx, const char* path) {
-	collecty_file* self = NULL;
 	int r;
 
 	// Create a new object
-	r = collecty_file_create(&self, ctx);
+	r = collecty_file_create(file, ctx);
 	if (r < 0)
 		goto ERROR;
 
 	// Open the file
-	self->f = fopen(path, "r");
-	if (!self->f) {
+	(*file)->f = fopen(path, "r");
+	if (!(*file)->f) {
 		ERROR(ctx, "Failed to open %s: %m\n", path);
 		r = -errno;
 		goto ERROR;
 	}
 
-	// Return the pointer
-	*file = self;
 	return 0;
 
 ERROR:
-	if (self)
-		collecty_file_unref(self);
+	if (*file)
+		collecty_file_unref(*file);
 
 	return r;
 }
 
 int collecty_file_open_buffer(collecty_file** file, collecty_ctx* ctx, collecty_buffer* buffer) {
-	collecty_file* self = NULL;
 	int r;
 
 	// Create a new object
-	r = collecty_file_create(&self, ctx);
+	r = collecty_file_create(file, ctx);
 	if (r < 0)
 		goto ERROR;
 
 	// Open the buffer as a file
-	self->f = collecty_buffer_fopen(buffer, "r");
-	if (!self->f) {
+	(*file)->f = collecty_buffer_fopen(buffer, "r");
+	if (!(*file)->f) {
 		r = -errno;
 		goto ERROR;
 	}
 
-	// Return the pointer
-	*file = self;
 	return 0;
 
 ERROR:
-	if (self)
-		collecty_file_unref(self);
+	if (*file)
+		collecty_file_unref(*file);
 
 	return r;
 }