[devel] IA: destination buffer overflow - ppp

[Alexander Bokovoy]

Led пишет:
> В сообщении от Wednesday 28 November 2007 11:35:52 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/