diff options
| author | David Pagan <dave.pagan@oracle.com> | 2018-05-02 17:22:26 +0000 |
|---|---|---|
| committer | Jeff Law <law@gcc.gnu.org> | 2018-05-02 11:22:26 -0600 |
| commit | f7584c811623675be258da5195d8e8daeb562975 (patch) | |
| tree | efbf7e13707325d8fd0fe006a25ea5ece053f911 /gcc/c/c-parser.c | |
| parent | 621af561737ae17f6b4cffad9b79b1189503f011 (diff) | |
| download | gcc-f7584c811623675be258da5195d8e8daeb562975.zip gcc-f7584c811623675be258da5195d8e8daeb562975.tar.gz gcc-f7584c811623675be258da5195d8e8daeb562975.tar.bz2 | |
re PR c/30552 (gcc crashes when compiling examples with GNU statement expressions in VLAs (also involved: nested functions declared K&R-style))
PR c/30552
* c-decl.c (old_style_parameter_scope): New function.
* c-parser.c (c_parser_postfix_expression): Check for statement
expressions in old-style function parameter list declarations.
* c-parser.h (old_style_parameter_scope): New extern declaration.
PR c/30552
* gcc.dg/noncompile/pr30552-1.c: New test.
* gcc.dg/noncompile/pr30552-2.c: New test.
* gcc.dg/noncompile/pr30552-3.c: New test.
* gcc.dg/noncompile/pr30552-4.c: New test.
From-SVN: r259849
Diffstat (limited to 'gcc/c/c-parser.c')
| -rw-r--r-- | gcc/c/c-parser.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/gcc/c/c-parser.c b/gcc/c/c-parser.c index 4772086..6b41a61 100644 --- a/gcc/c/c-parser.c +++ b/gcc/c/c-parser.c @@ -7930,7 +7930,10 @@ c_parser_postfix_expression (c_parser *parser) c_parser_consume_token (parser); brace_loc = c_parser_peek_token (parser)->location; c_parser_consume_token (parser); - if (!building_stmt_list_p ()) + /* If we've not yet started the current function's statement list, + or we're in the parameter scope of an old-style function + declaration, statement expressions are not allowed. */ + if (!building_stmt_list_p () || old_style_parameter_scope ()) { error_at (loc, "braced-group within expression allowed " "only inside a function"); |