bsp: Allow for arguments in passwd function #119

fbl · 2021-02-13T18:51:56+01:00

fbl commented

2021-02-13 18:51:56 +01:00

Previously, we have added a passwd function to our shell which executes
passwd and restarts uhttpd afterwards, so the WebUI password is updated.

This adds the ability to still pass command-line arguments to passwd.

Fixes: #117
Signed-off-by: Fabian Bläse fabian@blaese.de

Previously, we have added a passwd function to our shell which executes passwd and restarts uhttpd afterwards, so the WebUI password is updated. This adds the ability to still pass command-line arguments to passwd. Fixes: #117 Signed-off-by: Fabian Bläse <fabian@blaese.de>

fbl added this to the 20210218 milestone 2021-02-13 18:51:56 +01:00

fbl added the

bug

label 2021-02-13 18:51:56 +01:00

fbl self-assigned this 2021-02-13 18:51:56 +01:00

fbl added 1 commit 2021-02-13 18:51:56 +01:00

4dfc57bc3f bsp: Allow for arguments in passwd function

Previously, we have added a passwd function to our shell which executes
passwd and restarts uhttpd afterwards, so the WebUI password is updated.

This adds the ability to still pass command-line arguments to passwd.

Fixes: #117
Signed-off-by: Fabian Bläse <fabian@blaese.de>

rohammer approved these changes 2021-02-13 19:34:50 +01:00

rohammer left a comment

Das ist wichtig.

Reviewed-by: Robert Langhammer <rlanghammer@web.de>

Das ist wichtig. ``` Reviewed-by: Robert Langhammer <rlanghammer@web.de> ```

adschm commented

2021-02-13 20:39:19 +01:00

Auch noch ins stable release?

adschm reviewed 2021-02-13 20:39:58 +01:00

bsp/default/root_file_system/etc/profile

						
				@ -15,3 +15,3 @@

				# update uhttpd passwd on passwd-change

				[ -e /etc/rc.d/S50uhttpd ] && passwd() { /bin/passwd && /etc/init.d/uhttpd restart; }

				[ -e /etc/rc.d/S50uhttpd ] && passwd() { /bin/passwd "$@" && /etc/init.d/uhttpd restart; }

adschm commented

2021-02-13 20:39:58 +01:00

Machen die Anführungszeichen nicht aus mehreren Argumenten eines?

fbl commented

2021-02-13 21:57:19 +01:00

Erstaunlicherweise: nein.

Würde sich das $@ "normal" verhalten, wärs sowohl mit als auch ohne falsch:

Mit würde alles ein Argument werden
Ohne würden Argumente mit Leerzeichen zu mehreren werden

$@ ist aber ein "Special Parameter", für den besondere Regeln gelten:

When the expansion occurs within double quotes, each parameter expands to a separate word. That is, "$@" is equivalent to "$1" "$2" ...

(Aus der bash man-page)

Erstaunlicherweise: nein. Würde sich das $@ "normal" verhalten, wärs sowohl mit als auch ohne falsch: * Mit würde alles ein Argument werden * Ohne würden Argumente mit Leerzeichen zu mehreren werden $@ ist aber ein "Special Parameter", für den besondere Regeln gelten: > When the expansion occurs within double quotes, each parameter expands to a separate word. That is, "$@" is equivalent to "$1" "$2" ... (Aus der bash man-page)