mirrors/postgres
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix another small oversight in command_no_begin patch.

Browse Source 
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.
This commit is contained in:
Tom Lane 2010-09-28 14:47:25 -04:00
parent ae9acb6779
commit 112c3fc6db
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/bin/psql/common.c
View File

@ -1386,6 +1386,7 @@ command_no_begin(const char *query)
return true; return true;
if (wordlen == 10 && pg_strncasecmp(query, "tablespace", 10) == 0) if (wordlen == 10 && pg_strncasecmp(query, "tablespace", 10) == 0)
return true; return true;
return false;
} }
/* DISCARD ALL isn't allowed in xacts, but other variants are allowed. */ /* 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) if (wordlen == 3 && pg_strncasecmp(query, "all", 3) == 0)
return true; return true;
return false;
} }
return false; return false;