PeterPawn
commented
4 years ago Machbar ist das sicherlich ... nur ist es auch sinnvoll?
Kennst Du das Kommando hash?
pi@pi4:/tmp $ hash --help
hash: hash [-lr] [-p pathname] [-dt] [name ...]
Remember or display program locations.
Determine and remember the full pathname of each command NAME. If
no arguments are given, information about remembered commands is displayed.
Options:
-d forget the remembered location of each NAME
-l display in a format that may be reused as input
-p pathname use PATHNAME as the full pathname of NAME
-r forget all remembered locations
-t print the remembered location of each NAME, preceding
each location with the corresponding NAME if multiple
NAMEs are given
Arguments:
NAME Each NAME is searched for in $PATH and added to the list
of remembered commands.
Exit Status:
Returns success unless NAME is not found or an invalid option is given.
pi@pi4:/tmp $Wendet man das richtig an, kannst Du Dir alle weiteren Änderungen dieser Art einfach sparen - und das sollte dann ja vermutlich noch viele andere Stellen betreffen (ich verfolge die Commits in Freetz-NG ja nicht).
pi@pi4:~ $ cd /tmp
pi@pi4:/tmp $ command -v sed
/bin/sed
pi@pi4:/tmp $ mkdir tools
pi@pi4:/tmp $ printf '#!/bin/sh\nprintf "%%s\\n" "$*"\n/bin/sed $*\n' >tools/own_sed
pi@pi4:/tmp $ cat tools/own_sed
#!/bin/sh
printf "%s\n" "$*"
/bin/sed $*
pi@pi4:/tmp $ chmod u+x tools/own_sed
pi@pi4:/tmp $ hash -p /tmp/tools/own_sed sed
pi@pi4:/tmp $ sed -n -e p tools/own_sed
-n -e p tools/own_sed
#!/bin/sh
printf "%s\n" "$*"
/bin/sed $*
pi@pi4:/tmp $ command -v sed
/tmp/tools/own_sed
pi@pi4:/tmp $Hier wurde das normale sed-Kommando, das ansonsten über PATH gefunden würde, durch ein Shell-Skript ersetzt, das erst noch die Parameter des Aufrufs auf STDOUT ausgibt, bevor es das ursprüngliche /bin/sed mit ebendiesen Parametern aufruft (der Aufruf zeigt das gesamte Skript einfach noch einmal an).
Das /tmp/tools/own_sed könnte hier natürlich auch Dein Ersatz für sed sein.
Wenn sich Dein gesamtes Problem auch auf diesem Weg lösen ließe, braucht es den Patch hier auch nicht. Nur mal so als Tipp ... das kann sehr viel Arbeit sparen.
PS: ... und es ist auch ein Grund dafür, warum man bei sicherheitskritischen Skripten (bzw. bei einer Aufrufumgebung, die man nicht vollkommen unter Kontrolle hat und wo man mit Angriffen rechnen muß) am besten mit absoluten Pfaden beim Aufruf von Kommandos arbeiten sollte, wenn man ganz sicher sein will, daß man auch das richtige Kommando aufruft.
EDIT: Ich habe das Beispiel mal durch die Verwendung eines anderen Namens für das eigene Skript noch etwas anschaulicher bzw. übersichtlicher gemacht.
fda77
refs https://github.com/PeterPawn/YourFritz/issues/30