1

Написал небольшой скрипт(еще не дописан) на shell, но он некорректно работает. Вот код:

#!/bin/sh
#перейти на root
echo "Enter password for user root"
su root
if [ "$(id -u)" == "0" ]; then
    echo 'Вы вошли под пользователем root'
    #создание каталога /s и переход в него
    if [ -d /s ]; then
            echo 'Директория /s существует'
    else
            echo 'Отсутствие директории /s'
            mkdir s
            echo 'Create directory s'
    fi
    #создание каталога /proj
    echo 'Create directiry /proj'
    mkdir s/proj
    #передача прав на папку /s/proj пользователю
    user=""
    echo 'Enter your login: '
    read user
    echo "Передача папки пользователю"
    chown -R $user s/proj
    echo 'Выход из под пользователя root '
    exit
else
    echo 'Неверный пароль'
    exit 0
fi

Он не создает папку /proj и при входе под пользователем root не выводит 'Вы вошли под пользователем root'. Что не так с этим скриптом?

5
  • должно быть "$(id -u)" -eq 0 30 июл 2018 в 11:02
  • @SeniorPomidor, ничего не изменилось, все так же при входе не выводит сообщение и не создается папка
    – Zhuk
    30 июл 2018 в 11:05
  • опечатки: 1. третья и четвёртая строка — лишние. 2. вместо /proj дважды встречается s/proj 30 июл 2018 в 14:57
  • выполнив su root вы вышли из оболочки откуда был запущен скрипт и он прервался 30 июл 2018 в 19:30
  • первый s/proj для создания директории proj, а с командой chown мы передаем права доступа пользователю
    – Zhuk
    31 июл 2018 в 9:14

1 ответ 1

1

Самое главное:
Выполнив su root вы вышли из оболочки откуда был запущен скрипт и скрипт прервался

Ну и потом:

root_access_check ()
{
    if [[ "$(id -u)" -ne 0 ]] && grep -qv 'sudo' < <(id $UID); then
        echo 'For execute this script requires a root access.'
        eixt 1
    fi
}

1) [[ "$(id -u)" -ne 0 ]] Проверили не является пользователь root
2) grep -qv 'sudo' < <(id $UID) Проверили не состоит пользователь, от имени которого запущен скрипт, в sudo

Если и то и другое нет, вышли из скрипта
Если все ок, то дальше все команды, кроме read и echo через sudo

if [ -d /s ]; then
            echo 'Директория /s существует'
    else
            echo 'Отсутствие директории /s'
            mkdir /s
            echo 'Create directory s'
fi  

Меняем на sudo mkdir -p /s не так пафосно, зато практично ))

echo 'Enter your login: '
    read user  

Смело меняем на read -r -p 'Enter your login: ' user

else
    echo 'Неверный пароль'
    exit 0
fi

exit 0 Означает что скрипт завешен без ошибок

if [ "$(id -u)" == "0" ]; then
ИМХО корректней так:
if [ "$(id -u)" -eq 0 ]; then
Но работать будет и так и так

UPDATE
Запускаем это su -c './<script_name> "<путь_до_папки_где_хотите_создать_папку_юзеру>"'

#!/bin/bash -e
declare user_name folder_name  parrent_missing parrent_folder_path user_group

parrent_folder_path="$1"
[[ "$parrent_folder_path" ]] || parrent_folder_path='.'

[[ -d "$parrent_folder_path" ]] || parrent_missing=1

while :
do
    clear
    read -r -p 'Enter the username for which the folder is created: ' user_name
    if grep -wq "$user_name" '/etc/passwd'; then
        user_group="$(id -gn "$user_name")"
        break
    else
        echo -e "\e[31mUser $user_name is not registered.\\nPlease check your entry   and try again.\e[m"
        sleep 5
    fi
done

read -r -p 'Enter the desired folder name: ' folder_name

mkdir -p "${parrent_folder_path}/${folder_name}" && echo -e "\e[32mFolder '${parrent_folder_path}/${folder_name}' was successfully created\e[m"

[[ "${parrent_missing}" ]] && echo -e "\e[33mFolder '$parrent_folder_path' was missing, but created\e[m"

chown "${user_name}:${user_group}" "${parrent_folder_path}/${folder_name}"
exit 0

Если не указать "<путь_до_папки_где_хотите_создать_папку_юзеру>" то создаст в текущей папке

15
  • Проверяем /s, а создаём s. Разница есть? Зачем ключ -p для mkdir? Не вижу тут создания дерева подкаталогов. Shell — трудный язык программирования. Я вообще не понимаю, зачем народ в него постоянно пытается?
    – 0andriy
    30 июл 2018 в 22:48
  • man mkdir читаем до конца -p, --parents Создает недостающие родительские каталоги для каждого указанного каталога. Права доступа к родительским каталогам устанавливаются в значение umask, логически сложенное с 'u+wx'. Аргументы, соответствующие существующим каталогам, игнорируются. (Таким образом, если каталог /a существует, то 'mkdir /a' выдаст ошибку, а 'mkdir -p /a' -- нет.) 30 июл 2018 в 23:04
  • Читайте мой комментарий до конца. Хотя бы четвёртое предложение от начала. И да, я в курсе, что такое -p у mkdir.
    – 0andriy
    30 июл 2018 в 23:07
  • прочитал Не вижу тут создания дерева подкаталогов Не понял к чему. mkdir -p s выполнится без ошибок вне зависимости есть каталог s или нет. Если нет создаст если нет пройдет мимо. Что и требовалоась 30 июл 2018 в 23:08
  • Да всё к тому же, к ненужности ключа -p в данном конкретном случае.
    – 0andriy
    30 июл 2018 в 23:10

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.