[PATCH v2 2/2] fff-web: Fix setting hostname in settings.html

[Adrian Schmutzler]

Update:

Hab mir das gerade noch mal genauer angesehen:

Bei FW 20170110 machen beide Varianten keinen Unterschied, aber mit meiner
recht neuen LEDE 17.01.2-Firmware funktioniert nur die Variante mit echo
(auch in /bin/sh). Bei der Variante ohne echo kommt "root@?" raus. Hab jetzt
keine ältere alpha zum Testen da, wie es sich dort verhält.

Die Ausgabe wenn man nur "uci -q get ..." ohne Umleitung macht erfolgt bei
beiden. Ist also evtl. ein Bug, der aber mit dem Patch ohne Nachteile
behoben würde.

-----Original Message-----
From: franken-dev [mailto:franken-dev-bounces at freifunk.net] On Behalf Of
Dominik Heidler
Sent: Dienstag, 25. Juli 2017 00:12
To: Adrian Schmutzler <freifunk at adrianschmutzler.de>;
franken-dev at freifunk.net
Subject: Re: [PATCH v2 2/2] fff-web: Fix setting hostname in settings.html

What does this change?

Am 20.07.2017 um 15:29 schrieb Adrian Schmutzler:
> Signed-off-by: Adrian Schmutzler <freifunk at adrianschmutzler.de>
> ---
>  src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git
a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html
b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html
> index bdbd69b..7488af6 100755
> --- a/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html
> +++ b/src/packages/fff/fff-web/files/www/ssl/cgi-bin/settings.html
> @@ -14,7 +14,7 @@ if [ "$REQUEST_METHOD" == "POST" ] ; then
>  	else
>  		# Allgemeine Einstellungen
>  		uci -q set "system. at system[0].hostname=${POST_hostname}"
> -		uci -q get 'system. at system[0].hostname' >
/proc/sys/kernel/hostname
> +		echo $(uci -q get "system. at system[0].hostname") >
/proc/sys/kernel/hostname
>  		uci -q set
"system. at system[0].description=${POST_description}"
>  		uci -q set "system. at system[0].latitude=${POST_latitude}"
>  		uci -q set "system. at system[0].longitude=${POST_longitude}"
> 

-- 
franken-dev mailing list
franken-dev at freifunk.net
http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net