From d335cc9592afa74f219bbf564b01c612ea336e77 Mon Sep 17 00:00:00 2001
From: Robin Roevens <robin.roevens@disroot.org>
Date: Tue, 10 Sep 2024 23:32:32 +0200
Subject: [PATCH] header.pl: only get memory consumption when service is
 running

It probably doesn't matter much as the get_memory_consumption function just returns 0 when no pids are found. But it shouldn't even try as the mem var is never used when the service is not running.

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
---
 config/cfgroot/header.pl | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/config/cfgroot/header.pl b/config/cfgroot/header.pl
index a90a6f481..3778c4e59 100644
--- a/config/cfgroot/header.pl
+++ b/config/cfgroot/header.pl
@@ -950,9 +950,6 @@ EOF
 			@pids = &General::find_pids("${process}");
 		}
 
-		# Get memory consumption
-		my $mem = &General::get_memory_consumption(@pids);
-
 		print <<EOF;
 				<tr>
 					<th scope="row">
@@ -962,6 +959,9 @@ EOF
 
 		# Running?
 		if (scalar @pids) {
+			# Get memory consumption
+			my $mem = &General::get_memory_consumption(@pids);
+
 			# Format memory
 			$mem = &General::formatBytes($mem);
 
-- 
2.39.5