]> granicus.if.org Git - postgresql/commitdiff
Fix ecpg bugs caused by missing semicolons in the backend grammar.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 24 Feb 2019 17:51:51 +0000 (12:51 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 24 Feb 2019 17:51:51 +0000 (12:51 -0500)
The Bison documentation clearly states that a semicolon is required
after every grammar rule, and our scripts that generate ecpg's
grammar from the backend's implicitly assumed this is true.  But it
turns out that only ancient versions of Bison actually enforce that.
There have been a couple of rules without trailing semicolons in
gram.y for some time, and as a consequence, ecpg's grammar was faulty
and produced wrong output for the affected statements.

To fix, add the missing semis, and add some cross-checks to ecpg's
scripts so that they'll bleat if we mess this up again.

The cases that were broken were:
* "SET variable = DEFAULT" (but not "SET variable TO DEFAULT"),
  as well as allied syntaxes such as ALTER SYSTEM SET ... DEFAULT.
  These produced syntactically invalid output that the server
  would reject.
* Multiple type names in DROP TYPE/DOMAIN commands.  Only the
  first type name would be listed in the emitted command.

Per report from Daisuke Higuchi.  Back-patch to all supported versions.

Discussion: https://postgr.es/m/1803D792815FC24D871C00D17AE95905DB51CE@g01jpexmbkw24

src/backend/parser/gram.y
src/interfaces/ecpg/preproc/check_rules.pl
src/interfaces/ecpg/preproc/parse.pl

index 9f08926e772ff1e3967da78b95af04033ccf212c..96910cf6edd81c52e1ae63594b05023c5c8c3836 100644 (file)
@@ -1424,6 +1424,7 @@ generic_set:
                                        n->name = $1;
                                        $$ = n;
                                }
+               ;
 
 set_rest_more: /* Generic SET syntaxes: */
                        generic_set                                             {$$ = $1;}
@@ -5630,6 +5631,7 @@ attrs:            '.' attr_name
 type_name_list:
                        Typename                                                                { $$ = list_make1(list_make1($1)); }
                        | type_name_list ',' Typename                   { $$ = lappend($1, list_make1($3)); }
+               ;
 
 /*****************************************************************************
  *
index d5377733f229150f8cf91397825ca1464643be61..147d2e877a652ba98113acb2636ad54888479ae5 100644 (file)
@@ -1,7 +1,7 @@
 #!/usr/bin/perl
 # src/interfaces/ecpg/preproc/check_rules.pl
-# test parser generater for ecpg
-# call with backend parser as stdin
+# test parser generator for ecpg
+# call with backend grammar as stdin
 #
 # Copyright (c) 2009-2015, PostgreSQL Global Development Group
 #
@@ -47,6 +47,7 @@ my %replace_line = (
 
 my $block        = '';
 my $yaccmode     = 0;
+my $in_rule      = 0;
 my $brace_indent = 0;
 my (@arr, %found);
 my $comment     = 0;
@@ -131,10 +132,14 @@ while (<GRAM>)
                        $found{$block} = 1;
                        $cc++;
                        $block = '';
+                       $in_rule = 0 if $arr[$fieldIndexer] eq ';';
                }
                elsif (($arr[$fieldIndexer] =~ '[A-Za-z0-9]+:')
                        || $arr[ $fieldIndexer + 1 ] eq ':')
                {
+                       die "unterminated rule at grammar line $.\n"
+                         if $in_rule;
+                       $in_rule     = 1;
                        $non_term_id = $arr[$fieldIndexer];
                        $non_term_id =~ tr/://d;
                }
@@ -145,6 +150,9 @@ while (<GRAM>)
        }
 }
 
+die "unterminated rule at end of grammar\n"
+  if $in_rule;
+
 close GRAM;
 if ($verbose)
 {
index cfce4b192805d6b648fbe62724a959e3be0825a8..ac1c1e3d5bbac1fb3db122a925364c7ea39a5138 100644 (file)
@@ -22,6 +22,7 @@ $path = "." unless $path;
 my $copymode              = 0;
 my $brace_indent          = 0;
 my $yaccmode              = 0;
+my $in_rule               = 0;
 my $header_included       = 0;
 my $feature_not_supported = 0;
 my $tokenmode             = 0;
@@ -288,6 +289,7 @@ sub main
                                @fields  = ();
                                $infield = 0;
                                $line    = '';
+                               $in_rule = 0;
                                next;
                        }
 
@@ -365,6 +367,9 @@ sub main
                                $line    = '';
                                @fields  = ();
                                $infield = 1;
+                               die "unterminated rule at grammar line $.\n"
+                                 if $in_rule;
+                               $in_rule = 1;
                                next;
                        }
                        elsif ($copymode)
@@ -415,6 +420,9 @@ sub main
                        }
                }
        }
+       die "unterminated rule at end of grammar\n"
+         if $in_rule;
+       return;
 }