]> granicus.if.org Git - postgresql/blobdiff - src/bin/pg_dump/pg_backup_db.c
pg_dump: Reduce use of global variables
[postgresql] / src / bin / pg_dump / pg_backup_db.c
index a5ca77c545b70dc71e35ffc75200ca2c773d4b5e..07313f40df520f6ddb9e8980651f7f78f9380be4 100644 (file)
@@ -5,86 +5,63 @@
  *     Implements the basic DB functions used by the archiver.
  *
  * IDENTIFICATION
- *       $PostgreSQL: pgsql/src/bin/pg_dump/pg_backup_db.c,v 1.73 2006/07/18 17:42:00 momjian Exp $
+ *       src/bin/pg_dump/pg_backup_db.c
  *
  *-------------------------------------------------------------------------
  */
+#include "postgres_fe.h"
 
-#include "pg_backup_db.h"
 #include "dumputils.h"
+#include "pg_backup_archiver.h"
+#include "pg_backup_db.h"
+#include "pg_backup_utils.h"
 
 #include <unistd.h>
-
 #include <ctype.h>
-
 #ifdef HAVE_TERMIOS_H
 #include <termios.h>
 #endif
 
-#ifndef HAVE_STRDUP
-#include "strdup.h"
-#endif
 
+#define DB_MAX_ERR_STMT 128
+
+/* translator: this is a module name */
 static const char *modulename = gettext_noop("archiver (db)");
 
-static void _check_database_version(ArchiveHandle *AH, bool ignoreVersion);
+static void _check_database_version(ArchiveHandle *AH);
 static PGconn *_connectDB(ArchiveHandle *AH, const char *newdbname, const char *newUser);
 static void notice_processor(void *arg, const char *message);
-static char *_sendSQLLine(ArchiveHandle *AH, char *qry, char *eos);
-static char *_sendCopyLine(ArchiveHandle *AH, char *qry, char *eos);
-
-static bool _isIdentChar(unsigned char c);
-static bool _isDQChar(unsigned char c, bool atStart);
-
-#define DB_MAX_ERR_STMT 128
-
-static int
-_parse_version(ArchiveHandle *AH, const char *versionString)
-{
-       int                     v;
-
-       v = parse_version(versionString);
-       if (v < 0)
-               die_horribly(AH, modulename, "could not parse version string \"%s\"\n", versionString);
-
-       return v;
-}
 
 static void
