ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: "Sergey N.Yatskevich" <syatskevich@n21lab.gosniias.msk.ru>
To: devel@linux.iplabs.ru
Subject: Re: [devel] Re: Ошибка в shadow-utils (userdel)
Date: Sat, 13 Oct 2001 01:13:54 +0400
Message-ID: <20011013011354.28c992d0.syatskevich@n21lab.gosniias.msk.ru> (raw)
In-Reply-To: <20011013004429.B1989@localhost.localdomain>

On Sat, 13 Oct 2001 00:44:29 +0400
"Mikhail Zabaluev" <mhz@alt-linux.org> wrote:

> Позвольте мне немного программерского занудства.
:-))

> Здесь три раза вычисляется strlen(s1). Нехорошо.
> s1len = strlen(s1);
> return (strncmp(s2, s1, s1len) == 0 &&
>         (s2[s1len] == '\0' ||
>          s2[s1len] == '/'));
Да, так лучше :-)). Так и надо сделать.

> P.S. А если у кого-то домкаталог записан через точечки? А если через
> символьные ссылки? Впрочем, кара за подобное непотребство должна быть
> суровой :)
Я собственно не пытался предусмотреть все возможные ситуации, просто
при заведении большого количества пользователей на нашем сервере обнаружилось,
что для двух пользователей (ivk и ivkuznetsov) программа userdel -r
перестала работать корректно. А при большом количестве пользователей
такая ситуация может оказаться не так уж редка.

-- 
Sergey N. Yatskevich  <syatskevich@n21lab.gosniias.msk.ru>
_______________________________________________
Devel mailing list
Devel@linux.iplabs.ru
http://www.logic.ru/mailman/listinfo/devel


  reply	other threads:[~2001-10-12 21:13 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-10-12 10:08 [devel] " Sergey N.Yatskevich
2001-10-12 11:14 ` Dmitry V. Levin
2001-10-12 20:44 ` [devel] " Mikhail Zabaluev
2001-10-12 21:13   ` Sergey N.Yatskevich [this message]
2001-10-15  7:21   ` Dmitry V. Levin
2001-10-15  9:56     ` Mikhail Zabaluev

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20011013011354.28c992d0.syatskevich@n21lab.gosniias.msk.ru \
    --to=syatskevich@n21lab.gosniias.msk.ru \
    --cc=devel@linux.iplabs.ru \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git