[devel] IA: destination buffer overflow - ppp

[Alexander Bokovoy]

Dmitriy Khanzhin пишет:
> Slava Semushin пишет:
>> 2007/11/27, Dmitry V. Levin <ldv на altlinux.org>:
>>> 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/