diff options
| author | Magnus Hagander <magnus@hagander.net> | 2007-07-12 14:13:06 +0000 |
|---|---|---|
| committer | Magnus Hagander <magnus@hagander.net> | 2007-07-12 14:13:06 +0000 |
| commit | c7cca1c8f7e9bfbd81b49107d88aa4a4ca2071d0 (patch) | |
| tree | d56874e07437762f8988a3e044f4e8bd83282e86 /src/interfaces/libpq/fe-auth.c | |
| parent | 3df98a9e2fcada34de3ec8325f83b82426014c6d (diff) | |
| download | postgresql-c7cca1c8f7e9bfbd81b49107d88aa4a4ca2071d0.tar.gz postgresql-c7cca1c8f7e9bfbd81b49107d88aa4a4ca2071d0.zip | |
Fix freenig of names in Kerberos when using MIT - need to use the
free function provided in the Kerberos library.
This fixes a very hard to track down heap corruption on windows
when using debug runtimes.
Diffstat (limited to 'src/interfaces/libpq/fe-auth.c')
| -rw-r--r-- | src/interfaces/libpq/fe-auth.c | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/src/interfaces/libpq/fe-auth.c b/src/interfaces/libpq/fe-auth.c index 7725f52f354..d1f1a5c480c 100644 --- a/src/interfaces/libpq/fe-auth.c +++ b/src/interfaces/libpq/fe-auth.c @@ -10,7 +10,7 @@ * exceed INITIAL_EXPBUFFER_SIZE (currently 256 bytes). * * IDENTIFICATION - * $PostgreSQL: pgsql/src/interfaces/libpq/fe-auth.c,v 1.121 2006/10/04 00:30:12 momjian Exp $ + * $PostgreSQL: pgsql/src/interfaces/libpq/fe-auth.c,v 1.121.2.1 2007/07/12 14:13:06 mha Exp $ * *------------------------------------------------------------------------- */ @@ -64,6 +64,18 @@ #endif /* + * Heimdal doesn't have a free function for unparsed names. Just pass it to + * standard free() which should work in these cases. + */ +#ifndef HAVE_KRB5_FREE_UNPARSED_NAME +static void +krb5_free_unparsed_name(krb5_context context, char *val) +{ + free(val); +} +#endif + +/* * pg_an_to_ln -- return the local name corresponding to an authentication * name * @@ -180,8 +192,8 @@ pg_krb5_destroy(struct krb5_info * info) { krb5_free_principal(info->pg_krb5_context, info->pg_krb5_client); krb5_cc_close(info->pg_krb5_context, info->pg_krb5_ccache); + krb5_free_unparsed_name(info->pg_krb5_context, info->pg_krb5_name); krb5_free_context(info->pg_krb5_context); - free(info->pg_krb5_name); } |