The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]

форумы  помощь  поиск  регистрация  майллист  вход/выход  слежка  RSS
"Тематический каталог: Патч для mod_rewrite (избавляет от зац..."
Вариант для распечатки  
Пред. тема | След. тема 
Форумы Разговоры, обсуждение новостей (Public)
Изначальное сообщение [ Отслеживать ]

"Тематический каталог: Патч для mod_rewrite (избавляет от зац..."  
Сообщение от auto_topic (ok) on 31-Мрт-08, 21:11 
Обсуждение статьи тематического каталога: Патч для mod_rewrite (избавляет от зацикливания) (mod_rewrite apache p

Ссылка на текст статьи: http://www.opennet.me/base/patch/mod_rewrite_loop.txt.html

Высказать мнение | Ответить | Правка | Cообщить модератору

 Оглавление

Сообщения по теме [Сортировка по времени | RSS]


1. "Патч для mod_rewrite (избавляет от зацикливания) (mod_rewrit..."  
Сообщение от Michael Shigorin email(ok) on 31-Мрт-08, 21:11 
Надо же, до сих пор прикладывается (с -F 60).

Для удобства -- ссылка на письмо:
http://www.lexa.ru/apache-talk/msg06354.html
оригинальный патч:
http://fly.osdn.org.ua/~mike/works/apache/apache-1.3.23-unlo...
rediff для 1.3.41 (похоже, включу в сборку для ALT Linux):
http://fly.osdn.org.ua/~mike/works/apache/apache-1.3.41-unlo...
и на всякий вклеиваю его:

diff -Naur apache_1.3.41-orig/src/modules/standard/mod_rewrite.c apache_1.3.41/src/modules/standard/mod_rewrite.c
--- apache_1.3.41-orig/src/modules/standard/mod_rewrite.c    2006-07-27 20:09:03 +0300
+++ apache_1.3.41/src/modules/standard/mod_rewrite.c    2008-03-31 19:33:24 +0300
@@ -62,6 +62,17 @@
static LONG locking_sem = 0;
#endif

+#ifndef REWRITE_MAX_RULES
+#define REWRITE_MAX_RULES 300
+#endif
+#ifndef REWRITE_MAX_REDIRECTS
+#define REWRITE_MAX_REDIRECTS 50
+#endif
+
+#define REWRITE_FIXUP_NOTE "rewrite_fixup_redirect"
+
+int redirect_count;        /* Счетчик циклов для fixup */
+
/*
** +-------------------------------------------------------+
** |                                                       |
@@ -1345,6 +1356,34 @@
         return FORBIDDEN;
     }

+    /* Проверим и установим флаг первого запроса */
+    {
+        request_rec *q;
+        int note_found = 0;
+        for (q = r; q != NULL; q = q->prev) {
+            if( ap_table_get(q->notes, REWRITE_FIXUP_NOTE) != NULL ) {
+                note_found = 1;
+                break;
+            }
+        }
+        ap_table_set(r->notes, REWRITE_FIXUP_NOTE, "OK");
+        
+        if( !note_found ) {
+            redirect_count = REWRITE_MAX_REDIRECTS;
+            rewritelog(r, 3, "setting redirect count notice" );
+        }
+    }
+
+    /* Проверим количество циклов */
+    if( redirect_count-- <= 0 ) {
+        /* очень много итераций */
+        ap_log_rerror(APLOG_MARK, APLOG_ERR, r,
+            "too many rewrite internal redirects: %s", r->uri);
+        return HTTP_INTERNAL_SERVER_ERROR;
+    }
+
+    rewritelog(r, 3, "internal redirect count: %d", redirect_count );
+    
     /*
      *  remember the current filename before rewriting for later check
      *  to prevent deadlooping because of internal redirects
@@ -1359,6 +1398,13 @@
     if (rulestatus) {
         unsigned skip;

+        if (strlen(r->filename) > 20 &&
+            strncmp(r->filename, "too_many_iterations:", 20) == 0) {
+            /* очень много итераций */
+            ap_log_rerror(APLOG_MARK, APLOG_ERR, r,
+                "too many rewrite rules (possible loop): %s", r->uri);
+            return HTTP_INTERNAL_SERVER_ERROR;
+        } else
         if (strlen(r->filename) > 6 &&
             strncmp(r->filename, "proxy:", 6) == 0) {
             /* it should go on as an internal proxy request */
@@ -1671,6 +1717,7 @@
     int changed;
     int rc;
     int s;
+    int loopcount = REWRITE_MAX_RULES;

     /*
      *  Iterate over all existing rules
@@ -1679,6 +1726,13 @@
     changed = 0;
     loop:
     for (i = 0; i < rewriterules->nelts; i++) {
+        if( loopcount-- <= 0 ) {
+            rewritelog(r, 2, "too many iterations '%s'", r->filename);
+            r->filename = ap_pstrcat(r->pool, "too_many_iterations:", r->filename, NULL);
+            changed = ACTION_NORMAL;
+            break;
+        }
+        
         p = &entries[i];

         /*

Высказать мнение | Ответить | Правка | Наверх | Cообщить модератору

Архив | Удалить

Индекс форумов | Темы | Пред. тема | След. тема
Оцените тред (1=ужас, 5=супер)? [ 1 | 2 | 3 | 4 | 5 ] [Рекомендовать для помещения в FAQ]




Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру