httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Alexei Kosut <ako...@organic.com>
Subject Re: WWW Form Bug Report: "access to index.html broken under certain circumstances" on Solaris 2.x (fwd)
Date Mon, 01 Jul 1996 16:49:18 GMT
On Mon, 1 Jul 1996, Cliff Skolnick wrote:

> Reason seems to be the auth_type calls in 
> sub_req_lookup_uri and sub_req_lookup_file.  The
> 'same' code in process_request_internal was
> changed from auth_type() -> some_auth_required();
> the same change should probably applied to the
> sub_req_lookup_* routines, too.

Yep, this sounds right to me. RST was the one who made that change in the
first place, what does he think?

> Looking at the code in the two sub_req_lookup_*
> functions, I also noticed that location_walk isn't
> called, which could be a separate problem.

Could be, but isn't. At least, not to my way of thinking.

> 
> *** apache-1.1b4-orig/src/http_request.c        Mon Jun 17 22:28:19 1996
> --- apache-1.1b4/src/http_request.c     Mon Jul  1 11:45:37 1996
> ***************
> *** 464,469 ****
> --- 464,472 ----
>       return rnew;
>   }
>   
> + 
> + static int some_auth_required (request_rec *r);
> + 
>   request_rec *sub_req_lookup_uri (char *new_file, request_rec *r)
>   {
>       request_rec *rnew;
> ***************
> *** 510,516 ****
>        */
>       
>       if ((res = directory_walk (rnew))
> !       || (!auth_type (rnew) ? 0 :
>              ((res = check_user_id (rnew)) || (res = check_auth (rnew))))
>         || (res = check_access (rnew))
>         || (res = find_types (rnew))
> --- 513,519 ----
>        */
>       
>       if ((res = directory_walk (rnew))
> !       || (!some_auth_required (rnew) ? 0 :
>              ((res = check_user_id (rnew)) || (res = check_auth (rnew))))
>         || (res = check_access (rnew))
>         || (res = find_types (rnew))
> ***************
> *** 553,559 ****
>          
>       if ((res = directory_walk (rnew))
>         || (res = check_access (rnew))
> !       || (!auth_type (rnew) ? 0 :
>              ((res = check_user_id (rnew)) && (res = check_auth (rnew))))
>         || (res = find_types (rnew))
>         || (res = run_fixups (rnew))
> --- 556,562 ----
>          
>       if ((res = directory_walk (rnew))
>         || (res = check_access (rnew))
> !       || (!some_auth_required (rnew) ? 0 :
>              ((res = check_user_id (rnew)) && (res = check_auth (rnew))))
>         || (res = find_types (rnew))
>         || (res = run_fixups (rnew))
> 
> --
> 
> Backtrace:
> --
> 
> --
> 
> 

-- Alexei Kosut <akosut@organic.com>            The Apache HTTP Server 
   http://www.nueva.pvt.k12.ca.us/~akosut/      http://www.apache.org/


Mime
View raw message