From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <474D536D.5050208@altlinux.org> Date: Wed, 28 Nov 2007 14:39:25 +0300 From: Alexander Bokovoy User-Agent: Thunderbird 2.0.0.6 (X11/20071022) MIME-Version: 1.0 To: ALT Linux Team development discussions References: <20071125120814.GA22522@basalt.office.altlinux.org> <474D2CED.30401@sandy.ru> <474D3678.508@altlinux.org> <200711281218.41747.ledest@gmail.com> In-Reply-To: <200711281218.41747.ledest@gmail.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - glaucus.site5.com X-AntiAbuse: Original Domain - lists.altlinux.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - altlinux.org X-Source: X-Source-Args: X-Source-Dir: Subject: Re: [devel] IA: destination buffer overflow - ppp X-BeenThere: devel@lists.altlinux.org X-Mailman-Version: 2.1.9 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: Wed, 28 Nov 2007 11:39:29 -0000 Archived-At: List-Archive: List-Post: Led пишет: > В сообщении от Wednesday 28 November 2007 11:35:52 Alexander Bokovoy > написал(а): >> Dmitriy Khanzhin пишет: >>> Slava Semushin пишет: >>>> 2007/11/27, Dmitry V. Levin : >>>>> On Tue, Nov 27, 2007 at 06:32:40PM +0300, Dmitriy Khanzhin wrote: >>>>>> - strncat(ttyname, name, sizeof(ttyname)); >>>> + strncat(ttyname, name, sizeof(ttyname)-1); >>>> >>>>> Автор этого кода не справился с функцией strncat. Исправление >>>>> тривиально. >>>> Фикс должен быть таким (см. выше)? Или нет? >>> Вот так собралось без warning'а (аттач). >> Еще бы оно работало. Вы же просто повторили ерунду за авторами этого >> плагина. sizeof(ttyname), как и sizeof(name), равносильны sizeof(char *) >> и равны размеру указателя, но никак не размеру строки. Там нужно >> использовать strlen(ttyname). > > Угу, этим вы замените бред на тавтологию:) :-) Смысл был скорее в указании на функцию. Впрочем, код, показанный Турбиным, страшен не по реализации, а по сути своей. -- / Alexander Bokovoy Samba Team http://www.samba.org/ ALT Linux Team http://www.altlinux.org/ Midgard Project Ry http://www.midgard-project.org/