Re: nginx-0.7.34

[Igor Sysoev <is@xxxxxxxxxxxxx>]

On Wed, Feb 11, 2009 at 01:38:59AM +0300, Mike Baikov wrote:

> Igor Sysoev пишет:
> >On Tue, Feb 10, 2009 at 08:15:06PM +0300, Mike Baikov wrote:
> >
> >  
> >>Igor Sysoev пишет:
> >>    
> >>>Изменения в nginx 0.7.34                                          
> >>>10.02.2009
> >>>
> >>>   *) Исправление: в директиве rewrite, возвращающей редирект, старые 
> >>>      аргументы присоединялись к новым через символ "?" вместо "&";
> >>>      ошибка появилась в 0.1.18.
> >>>      Спасибо Максиму Дунину.
> >>>
> >>> 
> >>>      
> >>Игорь,
> >>
> >>Это будет закомитчено в 0.6 ветку?
> >>    
> >
> >Да, могу дать патч.
> >
> >
> >  
> Да, было бы замечательно!

Патч.


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

Index: src/http/ngx_http_script.c
===================================================================
--- src/http/ngx_http_script.c  (revision 1817)
+++ src/http/ngx_http_script.c  (revision 1818)
@@ -244,10 +244,21 @@
 
         name.data = &sc->source->data[i];
 
-        while (i < sc->source->len
-               && sc->source->data[i] != '$'
-               && !(sc->source->data[i] == '?' && sc->compile_args))
-        {
+        while (i < sc->source->len) {
+
+            if (sc->source->data[i] == '$') {
+                break;
+            }
+
+            if (sc->source->data[i] == '?') {
+
+                sc->args = 1;
+
+                if (sc->compile_args) {
+                    break;
+                }
+            }
+
             i++;
             name.len++;
         }