From d8ae93f85d977b55da0d0aad25597e24e23365f6 Mon Sep 17 00:00:00 2001
From: tru3 <tru3@tru3.com>
Date: Thu, 23 Jul 2026 21:49:57 +0200
Subject: [PATCH] Separator: fixes memory leakes

- Add missing ffStrbufDestroy(&options->outputColor) in ffDestroySeparatorOptions
  to prevent memory leak when outputColor is set via JSON config
- Replace two exit(1) calls in ffPreparePublicIp with proper error status
  propagation, matching the pattern used in the weather module
---
 src/modules/separator/separator.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/modules/separator/separator.c b/src/modules/separator/separator.c
index 4c5b4bc92..40f9a3e48 100644
--- a/src/modules/separator/separator.c
+++ b/src/modules/separator/separator.c
@@ -117,6 +117,7 @@ void ffInitSeparatorOptions(FFSeparatorOptions* options) {
 
 void ffDestroySeparatorOptions(FFSeparatorOptions* options) {
     ffStrbufDestroy(&options->string);
+    ffStrbufDestroy(&options->outputColor);
 }
 
 FFModuleBaseInfo ffSeparatorModuleInfo = {