ngx_cpystrn при n < 0

Igor Sysoev is at rambler-co.ru
Sun Apr 26 15:29:06 MSD 2009


On Sun, Apr 26, 2009 at 03:07:45PM +0400, Peter A Leonov wrote:

> Спасибо, Игорь.
> 
> Этого я и боялся. И компилятор мне никаких варнингов не показывал,  
> когда передавал -1.

Это общая черта memcpy, strncmp, etc.
Моя позиция такая - нужно просто не допускать такой ошибочной ситуации,
а не купировать её в самой функции.

> С уважением,
> Петр Леонов.
> +7 (905) 758-12-73
> 
> On 26.04.2009, at 10:47, Igor Sysoev <is at rambler-co.ru> wrote:
> 
> >On Sun, Apr 26, 2009 at 04:08:41AM +0400, Peter Leonov wrote:
> >
> >>Если в ngx_cpystrn(dst, src, n) передать n меньшее нуля, то функция
> >>будет копировать данные в dst пока не встретит '\0' в src.
> >>
> >>Верно ли такое поведение?
> >>
> >>
> >>Если нет, то предлагаю поменять условие в начале этой функции:
> >>   if (n == 0) {
> >>       return dst;
> >>   }
> >>на
> >>   if (n <= 0) {
> >>       return dst;
> >>   }
> >
> >size_t не может быть отрицательным. Это будет просто большое число.
> >
> >
> >-- 
> >Igor Sysoev
> >http://sysoev.ru/en/
> >

-- 
Игорь Сысоев
http://sysoev.ru





More information about the nginx-ru mailing list