]> granicus.if.org Git - postgresql/commitdiff
Defend against null input in analyze_requires_snapshot(), per report
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 30 Jan 2009 16:59:10 +0000 (16:59 +0000)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 30 Jan 2009 16:59:10 +0000 (16:59 +0000)
from Rushabh Lathia.

Back-patch of patch of 2009-01-08.  This is necessary in 8.3, as reported
by Bjorn Munch.  It's not currently necessary in 8.2, AFAICS, but seems
best to include it there too.

src/backend/parser/analyze.c

index 4a539a1e1e897eeaac3765da9ed21366f5845b1d..4c28de6aa255c98a244dce54c6904551bfcc6023 100644 (file)
@@ -17,7 +17,7 @@
  * Portions Copyright (c) 1996-2008, PostgreSQL Global Development Group
  * Portions Copyright (c) 1994, Regents of the University of California
  *
- *     $PostgreSQL: pgsql/src/backend/parser/analyze.c,v 1.371.2.1 2008/12/13 02:00:29 tgl Exp $
+ *     $PostgreSQL: pgsql/src/backend/parser/analyze.c,v 1.371.2.2 2009/01/30 16:59:10 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -232,13 +232,17 @@ transformStmt(ParseState *pstate, Node *parseTree)
  *             Returns true if a snapshot must be set before doing parse analysis
  *             on the given raw parse tree.
  *
- * Classification here should match transformStmt().
+ * Classification here should match transformStmt(); but we also have to
+ * allow a NULL input (for Parse/Bind of an empty query string).
  */
 bool
 analyze_requires_snapshot(Node *parseTree)
 {
        bool            result;
 
+       if (parseTree == NULL)
+               return false;
+
        switch (nodeTag(parseTree))
        {
                        /*