Re: Preliminary GSSAPI Patches - Mailing list pgsql-patches

From Stephen Frost
Subject Re: Preliminary GSSAPI Patches
Date
Msg-id 20070622162951.GY7531@tamriel.snowman.net
Whole thread Raw
In response to Re: Preliminary GSSAPI Patches  (Magnus Hagander <magnus@hagander.net>)
Responses Re: Preliminary GSSAPI Patches
List pgsql-patches
* Magnus Hagander (magnus@hagander.net) wrote:
> We enable the setting of the service name in the server configuration
> file, but we never use that variable anywhere. We do, however, use the
> service name on the client, in order to pick the correct key (and
> turning this off makes GSSAPI no longer work).
>
> If this is correct, we should not enable that parameter on the server.
> If it's not correct, we should be using it somewhere.

Uh, shouldn't you be acquiring the server credentials before accepting
the context?  That'd be done using gss_acquire_cred(), which takes the
service name (in gss_name_t structure) as an argument.  That would then
be passed in to gss_accept_sec_context() instead of using
GSS_C_NO_CREDENTIAL (in port->gss->cred).  I'm kind of suprised it's
working without that and rather curious as to what it's doing under the
hood to make that happen. :/

    Thanks,

        Stephen

Attachment

pgsql-patches by date:

Previous
From: Tom Lane
Date:
Subject: Re: Load Distributed Checkpoints, take 3
Next
From: Magnus Hagander
Date:
Subject: Re: Preliminary GSSAPI Patches