]> granicus.if.org Git - postgresql/commitdiff
Clean up for warning:
authorJoe Conway <mail@joeconway.com>
Wed, 12 Jul 2006 02:31:56 +0000 (02:31 +0000)
committerJoe Conway <mail@joeconway.com>
Wed, 12 Jul 2006 02:31:56 +0000 (02:31 +0000)
  fe-auth.c:573: warning: passing argument 1 of 'free' discards qualifiers
  from pointer target type
pg_krb5_authname used to return a (const char *) to memory allocated by
krb. Somewhere along the lines this was changed so that a copy was
made, returned, and freed instead. However the const modifier was never
removed.

src/interfaces/libpq/fe-auth.c

index 501cc39d0eec7e2b2d2d036757f88dc749477c01..1b8ac726ddb25639d90ca255a0242bd9156937dd 100644 (file)
@@ -10,7 +10,7 @@
  * exceed INITIAL_EXPBUFFER_SIZE (currently 256 bytes).
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/interfaces/libpq/fe-auth.c,v 1.115 2006/06/20 19:56:52 tgl Exp $
+ *       $PostgreSQL: pgsql/src/interfaces/libpq/fe-auth.c,v 1.116 2006/07/12 02:31:56 joe Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -188,10 +188,10 @@ pg_krb5_destroy(struct krb5_info *info)
 
 
 /*
- * pg_krb5_authname -- returns a pointer to static space containing whatever
- *                                        name the user has authenticated to the system
 */
-static const char *
+ * pg_krb5_authname -- returns a copy of whatever name the user
+ *                                        has authenticated to the system, or NULL
+ */
+static char *
 pg_krb5_authname(char *PQerrormsg)
 {
        char *tmp_name;
@@ -520,7 +520,7 @@ char *
 pg_fe_getauthname(char *PQerrormsg)
 {
 #ifdef KRB5
-       const char *krb5_name = NULL;
+       char       *krb5_name = NULL;
 #endif
        const char *name = NULL;
        char       *authn;