On Fri, May 11, 2012 at 10:31:28PM +0400, Dmitry V. Levin wrote:
> > +	/* wheather the end of realName ends with .service */
> > +	if (strncmp (realName+strlen(realName)-sizeof(".service"),
> > +		     ".service", sizeof(".service")) != 0) {
> 
> Если вдруг strlen(realName) окажется меньше sizeof(".service"), то
> может получиться не очень хорошо.

Спасибо, поправил, patch3 в аттачменте.

> Патч, в принципе, логичный, и (с точностью до кода проверки суффикса)
> применимый.  Только не логичнее ли применить это изменение прямо в
> systemctl?  Есть ли какие-нибудь причины патчить именно chkconfig,
> а не systemctl?

В systemctl этот патч может вызвать нежелательные эффекты, например,
сломать user overrides в /etc/systemd. Поэтому врядли этот патч
попадет в апстрим. См. аналогичной функциональности патч от Andrey Borzenkov'а
[ http://lists.freedesktop.org/archives/systemd-devel/2011-March/001725.html ]
и ответ на него Lennart'а Poettering'га
[ http://lists.freedesktop.org/archives/systemd-devel/2011-March/001790.html ]

А поскольку код systemd бурно развивается, без поддержки апстрима
поддерживать в нем патчи достаточно накладно.

В то же время chkconfig достаточно стабилен, и логика иметь у нас
переносимый между init-ами скрипт туда вписывается.

-- 

Dr. Igor Vlasenko
--------------------
Topology Department
Institute of Math
Kiev, Ukraine


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.