Re: падение мастер процесса

[Polonevich Ivan <loverjoni@xxxxxxxxx>]

С коллегами сделали такие выводы:

в функции ngx_shmtx_force_unlock выполняется операция atomic_cmp_set 
без предварительной проверки инициализирован ли mtx->lock или нет. Таким
образом если модуль выделил шаред память классическим способом но не
использовал mutex локи для этого блока то они остаются
неинициализрованными и при операции unlock_mutexes вызываемой по команде
reload мы получаем segmetation_fault. добавил проверку на то
инициализирован ли вообще mtx-lock, это снимает проблему. Насколько это
корректно или надо править модуль. Ниже диф:
[root@build-server nginx-1.3.0]# [root@build-server d]# diff ngx_shmtx.c
ngx_shmtx_patched.c
154c154
<     if (ngx_atomic_cmp_set(mtx->lock, pid, 0)) {
---
>     if (mtx->lock ? ngx_atomic_cmp_set(mtx->lock, pid, 0): 0) {

On 5/11/12 1:14 PM, Ivan Palanevich wrote:
> Вероятно вы правы.
>
> http://code.google.com/p/ustats/issues/detail?id=5
>
> On May 11, 2012, at 10:22 AM, Maxim Dounin wrote:
>
>> Hello!
>>
>> On Thu, May 10, 2012 at 08:36:06PM +0300, Ivan Palanevich wrote:
>>
>>> Заметил, что падает мастер процесс при отправке -HUP,
>>>
>>> cat /etc/redhat-release 
>>> CentOS release 5.8 (Final)
>>>
>>> Linux server 2.6.18-308.4.1.el5 #1 SMP Tue Apr 17 17:08:00 EDT 2012 x86_64 
>>> x86_64 x86_64 GNU/Linux
>>>
>>> nginx -V
>>> nginx version: nginx/1.2.0
>>> built by gcc 4.1.2 20080704 (Red Hat 4.1.2-52)
>>> TLS SNI support enabled
>>> configure arguments: --user=nginx --group=nginx --prefix=/usr/share/nginx 
>>> --sbin-path=/usr/sbin/nginx --conf-path=/etc/nginx/nginx.conf 
>>> --error-log-path=/var/log/nginx/error.log 
>>> --http-log-path=/var/log/nginx/access.log 
>>> --http-client-body-temp-path=/var/lib/nginx/tmp/client_body 
>>> --http-proxy-temp-path=/var/lib/nginx/tmp/proxy 
>>> --http-fastcgi-temp-path=/var/lib/nginx/tmp/fastcgi 
>>> --pid-path=/var/run/nginx.pid --lock-path=/var/lock/subsys/nginx 
>>> --with-openssl=/home/mockbuild/rpmbuild/BUILD/nginx-1.2.0/openssl-0.9.8t/ 
>>> --with-openssl-opt=enable-tlsext --with-http_secure_link_module 
>>> --with-http_random_index_module --with-http_ssl_module 
>>> --with-http_realip_module --with-http_addition_module 
>>> --with-http_sub_module --with-http_dav_module 
>>> --with-http_gzip_static_module --with-http_stub_status_module 
>>> --with-http_perl_module --with-http_geoip_module --with-debug --with-ipv6 
>>> --with-file-aio --with-cc-opt='-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 
>>> -fexceptions -fstack-protector --param=ssp-buffer-size=4 -m64 
>>> -mtune=generic' 
>>> --add-module=/home/mockbuild/rpmbuild/BUILD/nginx-1.2.0/nginx-upstream-fair 
>>> --add-module=/home/mockbuild/rpmbuild/BUILD/nginx-1.2.0/mod_ustats
>>>
>>> Из последних добавленных модификаций 
>>> --with-openssl=/home/mockbuild/rpmbuild/BUILD/nginx-1.2.0/openssl-0.9.8t/ 
>>>
>>> Началось до 1.2.0 , но с какой точно версии не помню.
>> [...]
>>
>>> Подскажите куда покапать.
>> Начать с простого: собраться без сторонних модулей и патчей, и 
>> попробовать воспроизвести проблему без них.  Я подозреваю, что 
>> проблема где-то в районе ustats.
>>
>> Maxim Dounin
>>
>> _______________________________________________
>> nginx-ru mailing list
>> nginx-ru@xxxxxxxxx
>> http://mailman.nginx.org/mailman/listinfo/nginx-ru


-- 
Signed, Polonevich Ivan
System  Administrator 
CООО Гейм Стрим / Wargaming.net 
http://www.wargaming.net/
http://www.game-stream.org/
icq 8158401
Skype jonilover

_______________________________________________
nginx-ru mailing list
nginx-ru@xxxxxxxxx
http://mailman.nginx.org/mailman/listinfo/nginx-ru