-_check_database_version(ArchiveHandle *AH, bool ignoreVersion)
+_check_database_version(ArchiveHandle *AH)
 {
-       int                     myversion;
        const char *remoteversion_str;
        int                     remoteversion;
 
-       myversion = _parse_version(AH, PG_VERSION);
-
        remoteversion_str = PQparameterStatus(AH->connection, "server_version");
-       if (!remoteversion_str)
-               die_horribly(AH, modulename, "could not get server_version from libpq\n");
+       remoteversion = PQserverVersion(AH->connection);
+       if (remoteversion == 0 || !remoteversion_str)
+               exit_horribly(modulename, "could not get server_version from libpq\n");
 
-       remoteversion = _parse_version(AH, remoteversion_str);
-
-       AH->public.remoteVersionStr = strdup(remoteversion_str);
+       AH->public.remoteVersionStr = pg_strdup(remoteversion_str);
        AH->public.remoteVersion = remoteversion;
+       if (!AH->archiveRemoteVersion)
+               AH->archiveRemoteVersion = AH->public.remoteVersionStr;
 
-       if (myversion != remoteversion
+       if (remoteversion != PG_VERSION_NUM
                && (remoteversion < AH->public.minRemoteVersion ||
                        remoteversion > AH->public.maxRemoteVersion))
        {
                write_msg(NULL, "server version: %s; %s version: %s\n",
                                  remoteversion_str, progname, PG_VERSION);
-               if (ignoreVersion)
-                       write_msg(NULL, "proceeding despite version mismatch\n");
-               else
-                       die_horribly(AH, NULL, "aborting because of version mismatch  (Use the -i option to proceed anyway.)\n");
+               exit_horribly(NULL, "aborting because of server version mismatch\n");
        }
 }
 
 /*
  * Reconnect to the server.  If dbname is not NULL, use that database,
  * else the one associated with the archive handle.  If username is
- * not NULL, use that user name, else the one from the handle. If
+ * not NULL, use that user name, else the one from the handle.  If
  * both the database and the user match the existing connection already,
  * nothing will be done.
  *
@@ -122,78 +99,103 @@ ReconnectToServer(ArchiveHandle *AH, const char *dbname, const char *username)
 
 /*
  * Connect to the db again.
+ *
+ * Note: it's not really all that sensible to use a single-entry password
+ * cache if the username keeps changing.  In current usage, however, the
+ * username never does change, so one savedPassword is sufficient.  We do
+ * update the cache on the off chance that the password has changed since the
+ * start of the run.
  */
 static PGconn *
 _connectDB(ArchiveHandle *AH, const char *reqdb, const char *requser)
 {
-       int                     need_pass;
        PGconn     *newConn;
-       char       *password = NULL;
-       int                     badPwd = 0;
-       int                     noPwd = 0;
-       char       *newdb;
-       char       *newuser;
+       const char *newdb;
+       const char *newuser;
+       char       *password = AH->savedPassword;
+       bool            new_pass;
 
        if (!reqdb)
                newdb = PQdb(AH->connection);
        else
-               newdb = (char *) reqdb;
+               newdb = reqdb;
 
-       if (!requser || (strlen(requser) == 0))
+       if (!requser || strlen(requser) == 0)
                newuser = PQuser(AH->connection);
        else
-               newuser = (char *) requser;
+               newuser = requser;
 
-       ahlog(AH, 1, "connecting to database \"%s\" as user \"%s\"\n", newdb, newuser);
+       ahlog(AH, 1, "connecting to database \"%s\" as user \"%s\"\n",
+                 newdb, newuser);
 
-       if (AH->requirePassword)
+       if (AH->promptPassword == TRI_YES && password == NULL)
        {
                password = simple_prompt("Password: ", 100, false);
                if (password == NULL)
-                       die_horribly(AH, modulename, "out of memory\n");
+                       exit_horribly(modulename, "out of memory\n");
        }
 
        do
        {
-               need_pass = false;
-               newConn = PQsetdbLogin(PQhost(AH->connection), PQport(AH->connection),
-                                                          NULL, NULL, newdb,
-                                                          newuser, password);
+#define PARAMS_ARRAY_SIZE      7
+               const char **keywords = pg_malloc(PARAMS_ARRAY_SIZE * sizeof(*keywords));
+               const char **values = pg_malloc(PARAMS_ARRAY_SIZE * sizeof(*values));
+
+               keywords[0] = "host";
+               values[0] = PQhost(AH->connection);
+               keywords[1] = "port";
+               values[1] = PQport(AH->connection);
+               keywords[2] = "user";
+               values[2] = newuser;
+               keywords[3] = "password";
+               values[3] = password;
+               keywords[4] = "dbname";
+               values[4] = newdb;
+               keywords[5] = "fallback_application_name";
+               values[5] = progname;
+               keywords[6] = NULL;
+               values[6] = NULL;
+
+               new_pass = false;
+               newConn = PQconnectdbParams(keywords, values, true);
+
+               free(keywords);
+               free(values);
+
                if (!newConn)
-                       die_horribly(AH, modulename, "failed to reconnect to database\n");
+                       exit_horribly(modulename, "failed to reconnect to database\n");
 
                if (PQstatus(newConn) == CONNECTION_BAD)
                {
-                       noPwd = (strcmp(PQerrorMessage(newConn),
-                                                       PQnoPasswordSupplied) == 0);
-                       badPwd = (strncmp(PQerrorMessage(newConn),
-                                               "Password authentication failed for user", 39) == 0);
+                       if (!PQconnectionNeedsPassword(newConn))
+                               exit_horribly(modulename, "could not reconnect to database: %s",
+                                                         PQerrorMessage(newConn));
+                       PQfinish(newConn);
 
-                       if (noPwd || badPwd)
-                       {
-                               if (badPwd)
-                                       fprintf(stderr, "Password incorrect\n");
+                       if (password)
+                               fprintf(stderr, "Password incorrect\n");
+
+                       fprintf(stderr, "Connecting to %s as %s\n",
+                                       newdb, newuser);
 
-                               fprintf(stderr, "Connecting to %s as %s\n",
-                                               newdb, newuser);
+                       if (password)
+                               free(password);
 
-                               need_pass = true;
-                               if (password)
-                                       free(password);
+                       if (AH->promptPassword != TRI_NO)
                                password = simple_prompt("Password: ", 100, false);
-                       }
                        else
-                               die_horribly(AH, modulename, "could not reconnect to database: %s",
-                                                        PQerrorMessage(newConn));
-                       PQfinish(newConn);
+                               exit_horribly(modulename, "connection needs password\n");
+
+                       if (password == NULL)
+                               exit_horribly(modulename, "out of memory\n");
+                       new_pass = true;
                }
-       } while (need_pass);
+       } while (new_pass);
 
-       if (password)
-               free(password);
+       AH->savedPassword = password;
 
        /* check for version mismatch */
-       _check_database_version(AH, true);
+       _check_database_version(AH);
 
        PQsetNoticeProcessor(newConn, notice_processor, NULL);
 
@@ -205,32 +207,33 @@ _connectDB(ArchiveHandle *AH, const char *reqdb, const char *requser)
  * Make a database connection with the given parameters.  The
  * connection handle is returned, the parameters are stored in AHX.
  * An interactive password prompt is automatically issued if required.
+ *
+ * Note: it's not really all that sensible to use a single-entry password
+ * cache if the username keeps changing.  In current usage, however, the
+ * username never does change, so one savedPassword is sufficient.
  */
-PGconn *
+void
 ConnectDatabase(Archive *AHX,
                                const char *dbname,
                                const char *pghost,
                                const char *pgport,
                                const char *username,
-                               const int reqPwd,
-                               const int ignoreVersion)
+                               trivalue prompt_password)
 {
        ArchiveHandle *AH = (ArchiveHandle *) AHX;
-       char       *password = NULL;
-       bool            need_pass = false;
+       char       *password = AH->savedPassword;
+       bool            new_pass;
 
        if (AH->connection)
-               die_horribly(AH, modulename, "already connected to a database\n");
+               exit_horribly(modulename, "already connected to a database\n");
 
-       if (reqPwd)
+       if (prompt_password == TRI_YES && password == NULL)
        {
                password = simple_prompt("Password: ", 100, false);
                if (password == NULL)
-                       die_horribly(AH, modulename, "out of memory\n");
-               AH->requirePassword = true;
+                       exit_horribly(modulename, "out of memory\n");
        }
-       else
-               AH->requirePassword = false;
+       AH->promptPassword = prompt_password;
 
        /*
         * Start the connection.  Loop until we have a password if requested by
@@ -238,41 +241,95 @@ ConnectDatabase(Archive *AHX,
         */
        do
        {
-               need_pass = false;
-               AH->connection = PQsetdbLogin(pghost, pgport, NULL, NULL,
-                                                                         dbname, username, password);
+#define PARAMS_ARRAY_SIZE      7
+               const char **keywords = pg_malloc(PARAMS_ARRAY_SIZE * sizeof(*keywords));
+               const char **values = pg_malloc(PARAMS_ARRAY_SIZE * sizeof(*values));
+
+               keywords[0] = "host";
+               values[0] = pghost;
+               keywords[1] = "port";
+               values[1] = pgport;
+               keywords[2] = "user";
+               values[2] = username;
+               keywords[3] = "password";
+               values[3] = password;
+               keywords[4] = "dbname";
+               values[4] = dbname;
+               keywords[5] = "fallback_application_name";
+               values[5] = progname;
+               keywords[6] = NULL;
+               values[6] = NULL;
+
+               new_pass = false;
+               AH->connection = PQconnectdbParams(keywords, values, true);
+
+               free(keywords);
+               free(values);
 
                if (!AH->connection)
-                       die_horribly(AH, modulename, "failed to connect to database\n");
+                       exit_horribly(modulename, "failed to connect to database\n");
 
                if (PQstatus(AH->connection) == CONNECTION_BAD &&
-                strcmp(PQerrorMessage(AH->connection), PQnoPasswordSupplied) == 0 &&
-                       !feof(stdin))
+                       PQconnectionNeedsPassword(AH->connection) &&
+                       password == NULL &&
+                       prompt_password != TRI_NO)
                {
                        PQfinish(AH->connection);
-                       need_pass = true;
-                       free(password);
-                       password = NULL;
                        password = simple_prompt("Password: ", 100, false);
+                       if (password == NULL)
+                               exit_horribly(modulename, "out of memory\n");
+                       new_pass = true;
                }
-       } while (need_pass);
+       } while (new_pass);
 
