sudoが新しいディレクトリが作成されるディレクトリを変更するのはなぜですか？

Question

를 사용하고 있기 때문입니다 $USERNAME. 실제로 작동하는 모습을 쉽게 볼 수 있습니다.

$ bash -c 'echo "USERNAME is $USERNAME"'
USERNAME is terdon
$ sudo bash -c 'echo "USERNAME is $USERNAME"'
[sudo] password for terdon: 
USERNAME is

위에서 볼 수 있듯이 $USERNAME변수는 사용될 때 정의되지 않습니다 sudo. 이는 다음 줄이 문제의 원인임을 의미합니다.

if grep -q "Arch" $linux_file
then
    # The host is an Arch based distribution
    media_dir=/run/media/$USERNAME/

elif grep -q "Ubuntu" $linux_file || grep -q "Pop" $linux_file
then
    # The host is an Ubuntu or Pop OS based distribution
    media_dir=/media/$USERNAME/

else
    # The host is not a compatible distribution
    echo "Linux distribution not supported, exiting"
    exit 1
fi

$USERNAME아직 설정되지 않았 으므로 /run/media/Arch와 /media/다른 두 개만 얻을 수 있습니다. 간단한 해결책은 다음을 사용하는 것입니다 $SUDO_USER.

$ bash -c 'echo "USERNAME is $SUDO_USER"'
USERNAME is 
$ sudo bash -c 'echo "USERNAME is $SUDO_USER"'
USERNAME is terdon

위에서 볼 수 있듯이 $SUDO_USER해당 명령을 호출한 사용자의 이름이 저장됩니다 sudo. 따라서 스크립트를 이 방식으로만 실행하려는 경우 이를 사용할 수 있습니다. 여전히 루트로 실행되지 않는 옵션을 허용하려면 테스트를 추가할 수 있습니다.

if [ -n "$SUDO_USER" ]; then
  username="$SUDO_USER"
else
  username="$USERNAME"
fi

그런 다음 스크립트의 나머지 부분에서 $username어디에서나 사용했다면 $USERNAME.

$USER또는 사용 시에도 설정되는 표준 변수를 사용할 수 있습니다 sudo.

$ bash -c 'echo "USERNAME is $USER"'
USERNAME is terdon
$ sudo bash -c 'echo "USERNAME is $USER"'
USERNAME is root

root그러나 이는 다음으로 실행될 때 반환되므로 어쨌든 sudo해당 방법을 사용하는 것이 더 좋습니다 SUDO_USER.

마지막으로 스크립트에 대한 몇 가지 참고 사항은 다음과 같습니다.

따옴표가 없는 변수가 많이 있는데 이는 일반적으로 좋지 않은 생각이지만 임의의 경로와 파일 이름을 다루는 경우에는 특히 좋지 않습니다. 바라보다공백이나 기타 특수 문자 때문에 쉘 스크립트가 멈추는 이유는 무엇입니까?그리고bash/POSIX 쉘에서 변수를 인용하는 것을 잊어버리는 보안 위험.
오류 검사를 수행하지 않고 모든 명령이 유효하다고 가정합니다. 이는 명령이 포함된 스크립트를 실행할 sudo rm때 매우 위험합니다 . 변수가 설정되지 않거나 cd실패하여(예: 설정되지 않거나 유사한 이유로 인해 ) 존재하지 않는 디렉터리 $USERNAME로 이어지는 경우 전체 시스템을 쉽게 중단시킬 수 있습니다.cd

실제로 이런 일이 일어날 수 있습니다. 몇 년 전의 유명한 사례는 다음과 같습니다.Linux용 끔찍한 Steam 버그로 인해 PC의 모든 개인 파일이 삭제됩니다.. 따라서 스크립트 시작 부분 || exit에도 다양한 중요한 명령을 추가해야 합니다 .set -x

여러 인스턴스가 구문 분석되고 있습니다 ls. 이것은역시 아주 나쁜 생각이야그리고 쉽게 부서집니다. 따라서 다음 대신:

potential_dirs=($(ls -l $media_dir | awk '/^d/ {print $9}'))

이것을 사용하십시오 :

potential_dirs=( "$media_dir"/*/ )

사용중인 다음 부분에는 다양한 문제가 있습니다 ls.

# Change to backup directory
cd $base_dir

# Delete the oldest directories if necessary
if [ $num_delete -gt 0 ] ; then
    dirs=`ls -d */ | cut -f1 -d'/' | head -n $num_delete`
    for variable in $dirs
    do
        echo "Removing $variable directory"
        sudo rm -r $variable
    done
fi

첫째, 가장 오래된 디렉터리를 삭제하는 대신 알파벳순으로 첫 번째 디렉터리를 삭제합니다. 다음으로, 디렉토리 이름에 개행 문자가 포함되어 있으면 ls및 둘 다 실패하며 , 따옴표가 없으므로 공백이 있어도 실패합니다. 게다가 실제로 오류 검사가 없기 때문에 이전 검사가 어떤 이유로든 실패하면 다른 곳에서 파일을 삭제하게 됩니다!rmrm"$variable"cd $base_dir

したがって、削除するには、上記の内容を次のように変更します。$num_delete 最も古い目次：


# Delete the oldest directories if necessary
if [ $num_delete -gt 0 ] ; then
    readarray -d '' dirs < <(stat --printf='%Y %n\0' "$base_dir"/*/ |
                              sort -znk1,1 | head -zn "$num_delete" | 
                              cut -z -d ' ' -f2-)
    for variable in "${dirs[@]}"
    do
        echo "Removing $variable directory"
        sudo rm -r -- "$variable"
    done
fi

これは必ずしも必要ではありませんcd。

名前に改行文字が含まれている場合、ディレクトリの計算方法は失敗します。
```
dir_num=`$nfind $base_dir -mindepth 1 -maxdepth 1 -type d | wc -l`
```
代わりにこれを使用すると動作しますどの名前:
```
dir_num=$($nfind $base_dir -mindepth 1 -maxdepth 1 -type d -printf '.\n' | wc -l)
```
逆引用符（`command`）は使用しないでください$(command)。廃止予定です。存在する任意の場所var=`command`に置き換えることができますvar=$(command)。
いいえcur_dir=`pwd` 。現在のディレクトリはすでに特殊$PWD変数に格納されているので、単に$PWD。
スクリプトの最後には次の内容があります。
```
cd `pwd`
```
「コマンドが返すディレクトリに移動する」と言うので、これはデフォルトでは機能しません。したがって、現在のディレクトリのみが印刷されるため、pwd実際にはどこにも移動できません。`pwd`本気だったようですが、cd "$cur_dir"それさえ不要です。上記の3番目の項目に示すように、cd最初はそうする必要はありません。

Answer 1