Re: upload module 2.0.7

["Valery Kholodkov" <valery+nginxru@xxxxxxxxxxx>]

Да, вот патч.

> cc1: warnings being treated as errors
> /root/install/nginx_upload_module-2.0.7/ngx_http_upload_module.c: In
> function 'ngx_http_upload_flush_output_buffer':
> /root/install/nginx_upload_module-2.0.7/ngx_http_upload_module.c:983:
> warning: comparison between signed and unsigned
> make[1]: ***
> [objs/addon/nginx_upload_module-2.0.7/ngx_http_upload_module.o]
> Error 1
> make[1]: Leaving directory `/root/install/nginx-0.7.19'
> make: *** [build] Error 2
>
> CentOS 5.2 x86_64
>
> Пришлось в objs/Makefile убирать -Werror.
>
> 2008/10/20 Rauan Maemirov <rauan@xxxxxxxxxxxx>:
>> Ага, понял.
>>
>> Спасибо, теперь mime-типы определяются
>> правильно.
>>
>> 2008/10/20 Valery Kholodkov <valery+nginxru@xxxxxxxxxxx>:
>>>
>>> В <nginx installation dir>/conf/mime.types
>>>
>>>> А где именно этот блок types?
>>>>
>>>> 2008/10/19 Valery Kholodkov <valery+nginxru@xxxxxxxxxxx>:
>>>>>
>>>>> Забыл упомянуть, что в блок types {} нужно
>>>>> будет добавить расширение mkv с
>>>>> соответствующим mime-типом.
>>>>>
>>>>>> Я не хочу ломать ветку 2.0, поэтому
>>>>>> решу
>>>>>> эту проблему отдельным патчем. См.
>>>>>> аттач.
>>>>>>
>>>>>>> Не исправили. :(
>>>>>>> Она все еще воспроизводится.
>>>>>>>
>>>>>>> 2008/10/18 Rauan Maemirov <rauan@xxxxxxxxxxxx>:
>>>>>>>> Здравствуйте, Валерий.
>>>>>>>>
>>>>>>>> Была ли решена проблема с
>>>>>>>> application/octet-stream?
>>>>>>>>
>>>>>>>> Т.е. модуль принимал mime-type'ы от
>>>>>>>> браузеров "как есть". Была
>>>>>>>> проблема с Оперой, когда она вместо
>>>>>>>> .mkv-шного типа (matroska)
>>>>>>>> отправляла application/octet-stream.
>>>>>>>>
>>>>>>>> 2008/10/18 Valery Kholodkov <valery+nginxru@xxxxxxxxxxx>:
>>>>>>>>>
>>>>>>>>> Здравствуйте!
>>>>>>>>>
>>>>>>>>> После переезда я рад вернуться к
>>>>>>>>> OSS-проектам. На этот раз секьюрити
>>>>>>>>> релиз:
>>>>>>>>>
>>>>>>>>> Версия 2.0.7 (от 18 октября 2008):
>>>>>>>>>  * Изменение: ограничение размеров
>>>>>>>>> файлов и тела результативного
>>>>>>>>> запроса
>>>>>>>>> с
>>>>>>>>> помощью директив upload_max_file_size и
>>>>>>>>> upload_max_output_body_len
>>>>>>>>>  * Добавление: директива upload_pass_args
>>>>>>>>> позволяет передавать аргументы
>>>>>>>>> исходного запроса бакэнду. Спасибо
>>>>>>>>> Todd
>>>>>>>>> Fisher.
>>>>>>>>>
>>>>>>>>> Максимальный размер тела
>>>>>>>>> результативного запроса
>>>>>>>>> по-умолчанию
>>>>>>>>> установлен в 100 килобайт. Этого
>>>>>>>>> достаточно, чтобы содержать
>>>>>>>>> большинство
>>>>>>>>> форм, и в то же время предотвращает
>>>>>>>>> заполнение памяти большими
>>>>>>>>> нефайловыми
>>>>>>>>> полями.
>>>>>>>>>
>>>>>>>>> Подробности на странице модуля:
>>>>>>>>> http://www.grid.net.ru/nginx/upload.ru.html
>>>>>>
>>>>>> --
>>>>>> Best regards,
>>>>>> Valery Kholodkov
>>>>>
>>>>>
>>>>> --
>>>>> Best regards,
>>>>> Valery Kholodkov
>>>>>
>>>>>
>>>>
>>>
>>>
>>> --
>>> Best regards,
>>> Valery Kholodkov
>>>
>>>
>>
>


-- 
Best regards,
Valery Kholodkov

--- nginx_upload_module-2.0.7/ngx_http_upload_module.c  2008-10-18 
11:18:18.000000000 +0200
+++ nginx_upload_module-2.0.7-mine/ngx_http_upload_module.c     2008-10-21 
10:24:53.000000000 +0200
@@ -978,7 +978,7 @@
             ngx_crc32_update(&u->crc32, buf, len);
 
         if(ulcf->max_file_size != 0) {
-            if(u->output_file.offset + len > ulcf->max_file_size)
+            if(u->output_file.offset + (off_t)len > ulcf->max_file_size)
                 return NGX_UPLOAD_TOOLARGE;
         }