Fix another small oversight in command_no_begin patch.

Need a "return false" to prevent tests from continuing after we've moved
the "query" pointer.  As it stood, it'd accept "DROP DISCARD ALL" as a
match.

diff --git a/src/bin/psql/common.c b/src/bin/psql/common.c
index 64577084a2f6f0f6c0670d5578d918edbc03f71c..9670dc62b8856b94f58d36a581e923f212325b7a 100644 (file)
--- a/src/bin/psql/common.c
+++ b/src/bin/psql/common.c
@@ -1350,6 +1350,7 @@ command_no_begin(const char *query)
            return true;
        if (wordlen == 10 && pg_strncasecmp(query, "tablespace", 10) == 0)
            return true;
+       return false;
    }
 
    /* DISCARD ALL isn't allowed in xacts, but other variants are allowed. */
@@ -1365,6 +1366,7 @@ command_no_begin(const char *query)
 
        if (wordlen == 3 && pg_strncasecmp(query, "all", 3) == 0)
            return true;
+       return false;
    }
 
    return false;