Greetings,

* Stephen Frost (sfrost(at)snowman(dot)net) wrote:
>   I've now tested this patch at home w/ 8.2HEAD and it seems to fix the
>   bug.  I plan on testing it under 8.1.2 at work tommorow with
>   mod_auth_krb5, etc, and expect it'll work there.  Assuming all goes
>   well and unless someone objects I'll forward the patch to -patches.
>   It'd be great to have this fixed as it'll allow us to use Kerberos to
>   authenticate to phppgadmin and other web-based tools which use
>   Postgres.

  While playing with this patch under 8.1.2 at home I discovered a
  mistake in how I manually applied one of the hunks to fe-auth.c.
  Basically, the base code had changed and so the patch needed to be 
  modified slightly.  This is because the code no longer either has a
  freeable pointer under 'name' or has 'name' as NULL.

  The attached patch correctly frees the string from pg_krb5_authname
  (where it had been strdup'd) if and only if pg_krb5_authname returned
  a string (as opposed to falling through and having name be set using
  name = pw->name;).  Also added a comment to this effect. 
  Please review.

  	Thanks,

		Stephen

In response to

• Re: Krb5 & multiple DB connections at 2006-02-05 17:53:01 from Stephen Frost

Responses

• Re: [HACKERS] Krb5 & multiple DB connections at 2006-02-06 18:55:20 from Stephen Frost
• Re: Krb5 & multiple DB connections at 2006-02-12 20:05:09 from Bruce Momjian
• Re: Krb5 & multiple DB connections at 2006-03-06 17:59:37 from Bruce Momjian

pgsql-hackers by date

Next:	From: Josh Berkus	Date: 2006-02-05 21:13:31
	Subject: Re: Copy From & Insert UNLESS
Previous:	From: Andrew Dunstan	Date: 2006-02-05 19:58:19
	Subject: Re: drop if exists remainder

pgsql-patches by date

Next:	From: Tom Lane	Date: 2006-02-05 21:00:24
	Subject: Re: pg_restore COPY error handling
Previous:	From: Andrew Dunstan	Date: 2006-02-05 19:58:19
	Subject: Re: drop if exists remainder