-       if (password)
-               free(password);
+       AH->savedPassword = password;
 
        /* check to see that the backend connection was successfully made */
        if (PQstatus(AH->connection) == CONNECTION_BAD)
-               die_horribly(AH, modulename, "connection to database \"%s\" failed: %s",
-                                        PQdb(AH->connection), PQerrorMessage(AH->connection));
+               exit_horribly(modulename, "connection to database \"%s\" failed: %s",
+                                         PQdb(AH->connection) ? PQdb(AH->connection) : "",
+                                         PQerrorMessage(AH->connection));
 
        /* check for version mismatch */
-       _check_database_version(AH, ignoreVersion);
+       _check_database_version(AH);
 
        PQsetNoticeProcessor(AH->connection, notice_processor, NULL);
+}
 
-       return AH->connection;
+/*
+ * Close the connection to the database and also cancel off the query if we
+ * have one running.
+ */
+void
+DisconnectDatabase(Archive *AHX)
+{
+       ArchiveHandle *AH = (ArchiveHandle *) AHX;
+       PGcancel   *cancel;
+       char            errbuf[1];
+
+       if (!AH->connection)
+               return;
+
+       if (PQtransactionStatus(AH->connection) == PQTRANS_ACTIVE)
+       {
+               if ((cancel = PQgetCancel(AH->connection)))
+               {
+                       PQcancel(cancel, errbuf, sizeof(errbuf));
+                       PQfreeCancel(cancel);
+               }
+       }
+
+       PQfinish(AH->connection);
+       AH->connection = NULL;
 }
 
