From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Schantl <stefan.schantl@ipfire.org>
To: development@lists.ipfire.org
Subject: [PATCH] qos.cgi: Fix truncated status output
Date: Fri, 30 Jul 2021 19:57:01 +0200
Message-ID: <20210730175701.2930-1-stefan.schantl@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============4512717957292798956=="
List-Id: <development.lists.ipfire.org>

--===============4512717957292798956==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

In the past only the fist line of the status output has been passed
to the cleanhtml() function and displayed. Now the whole output will be
converted to a string, cleaned and displyed on the WUI again.

Fixes #12666.

Signed-off-by: Stefan Schantl <stefan.schantl(a)ipfire.org>
---
 html/cgi-bin/qos.cgi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/html/cgi-bin/qos.cgi b/html/cgi-bin/qos.cgi
index fa566b523..d768f95f5 100644
--- a/html/cgi-bin/qos.cgi
+++ b/html/cgi-bin/qos.cgi
@@ -543,7 +543,7 @@ elsif ($qossettings{'ACTION'} eq $Lang::tr{'status'} )
 	if ($qossettings{'ENABLED'} eq 'on'){
 		my $output = "";
 		my @output = &General::system_output("/usr/local/bin/qosctrl", "status");
-		$output = &Header::cleanhtml(@output[0],"y");
+		$output = &Header::cleanhtml(join("", @output), "y");
 		print "<pre>$output</pre>\n";
 	} else { print "$Lang::tr{'QoS not enabled'}"; }
 	&Header::closebox();
-- 
2.30.2


--===============4512717957292798956==--