Tweak psql's \connect command to not downcase unquoted database and user

names.  This is a temporary measure to allow backwards compatibility with
7.2 and earlier pg_dump.  7.2.1 and later pg_dump will double-quote mixed
case names in \connect.  Once we feel that older dumps are not a problem
anymore, we can revert this change and treat \connect arguments as normal
SQL identifiers.

diff --git a/src/bin/psql/command.c b/src/bin/psql/command.c
index 5bd8a5348d6488278d8de0f59ee007e57d3e140b..2a078bb6b9596986d5d498cb507a37667ef98ae7 100644 (file)
--- a/src/bin/psql/command.c
+++ b/src/bin/psql/command.c
@@ -3,7 +3,7 @@
  *
  * Copyright 2000 by PostgreSQL Global Development Group
  *
- * $Header: /cvsroot/pgsql/src/bin/psql/command.c,v 1.65 2002/02/20 22:47:12 tgl Exp $
+ * $Header: /cvsroot/pgsql/src/bin/psql/command.c,v 1.66 2002/02/25 21:37:42 tgl Exp $
  */
 #include "postgres_fe.h"
 #include "command.h"
@@ -54,11 +54,17 @@ static backslashResult exec_command(const char *cmd,
             const char **continue_parse,
             PQExpBuffer query_buf);
 
+/* different ways for scan_option to handle parameter words */
 enum option_type
 {
-   OT_NORMAL, OT_SQLID, OT_FILEPIPE
+   OT_NORMAL,                  /* normal case */
+   OT_SQLID,                   /* treat as SQL identifier */
+   OT_SQLIDHACK,               /* SQL identifier, but don't downcase */
+   OT_FILEPIPE                 /* it's a file or pipe */
 };
-static char *scan_option(char **string, enum option_type type, char *quote, bool semicolon);
+
+static char *scan_option(char **string, enum option_type type,
+                        char *quote, bool semicolon);
 static char *unescape(const unsigned char *source, size_t len);
 
 static bool do_edit(const char *filename_arg, PQExpBuffer query_buf);
@@ -243,8 +249,17 @@ exec_command(const char *cmd,
        char        opt1q,
                    opt2q;
 
-       opt1 = scan_option(&string, OT_SQLID, &opt1q, true);
-       opt2 = scan_option(&string, OT_SQLID, &opt2q, true);
+       /*
+        * Ideally we should treat the arguments as SQL identifiers.  But for
+        * backwards compatibility with 7.2 and older pg_dump files, we have
+        * to take unquoted arguments verbatim (don't downcase them).
+        * For now, double-quoted arguments may be stripped of double quotes
+        * (as if SQL identifiers).  By 7.4 or so, pg_dump files can be
+        * expected to double-quote all mixed-case \connect arguments,
+        * and then we can get rid of OT_SQLIDHACK.
+        */
+       opt1 = scan_option(&string, OT_SQLIDHACK, &opt1q, true);
+       opt2 = scan_option(&string, OT_SQLIDHACK, &opt2q, true);
 
        if (opt2)
            /* gave username */
@@ -909,7 +924,7 @@ scan_option(char **string, enum option_type type, char *quote, bool semicolon)
                 * then we strip out the double quotes
                 */
 
-               if (type == OT_SQLID)
+               if (type == OT_SQLID || type == OT_SQLIDHACK)
                {
                    unsigned int k,
                                cc;
@@ -930,7 +945,6 @@ scan_option(char **string, enum option_type type, char *quote, bool semicolon)
                    }
                    return_val[cc] = '\0';
                }
-
                else
                {
                    strncpy(return_val, &options_string[pos], jj - pos + 1);