From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <bga@altlinux.org>
Date: Wed, 28 Nov 2007 16:59:25 +0300
From: "Grigory Batalov" <bga@altlinux.org>
To: devel@lists.altlinux.org
Message-Id: <20071128165925.91b576b9.bga@altlinux.org>
In-Reply-To: <20071128101523.GA13629@localhost.localdomain>
References: <20071125120814.GA22522@basalt.office.altlinux.org>
	<20071125195520.GS28492@osdn.org.ua>
	<20071125211632.GC30421@basalt.office.altlinux.org>
	<20071126085538.GB13915@mw.local.seiros.ru>
	<474C3898.6090500@sandy.ru>
	<20071127154518.GM11953@basalt.office.altlinux.org>
	<75e139a00711272256i770bd25ci550debea6f99b693@mail.gmail.com>
	<474D2CED.30401@sandy.ru> <474D3678.508@altlinux.org>
	<20071128101523.GA13629@localhost.localdomain>
Organization: ALT Linux Team
X-Mailer: Sylpheed version 2.2.9 (GTK+ 2.10.6; x86_64-alt-linux-gnu)
X-Face: ")Qu!$R3ym>`zI`H#Scijb6gliZ{v(u-Y6muu{*\y^\%Chb}{eEo4=L^jB]/>jhI]yW._DV')>_Yj#L`{-(%kaIt76Smg&jk>R>@!QO(p{:dr},@;
Mime-Version: 1.0
Content-Type: multipart/signed; protocol="application/pgp-signature";
	micalg="PGP-SHA1";
	boundary="Signature=_Wed__28_Nov_2007_16_59_25_+0300_+gY4jCx7Vr+P3AnU"
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 <devel@lists.altlinux.org>
List-Id: ALT Linux Team development discussions <devel.lists.altlinux.org>
List-Unsubscribe: <https://lists.altlinux.org/mailman/listinfo/devel>,
	<mailto:devel-request@lists.altlinux.org?subject=unsubscribe>
List-Archive: <http://lists.altlinux.org/pipermail/devel>
List-Post: <mailto:devel@lists.altlinux.org>
List-Help: <mailto:devel-request@lists.altlinux.org?subject=help>
List-Subscribe: <https://lists.altlinux.org/mailman/listinfo/devel>,
	<mailto:devel-request@lists.altlinux.org?subject=subscribe>
X-List-Received-Date: Wed, 28 Nov 2007 13:59:31 -0000
Archived-At: <http://lore.altlinux.org/devel/20071128165925.91b576b9.bga@altlinux.org/>
List-Archive: <http://lore.altlinux.org/devel/>
List-Post: <mailto:devel@altlinux.org>

--Signature=_Wed__28_Nov_2007_16_59_25_+0300_+gY4jCx7Vr+P3AnU
Content-Type: text/plain; charset=KOI8-R
Content-Disposition: inline
Content-Transfer-Encoding: 8bit

On Wed, 28 Nov 2007 12:15:23 +0200
Kirill A. Shutemov wrote:

> > >>>> -     strncat(ttyname, name, sizeof(ttyname));
> > >> + strncat(ttyname, name, sizeof(ttyname)-1);
> > >>
> > >>> Автор этого кода не справился с функцией strncat. Исправление
> > >>> тривиально.
> > >> Фикс должен быть таким (см. выше)? Или нет?
> > >>
> > > Вот так собралось без warning'а (аттач).
> > Еще бы оно работало. Вы же просто повторили ерунду за авторами этого
> > плагина. sizeof(ttyname), как и sizeof(name), равносильны sizeof(char *)
> > и равны размеру указателя, но никак не размеру строки. Там нужно
> > использовать strlen(ttyname).
>
> Если ttyname объявлен как
> char ttyname[PATH_MAX];
> то sizeof(ttyname) будет равен PATH_MAX;

  Запишите и меня в эту партию =)

--Signature=_Wed__28_Nov_2007_16_59_25_+0300_+gY4jCx7Vr+P3AnU
Content-Type: application/pgp-signature

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.1 (GNU/Linux)

iD8DBQFHTXRCTSOu/blIq24RAuZSAJ90FceR5uBagUAnm8M3Xu9ebcopOQCfVCK1
0QNEJmXQQM1o7iuGgDgVmEE=
=h3kb
-----END PGP SIGNATURE-----

--Signature=_Wed__28_Nov_2007_16_59_25_+0300_+gY4jCx7Vr+P3AnU--