diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2010-09-28 14:47:25 -0400 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2010-09-28 14:51:04 -0400 |
| commit | 3fe9f0e5c734c79b604abfdfd3766e85c1e4b2b7 (patch) | |
| tree | 3993e33a5f9fa94aa77a013ad1c5d9b9bc204e1c /src | |
| parent | 798cbc015b6512b54a41bf3d40fd235322125292 (diff) | |
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.
Diffstat (limited to 'src')
| -rw-r--r-- | src/bin/psql/common.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/bin/psql/common.c b/src/bin/psql/common.c index 72115f8ae65..877d6f7cdbc 100644 --- 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; |