]> granicus.if.org Git - postgresql/commit
* Stephen Frost (sfrost@snowman.net) wrote:
authorBruce Momjian <bruce@momjian.us>
Mon, 6 Mar 2006 17:59:42 +0000 (17:59 +0000)
committerBruce Momjian <bruce@momjian.us>
Mon, 6 Mar 2006 17:59:42 +0000 (17:59 +0000)
commita0af608adcd52b873fea1c446576b224a8b3e74d
tree4cc53ce129548e249ffd25b4a7a06c8dca659dc8
parenta643385eafb215220a560ef6e81e0ce8c41ff8a8
* Stephen Frost (sfrost@snowman.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.

Backpatch to 8.1.X.

Stephen Frost
src/interfaces/libpq/fe-auth.c