+PGconn *
+GetConnection(Archive *AHX)
+{
+       ArchiveHandle *AH = (ArchiveHandle *) AHX;
+
+       return AH->connection;
+}
 
 static void
 notice_processor(void *arg, const char *message)
@@ -280,30 +337,69 @@ notice_processor(void *arg, const char *message)
        write_msg(NULL, "%s", message);
 }
 
+/* Like exit_horribly(), but with a complaint about a particular query. */
+static void
+die_on_query_failure(ArchiveHandle *AH, const char *modulename, const char *query)
+{
+       write_msg(modulename, "query failed: %s",
+                         PQerrorMessage(AH->connection));
+       exit_horribly(modulename, "query was: %s\n", query);
+}
+
+void
+ExecuteSqlStatement(Archive *AHX, const char *query)
+{
+       ArchiveHandle *AH = (ArchiveHandle *) AHX;
+       PGresult   *res;
+
+       res = PQexec(AH->connection, query);
+       if (PQresultStatus(res) != PGRES_COMMAND_OK)
+               die_on_query_failure(AH, modulename, query);
+       PQclear(res);
+}
 
-/* Public interface */
-/* Convenience function to send a query. Monitors result to handle COPY statements */
-static int
-ExecuteSqlCommand(ArchiveHandle *AH, PQExpBuffer qry, char *desc)
+PGresult *
+ExecuteSqlQuery(Archive *AHX, const char *query, ExecStatusType status)
+{
+       ArchiveHandle *AH = (ArchiveHandle *) AHX;
+       PGresult   *res;
+
+       res = PQexec(AH->connection, query);
+       if (PQresultStatus(res) != status)
+               die_on_query_failure(AH, modulename, query);
+       return res;
+}
+
+/*
+ * Convenience function to send a query.
+ * Monitors result to detect COPY statements
+ */
+static void
+ExecuteSqlCommand(ArchiveHandle *AH, const char *qry, const char *desc)
 {
        PGconn     *conn = AH->connection;
        PGresult   *res;
        char            errStmt[DB_MAX_ERR_STMT];
 
-       /* fprintf(stderr, "Executing: '%s'\n\n", qry->data); */
-       res = PQexec(conn, qry->data);
-       if (!res)
-               die_horribly(AH, modulename, "%s: no result from server\n", desc);
+#ifdef NOT_USED
+       fprintf(stderr, "Executing: '%s'\n\n", qry);
+#endif
+       res = PQexec(conn, qry);
 
-       if (PQresultStatus(res) != PGRES_COMMAND_OK && PQresultStatus(res) != PGRES_TUPLES_OK)
+       switch (PQresultStatus(res))
        {
-               if (PQresultStatus(res) == PGRES_COPY_IN)
-               {
+               case PGRES_COMMAND_OK:
+               case PGRES_TUPLES_OK:
+               case PGRES_EMPTY_QUERY:
+                       /* A-OK */
+                       break;
+               case PGRES_COPY_IN:
+                       /* Assume this is an expected result */
                        AH->pgCopyIn = true;
-               }
-               else
-               {
-                       strncpy(errStmt, qry->data, DB_MAX_ERR_STMT);
+                       break;
+               default:
+                       /* trouble */
+                       strncpy(errStmt, qry, DB_MAX_ERR_STMT);
                        if (errStmt[DB_MAX_ERR_STMT - 1] != '\0')
                        {
                                errStmt[DB_MAX_ERR_STMT - 4] = '.';
@@ -311,231 +407,81 @@ ExecuteSqlCommand(ArchiveHandle *AH, PQExpBuffer qry, char *desc)
                                errStmt[DB_MAX_ERR_STMT - 2] = '.';
                                errStmt[DB_MAX_ERR_STMT - 1] = '\0';
                        }
-                       warn_or_die_horribly(AH, modulename, "%s: %s    Command was: %s\n",
-                                                                desc, PQerrorMessage(AH->connection),
-                                                                errStmt);
-               }
+                       warn_or_exit_horribly(AH, modulename, "%s: %s    Command was: %s\n",
+                                                                 desc, PQerrorMessage(conn), errStmt);
+                       break;
        }
 
        PQclear(res);
-
-       return strlen(qry->data);
 }
 
+
 /*
- * Used by ExecuteSqlCommandBuf to send one buffered line when running a COPY command.
+ * Process non-COPY table data (that is, INSERT commands).
+ *
+ * The commands have been run together as one long string for compressibility,
+ * and we are receiving them in bufferloads with arbitrary boundaries, so we
+ * have to locate command boundaries and save partial commands across calls.
+ * All state must be kept in AH->sqlparse, not in local variables of this
+ * routine.  We assume that AH->sqlparse was filled with zeroes when created.
+ *
+ * We have to lex the data to the extent of identifying literals and quoted
+ * identifiers, so that we can recognize statement-terminating semicolons.
+ * We assume that INSERT data will not contain SQL comments, E'' literals,
+ * or dollar-quoted strings, so this is much simpler than a full SQL lexer.
+ *
+ * Note: when restoring from a pre-9.0 dump file, this code is also used to
+ * process BLOB COMMENTS data, which has the same problem of containing
+ * multiple SQL commands that might be split across bufferloads.  Fortunately,
+ * that data won't contain anything complicated to lex either.
  */
