https://bz.apache.org/bugzilla/show_bug.cgi?id=69260

--- Comment #10 from venki <venkigoud...@gmail.com> ---
I see different in the lines between provided patch and 2.4.62 code. 

2.4.62 lines:
-------------------------
   5223         if (ACTION_STATUS == rulestatus) {
   5224             int n = r->status;
   5225
   5226             r->status = HTTP_OK;
   5227             return n;
   5228         }
   5229         else if (ACTION_STATUS_SET == rulestatus) {
   5230             return r->status;
   5231         }
   5232
   5233         if (to_proxyreq) {
   5234             /* it should go on as an internal proxy request */
   5235
   5236             /* make sure the QUERY_STRING and
   5237              * PATH_INFO parts get incorporated
   5238              * (r->path_info was already appended by the
   5239              * rewriting engine because of the per-dir context!)
   5240              */
   5241             if (r->args != NULL) {
   5242                 /* see proxy_http:proxy_http_canon() */
   5243                 r->filename = apr_pstrcat(r->pool, r->filename,
   5244                                           "?", r->args, NULL);
   5245             }


patch file:
-----------------
     return HTTP_FORBIDDEN;
5393                }   5379                }
5394    
5380    
5395                /* make sure the QUERY_STRING and   5381                if
(rulestatus == ACTION_NOESCAPE) {
5396                 * PATH_INFO parts get incorporated 5382                   
apr_table_setn(r->notes, "proxy-nocanon", "1");
5383                }
5384    
5385                /* make sure the QUERY_STRING gets incorporated, but only
5386                 * if we do not escape the target. Otherwise the mod_proxy
5387                 * canon handler will take care to do this.
5397                 * (r->path_info was already appended by the        5388   
             * (r->path_info was already appended by the
5398                 * rewriting engine because of the per-dir context!)       
5389                 * rewriting engine because of the per-dir context!)
5399                 */ 5390                 */
5400                if (r->args != NULL) {      5391                if
((r->args != NULL) && (rulestatus == ACTION_NOESCAPE)) {
5401                    /* see proxy_http:proxy_http_canon() */ 5392           
        /* see proxy_http:proxy_http_canon() */
5402                    r->filename = apr_pstrcat(r->pool, r->filename, 5393   
                r->filename = apr_pstrcat(r->pool, r->filename,
5403                                              "?", r->args, NULL);  5394   
                                          "?", r->args, NULL);
   5246


Could you pls suggest the line numbers and above rulestatus checks.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscr...@httpd.apache.org
For additional commands, e-mail: bugs-h...@httpd.apache.org

Reply via email to