Re: Bug review request

From: Henrik Nordstrom <henrik@dont-contact.us>
Date: Tue, 15 Aug 2006 10:22:37 +0200

On Mon, 2006-08-14 at 10:32 +0200, Guido Serassio wrote:

> err->request = requestLink(request);
>
> But not always err->request is linked to a request, it could be a problem ?

You have found places where err->request is assigned without a link?

> > ->src_addr
>
> Some confusion here, sometimes:
>
> err->src_addr = conn->peer.sin_addr;
> or err->src_addr = http->conn->peer.sin_addr;
>
> some other times:
>
> err->src_addr = request->client_addr;
>
> What is correct ?

request when you have a request. The few places where errorCon is called
before we have a request NULL can be passed, and the caller has to fill
in the available details after as is done now.

> In client_side.c (clientAccessCheckDone()) there is:
>
> if (http->conn->auth_user_request)
> err->auth_user_request = http->conn->auth_user_request;
> else if (http->request->auth_user_request)
> err->auth_user_request = http->request->auth_user_request;
>
> where http is a clientHttpRequest.

I think we always have request->auth_user_request now even on connection
oriented auth. If not that should be easy to fix.

Regards
Henrik
Received on Tue Aug 15 2006 - 02:21:56 MDT

This archive was generated by hypermail pre-2.1.9 : Fri Sep 01 2006 - 12:00:03 MDT