[devel] usermod -m -d ...

[devel] usermod -m -d ...

[Timur Batyrshin]

[-- Attachment #1: Type: text/plain, Size: 1217 bytes --]

Случайно заглянул в исходники usermod и возник такой вопрос:


static void move_home (void)
{
. . .
      if (errno == EXDEV) {
        if (mkdir
              (user_newhome, sb.st_mode & 0777)) {
                 fprintf (stderr,
                          _
                         ("%s: can't create %s\n"),
                         Prog, user_newhome);
        }

        if (chown (user_newhome,
                   sb.st_uid, sb.st_gid)) {
          fprintf (stderr,
                   _("%s: can't chown %s\n"),
                   Prog, user_newhome);
          rmdir (user_newhome);
          fail_exit (E_HOMEDIR);
        }
. . .
}

Если не удается создать каталог user_newhome выводится ошибка
об этом, но аварийный выход не осуществляется. Это так и задумано?

Ведь если указанный каталог не существует (проверка на это проходит
чуть раньше) и его не удается создать, то смысла в chown и последующих
операциях нет.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 197 bytes --]

Re: [devel] usermod -m -d ...

[Slava Semushin]

16 сентября 2008 г. 14:14 пользователь Timur Batyrshin
<batyrshin%ieml.ru> написал:
> Случайно заглянул в исходники usermod и возник такой вопрос:

Ну вы даёте :)

[...]
> Если не удается создать каталог user_newhome выводится ошибка
> об этом, но аварийный выход не осуществляется. Это так и задумано?
>
> Ведь если указанный каталог не существует (проверка на это проходит
> чуть раньше) и его не удается создать, то смысла в chown и последующих
> операциях нет.

Так если каталог не существует, то ведь и chown, вероятно, не
отработает? Вернёт ошибку и уже тогда произойдёт аварийный выход.

Хотя более очевидно было бы сделать этот аварийный выход явным IMHO.


-- 
+ Slava Semushin | slava.semushin @ gmail.com
+ ALT Linux Team | php-coder @ altlinux.ru