-static char *
-_sendCopyLine(ArchiveHandle *AH, char *qry, char *eos)
+static void
+ExecuteSimpleCommands(ArchiveHandle *AH, const char *buf, size_t bufLen)
 {
-       size_t          loc;                    /* Location of next newline */
-       int                     pos = 0;                /* Current position */
-       int                     sPos = 0;               /* Last pos of a slash char */
-       int                     isEnd = 0;
-
-       /* loop to find unquoted newline ending the line of COPY data */
-       for (;;)
-       {
-               loc = strcspn(&qry[pos], "\n") + pos;
-
-               /* If no match, then wait */
-               if (loc >= (eos - qry)) /* None found */
-               {
-                       appendBinaryPQExpBuffer(AH->pgCopyBuf, qry, (eos - qry));
-                       return eos;
-               }
-
-               /*
-                * fprintf(stderr, "Found cr at %d, prev char was %c, next was %c\n",
-                * loc, qry[loc-1], qry[loc+1]);
-                */
-
-               /* Count the number of preceding slashes */
-               sPos = loc;
-               while (sPos > 0 && qry[sPos - 1] == '\\')
-                       sPos--;
+       const char *qry = buf;
+       const char *eos = buf + bufLen;
 
-               sPos = loc - sPos;
+       /* initialize command buffer if first time through */
+       if (AH->sqlparse.curCmd == NULL)
+               AH->sqlparse.curCmd = createPQExpBuffer();
 
-               /*
-                * If an odd number of preceding slashes, then \n was escaped so set
-                * the next search pos, and loop (if any left).
-                */
-               if ((sPos & 1) == 1)
-               {
-                       /* fprintf(stderr, "cr was escaped\n"); */
-                       pos = loc + 1;
-                       if (pos >= (eos - qry))
-                       {
-                               appendBinaryPQExpBuffer(AH->pgCopyBuf, qry, (eos - qry));
-                               return eos;
-                       }
-               }
-               else
-                       break;
-       }
-
-       /* We found an unquoted newline */
-       qry[loc] = '\0';
-       appendPQExpBuffer(AH->pgCopyBuf, "%s\n", qry);
-       isEnd = (strcmp(AH->pgCopyBuf->data, "\\.\n") == 0);
-
-       /*
-        * Note that we drop the data on the floor if libpq has failed to
-        * enter COPY mode; this allows us to behave reasonably when trying
-        * to continue after an error in a COPY command.
-        */
-       if (AH->pgCopyIn &&
-               PQputCopyData(AH->connection, AH->pgCopyBuf->data,
-                                         AH->pgCopyBuf->len) <= 0)
-               die_horribly(AH, modulename, "error returned by PQputCopyData: %s",
-                                        PQerrorMessage(AH->connection));
-
-       resetPQExpBuffer(AH->pgCopyBuf);
-
-       if (isEnd && AH->pgCopyIn)
+       for (; qry < eos; qry++)
        {
-               PGresult   *res;
-
-               if (PQputCopyEnd(AH->connection, NULL) <= 0)
-                       die_horribly(AH, modulename, "error returned by PQputCopyEnd: %s",
-                                                PQerrorMessage(AH->connection));
-
-               /* Check command status and return to normal libpq state */
-               res = PQgetResult(AH->connection);
-               if (PQresultStatus(res) != PGRES_COMMAND_OK)
-                       warn_or_die_horribly(AH, modulename, "COPY failed: %s",
-                                                                PQerrorMessage(AH->connection));
-               PQclear(res);
+               char            ch = *qry;
 
-               AH->pgCopyIn = false;
-       }
+               /* For neatness, we skip any newlines between commands */
+               if (!(ch == '\n' && AH->sqlparse.curCmd->len == 0))
+                       appendPQExpBufferChar(AH->sqlparse.curCmd, ch);
 
-       return qry + loc + 1;
-}
-
-/*
- * Used by ExecuteSqlCommandBuf to send one buffered line of SQL
- * (not data for the copy command).
- */
-static char *
-_sendSQLLine(ArchiveHandle *AH, char *qry, char *eos)
-{
-       /*
-        * The following is a mini state machine to assess the end of an SQL
-        * statement. It really only needs to parse good SQL, or at least that's
-        * the theory... End-of-statement is assumed to be an unquoted,
-        * un-commented semi-colon that's not within any parentheses.
-        *
-        * Note: the input can be split into bufferloads at arbitrary boundaries.
-        * Therefore all state must be kept in AH->sqlparse, not in local
-        * variables of this routine.  We assume that AH->sqlparse was filled with
-        * zeroes when created.
-        */
-       for (; qry < eos; qry++)
-       {
                switch (AH->sqlparse.state)
                {
                        case SQL_SCAN:          /* Default state == 0, set in _allocAH */
-                               if (*qry == ';' && AH->sqlparse.braceDepth == 0)
+                               if (ch == ';')
                                {
                                        /*
                                         * We've found the end of a statement. Send it and reset
                                         * the buffer.
                                         */
-                                       appendPQExpBufferChar(AH->sqlBuf, ';');         /* inessential */
-                                       ExecuteSqlCommand(AH, AH->sqlBuf,
+                                       ExecuteSqlCommand(AH, AH->sqlparse.curCmd->data,
                                                                          "could not execute query");
-                                       resetPQExpBuffer(AH->sqlBuf);
-                                       AH->sqlparse.lastChar = '\0';
-
-                                       /*
-                                        * Remove any following newlines - so that embedded COPY
-                                        * commands don't get a starting newline.
-                                        */
-                                       qry++;
-                                       while (qry < eos && *qry == '\n')
-                                               qry++;
-
-                                       /* We've finished one line, so exit */
-                                       return qry;
+                                       resetPQExpBuffer(AH->sqlparse.curCmd);
                                }
-                               else if (*qry == '\'')
+                               else if (ch == '\'')
                                {
-                                       if (AH->sqlparse.lastChar == 'E')
-                                               AH->sqlparse.state = SQL_IN_E_QUOTE;
-                                       else
-                                               AH->sqlparse.state = SQL_IN_SINGLE_QUOTE;
+                                       AH->sqlparse.state = SQL_IN_SINGLE_QUOTE;
                                        AH->sqlparse.backSlash = false;
                                }
-                               else if (*qry == '"')
+                               else if (ch == '"')
                                {
                                        AH->sqlparse.state = SQL_IN_DOUBLE_QUOTE;
                                }
-
-                               /*
-                                * Look for dollar-quotes. We make the assumption that
-                                * $-quotes will not have an ident character just before them
-                                * in pg_dump output.  XXX is this good enough?
-                                */
-                               else if (*qry == '$' && !_isIdentChar(AH->sqlparse.lastChar))
-                               {
-                                       AH->sqlparse.state = SQL_IN_DOLLAR_TAG;
-                                       /* initialize separate buffer with possible tag */
-                                       if (AH->sqlparse.tagBuf == NULL)
-                                               AH->sqlparse.tagBuf = createPQExpBuffer();
-                                       else
-                                               resetPQExpBuffer(AH->sqlparse.tagBuf);
-                                       appendPQExpBufferChar(AH->sqlparse.tagBuf, *qry);
-                               }
-                               else if (*qry == '-' && AH->sqlparse.lastChar == '-')
-                                       AH->sqlparse.state = SQL_IN_SQL_COMMENT;
-                               else if (*qry == '*' && AH->sqlparse.lastChar == '/')
-                                       AH->sqlparse.state = SQL_IN_EXT_COMMENT;
-                               else if (*qry == '(')
-                                       AH->sqlparse.braceDepth++;
-                               else if (*qry == ')')
-                                       AH->sqlparse.braceDepth--;
-                               break;
-
-                       case SQL_IN_SQL_COMMENT:
-                               if (*qry == '\n')
-                                       AH->sqlparse.state = SQL_SCAN;
-                               break;
-
-                       case SQL_IN_EXT_COMMENT:
-
-                               /*
-                                * This isn't fully correct, because we don't account for
-                                * nested slash-stars, but pg_dump never emits such.
-                                */
-                               if (AH->sqlparse.lastChar == '*' && *qry == '/')
-                                       AH->sqlparse.state = SQL_SCAN;
                                break;
 
                        case SQL_IN_SINGLE_QUOTE:
                                /* We needn't handle '' specially */
-                               if (*qry == '\'' && !AH->sqlparse.backSlash)
+                               if (ch == '\'' && !AH->sqlparse.backSlash)
                                        AH->sqlparse.state = SQL_SCAN;
-                               else if (*qry == '\\')
-                                       AH->sqlparse.backSlash = !AH->sqlparse.backSlash;
-                               else
-                                       AH->sqlparse.backSlash = false;
-                               break;
-
-                       case SQL_IN_E_QUOTE:
-
-                               /*
-                                * Eventually we will need to handle '' specially, because
-                                * after E'...''... we should still be in E_QUOTE state.
-                                *
-                                * XXX problem: how do we tell whether the dump was made by a
-                                * version that thinks backslashes aren't special in non-E
-                                * literals??
-                                */
-                               if (*qry == '\'' && !AH->sqlparse.backSlash)
-                                       AH->sqlparse.state = SQL_SCAN;
-                               else if (*qry == '\\')
+                               else if (ch == '\\' && !AH->public.std_strings)
                                        AH->sqlparse.backSlash = !AH->sqlparse.backSlash;
                                else
                                        AH->sqlparse.backSlash = false;
@@ -543,152 +489,135 @@ _sendSQLLine(ArchiveHandle *AH, char *qry, char *eos)
 
                        case SQL_IN_DOUBLE_QUOTE:
                                /* We needn't handle "" specially */
-                               if (*qry == '"')
-                                       AH->sqlparse.state = SQL_SCAN;
-                               break;
-
-                       case SQL_IN_DOLLAR_TAG:
-                               if (*qry == '$')
-                               {
-                                       /* Do not add the closing $ to tagBuf */
-                                       AH->sqlparse.state = SQL_IN_DOLLAR_QUOTE;
-                                       AH->sqlparse.minTagEndPos = AH->sqlBuf->len + AH->sqlparse.tagBuf->len + 1;
-                               }
-                               else if (_isDQChar(*qry, (AH->sqlparse.tagBuf->len == 1)))
-                               {
-                                       /* Valid, so add to tag */
-                                       appendPQExpBufferChar(AH->sqlparse.tagBuf, *qry);
-                               }
-                               else
-                               {
-                                       /*
-                                        * Ooops, we're not really in a dollar-tag.  Valid tag
-                                        * chars do not include the various chars we look for in
-                                        * this state machine, so it's safe to just jump from this
-                                        * state back to SCAN.  We have to back up the qry pointer
-                                        * so that the current character gets rescanned in SCAN
-                                        * state; and then "continue" so that the bottom-of-loop
-                                        * actions aren't done yet.
-                                        */
-                                       AH->sqlparse.state = SQL_SCAN;
-                                       qry--;
-                                       continue;
-                               }
-                               break;
-
-                       case SQL_IN_DOLLAR_QUOTE:
-
-                               /*
-                                * If we are at a $, see whether what precedes it matches
-                                * tagBuf.      (Remember that the trailing $ of the tag was not
-                                * added to tagBuf.)  However, don't compare until we have
-                                * enough data to be a possible match --- this is needed to
-                                * avoid false match on '$a$a$...'
-                                */
-                               if (*qry == '$' &&
-                                       AH->sqlBuf->len >= AH->sqlparse.minTagEndPos &&
-                                       strcmp(AH->sqlparse.tagBuf->data,
-                                                  AH->sqlBuf->data + AH->sqlBuf->len - AH->sqlparse.tagBuf->len) == 0)
+                               if (ch == '"')
                                        AH->sqlparse.state = SQL_SCAN;
                                break;
                }
-
-               appendPQExpBufferChar(AH->sqlBuf, *qry);
-               AH->sqlparse.lastChar = *qry;
        }
-
-       /*
-        * If we get here, we've processed entire bufferload with no complete SQL
-        * stmt
-        */
-       return eos;
 }
 
 
-/* Convenience function to send one or more queries. Monitors result to handle COPY statements */
+/*
+ * Implement ahwrite() for direct-to-DB restore
+ */
 int
-ExecuteSqlCommandBuf(ArchiveHandle *AH, void *qryv, size_t bufLen)
+ExecuteSqlCommandBuf(Archive *AHX, const char *buf, size_t bufLen)
 {
-       char       *qry = (char *) qryv;
-       char       *eos = qry + bufLen;
-
-       /*
-        * fprintf(stderr, "\n\n*****\n Buffer:\n\n%s\n*******************\n\n",
-        * qry);
-        */
+       ArchiveHandle *AH = (ArchiveHandle *) AHX;
 
-       /* Could switch between command and COPY IN mode at each line */
-       while (qry < eos)
+       if (AH->outputKind == OUTPUT_COPYDATA)
        {
                /*
-                * If libpq is in CopyIn mode *or* if the archive structure shows we
-                * are sending COPY data, treat the data as COPY data.  The pgCopyIn
-                * check is only needed for backwards compatibility with ancient
-                * archive files that might just issue a COPY command without marking
-                * it properly.  Note that in an archive entry that has a copyStmt,
-                * all data up to the end of the entry will go to _sendCopyLine, and
-                * therefore will be dropped if libpq has failed to enter COPY mode.
-                * Also, if a "\." data terminator is found, anything remaining in the
-                * archive entry will be dropped.
+                * COPY data.
+                *
+                * We drop the data on the floor if libpq has failed to enter COPY
+                * mode; this allows us to behave reasonably when trying to continue
+                * after an error in a COPY command.
                 */
-               if (AH->pgCopyIn || AH->writingCopyData)
-                       qry = _sendCopyLine(AH, qry, eos);
+               if (AH->pgCopyIn &&
+                       PQputCopyData(AH->connection, buf, bufLen) <= 0)
+                       exit_horribly(modulename, "error returned by PQputCopyData: %s",
+                                                 PQerrorMessage(AH->connection));
+       }
+       else if (AH->outputKind == OUTPUT_OTHERDATA)
+       {
+               /*
+                * Table data expressed as INSERT commands; or, in old dump files,
+                * BLOB COMMENTS data (which is expressed as COMMENT ON commands).
+                */
+               ExecuteSimpleCommands(AH, buf, bufLen);
+       }
+       else
+       {
+               /*
+                * General SQL commands; we assume that commands will not be split
+                * across calls.
+                *
+                * In most cases the data passed to us will be a null-terminated
+                * string, but if it's not, we have to add a trailing null.
+                */
+               if (buf[bufLen] == '\0')
+                       ExecuteSqlCommand(AH, buf, "could not execute query");
                else
-                       qry = _sendSQLLine(AH, qry, eos);
+               {
+                       char       *str = (char *) pg_malloc(bufLen + 1);
+
+                       memcpy(str, buf, bufLen);
+                       str[bufLen] = '\0';
+                       ExecuteSqlCommand(AH, str, "could not execute query");
+                       free(str);
+               }
        }
 
-       return 1;
+       return bufLen;
 }
 
+/*
+ * Terminate a COPY operation during direct-to-DB restore
+ */
 void
-StartTransaction(ArchiveHandle *AH)
+EndDBCopyMode(Archive *AHX, const char *tocEntryTag)
 {
-       PQExpBuffer qry = createPQExpBuffer();
+       ArchiveHandle *AH = (ArchiveHandle *) AHX;
 
-       appendPQExpBuffer(qry, "BEGIN");
+       if (AH->pgCopyIn)
+       {
+               PGresult   *res;
 
-       ExecuteSqlCommand(AH, qry, "could not start database transaction");
+               if (PQputCopyEnd(AH->connection, NULL) <= 0)
+                       exit_horribly(modulename, "error returned by PQputCopyEnd: %s",
+                                                 PQerrorMessage(AH->connection));
 
-       destroyPQExpBuffer(qry);
+               /* Check command status and return to normal libpq state */
+               res = PQgetResult(AH->connection);
+               if (PQresultStatus(res) != PGRES_COMMAND_OK)
+                       warn_or_exit_horribly(AH, modulename, "COPY failed for table \"%s\": %s",
+                                                                 tocEntryTag, PQerrorMessage(AH->connection));
+               PQclear(res);
+
+               AH->pgCopyIn = false;
+       }
 }
 
 void
-CommitTransaction(ArchiveHandle *AH)
+StartTransaction(Archive *AHX)
 {
-       PQExpBuffer qry = createPQExpBuffer();
-
-       appendPQExpBuffer(qry, "COMMIT");
-
-       ExecuteSqlCommand(AH, qry, "could not commit database transaction");
+       ArchiveHandle *AH = (ArchiveHandle *) AHX;
 
-       destroyPQExpBuffer(qry);
+       ExecuteSqlCommand(AH, "BEGIN", "could not start database transaction");
 }
 
-static bool
-_isIdentChar(unsigned char c)
+void
+CommitTransaction(Archive *AHX)
 {
-       if ((c >= 'a' && c <= 'z')
-               || (c >= 'A' && c <= 'Z')
-               || (c >= '0' && c <= '9')
-               || (c == '_')
-               || (c == '$')
-               || (c >= (unsigned char) '\200')                /* no need to check <= \377 */
-               )
-               return true;
-       else
-               return false;
+       ArchiveHandle *AH = (ArchiveHandle *) AHX;
+
+       ExecuteSqlCommand(AH, "COMMIT", "could not commit database transaction");
 }
 
-static bool
-_isDQChar(unsigned char c, bool atStart)
+void
+DropBlobIfExists(ArchiveHandle *AH, Oid oid)
 {
-       if ((c >= 'a' && c <= 'z')
-               || (c >= 'A' && c <= 'Z')
-               || (c == '_')
-               || (!atStart && c >= '0' && c <= '9')
-               || (c >= (unsigned char) '\200')                /* no need to check <= \377 */
-               )
-               return true;
+       /*
+        * If we are not restoring to a direct database connection, we have to
+        * guess about how to detect whether the blob exists.  Assume new-style.
+        */
+       if (AH->connection == NULL ||
+               PQserverVersion(AH->connection) >= 90000)
+       {
+               ahprintf(AH,
+                                "SELECT pg_catalog.lo_unlink(oid) "
+                                "FROM pg_catalog.pg_largeobject_metadata "
+                                "WHERE oid = '%u';\n",
+                                oid);
+       }
        else
-               return false;
+       {
+               /* Restoring to pre-9.0 server, so do it the old way */
+               ahprintf(AH,
+                                "SELECT CASE WHEN EXISTS("
+                                "SELECT 1 FROM pg_catalog.pg_largeobject WHERE loid = '%u'"
+                                ") THEN pg_catalog.lo_unlink('%u') END;\n",
+                                oid, oid);
+       }
 }