diff options
| author | Nathan Sidwell <nathan@acm.org> | 2020-10-16 09:22:22 -0700 |
|---|---|---|
| committer | Nathan Sidwell <nathan@acm.org> | 2020-10-16 09:25:47 -0700 |
| commit | a16da48bf19bb139e5461e5b5b7f072d5369b054 (patch) | |
| tree | 092a2a23d7414f69e519bb1440c8560a3d5b8f52 /gcc/cp/parser.c | |
| parent | a6590686f72e2a99d290b5e96e54e550446009de (diff) | |
| download | gcc-a16da48bf19bb139e5461e5b5b7f072d5369b054.zip gcc-a16da48bf19bb139e5461e5b5b7f072d5369b054.tar.gz gcc-a16da48bf19bb139e5461e5b5b7f072d5369b054.tar.bz2 | |
c++: Fix null deref at EOF [PR96258]
cp_parser_declaration peeks at 1 or 2 tokens, when I changed it not to
peek past EOF, I set the second token to NULL. But there are paths
through the function that just look at the second token. Fixed by
setting that token to EOF rather than NULL in this case.
PR c++/96258
gcc/cp/
* parser.c (cp_parser_declaration): Make token2 point to EOF if
token1 was EOF.
gcc/testsuite/
* g++.dg/parse/pr96258.C: New.
Diffstat (limited to 'gcc/cp/parser.c')
| -rw-r--r-- | gcc/cp/parser.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/gcc/cp/parser.c b/gcc/cp/parser.c index 592ce95..7ec7d42 100644 --- a/gcc/cp/parser.c +++ b/gcc/cp/parser.c @@ -13497,10 +13497,8 @@ cp_parser_declaration (cp_parser* parser) /* Try to figure out what kind of declaration is present. */ cp_token *token1 = cp_lexer_peek_token (parser->lexer); - cp_token *token2 = NULL; - - if (token1->type != CPP_EOF) - token2 = cp_lexer_peek_nth_token (parser->lexer, 2); + cp_token *token2 = (token1->type == CPP_EOF + ? token1 : cp_lexer_peek_nth_token (parser->lexer, 2)); /* Get the high-water mark for the DECLARATOR_OBSTACK. */ void *p = obstack_alloc (&declarator_obstack, 0); |