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 72115f8ae65a57688c9d8f3a9e0c5dc5268d2780..877d6f7cdbc8a1f653d3939f291e05657607683a 100644 (file)
--- a/src/bin/psql/common.c
+++ b/src/bin/psql/common.c
@@ -1386,6 +1386,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. */
@@ -1401,6 +1402,7 @@ command_no_begin(const char *query)
 
                if (wordlen == 3 && pg_strncasecmp(query, "all", 3) == 0)
                        return true;
+               return false;
        }
 
        return false;