From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on sa.int.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-1.3 required=5.0 tests=AWL,BAYES_00, DNS_FROM_OPENWHOIS autolearn=no version=3.2.5 Date: Mon, 12 Jul 2010 13:30:14 +0300 From: Michael Shigorin To: ALT Linux Team development discussions Message-ID: <20100712103014.GH8084@osdn.org.ua> Mail-Followup-To: ALT Linux Team development discussions References: <20100710184538.GN24468@osdn.org.ua> <20100711200345.GA28209@altlinux.org> <20100711222623.GB13854@wo.int.altlinux.org> <4C3A59EC.7060102@altlinux.ru> Mime-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <4C3A59EC.7060102@altlinux.ru> User-Agent: Mutt/1.4.2.1i Subject: Re: [devel] =?koi8-r?b?W2Nyb3NzXSBJOiDOwdPeo9QgIvDz8O81IiDJINPU0sHO?= =?koi8-r?b?ztnIINfP0NLP08/XINcg1sHCxdLF?= X-BeenThere: devel@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: ALT Linux Team development discussions List-Id: ALT Linux Team development discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 12 Jul 2010 10:30:30 -0000 Archived-At: List-Archive: List-Post: On Mon, Jul 12, 2010 at 03:55:24AM +0400, Ilya Mashkin wrote: > >Загляни в этот репозиторий: > >http://git.altlinux.org/people/dkoryavov/packages/?p=consolehelper.git;a=log > >Когда видишь подобное, очень сложно подавить в себе потребность > >высказать личное мнение в технической рассылке. Дим, всё-таки лучше высказывать его построчно. Может, ещё кому пригодится, раз уж время на анализ затрачено. > Объясните мне, неграмотному, что не так в этом патче? > По-моему 3 попытки ввода пароля это очень хорошо, ибо я вот, > например ввожу свои пароли иногда с 10-й попытки. Функциональность действительно полезная, но при доработке программ, имеющих отношение к безопасности, стоит быть более строгим к себе и не переоценивать свою квалификацию. Давно не писал на C, но сходу не понравилось: --- consolehelper/src/client/gui.c int rc, attempt; --- consolehelper/src/server/pam.c int attempt = 0; Здесь видно, что человек вроде как в курсе про явную инициализацию переменных, но пишет неряшливо и не перечитывает свои git diff (или устал до состояния пропускания такого). Не нравится кусок кода около strdup, но вчитываться с утра не берусь. "const char *msg = ..." непосредственно в теле consolehelper/src/server/pam.c::do_auth() тоже аккуратной реализацией назвать не могу. Мелочи вроде неаккуратного оформления коммитов (например, спек плюс реверс-дифф) также являются плохим признаком для ответственного пакета. -- ---- WBR, Michael Shigorin ------ Linux.Kiev http://www.linux.kiev.ua/