From cc811a8ae6c760955ac2dca51d8359d003c8e4f2 Mon Sep 17 00:00:00 2001 From: Bernd Edlinger Date: Tue, 30 Sep 2014 16:08:53 +0000 Subject: [PATCH] re PR preprocessor/58893 (:0:0: internal compiler error: Segmentation fault) 2014-09-30 Bernd Edlinger PR preprocessor/58893 * errors.c (cpp_diagnostic): Fix possible out of bounds access. * files.c (_cpp_stack_include): Initialize src_loc for IT_CMDLINE. testsuite: 2014-09-30 Bernd Edlinger PR preprocessor/58893 * gcc.dg/pr58893.c: New test case. * gcc.dg/pr58893-0.h: New include. From-SVN: r215730 --- gcc/testsuite/ChangeLog | 6 ++++++ gcc/testsuite/gcc.dg/pr58893-0.h | 1 + gcc/testsuite/gcc.dg/pr58893.c | 5 +++++ libcpp/ChangeLog | 6 ++++++ libcpp/errors.c | 5 +---- libcpp/files.c | 12 ++++++++++++ 6 files changed, 31 insertions(+), 4 deletions(-) create mode 100644 gcc/testsuite/gcc.dg/pr58893-0.h create mode 100644 gcc/testsuite/gcc.dg/pr58893.c diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 858df23c8ee..6ffbbee2720 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2014-09-30 Bernd Edlinger + + PR preprocessor/58893 + * gcc.dg/pr58893.c: New test case. + * gcc.dg/pr58893-0.h: New include. + 2014-09-30 Ilya Tocar PR middle-end/62120 diff --git a/gcc/testsuite/gcc.dg/pr58893-0.h b/gcc/testsuite/gcc.dg/pr58893-0.h new file mode 100644 index 00000000000..957bcdc1173 --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr58893-0.h @@ -0,0 +1 @@ +#pragma GCC visibility push(hidden) diff --git a/gcc/testsuite/gcc.dg/pr58893.c b/gcc/testsuite/gcc.dg/pr58893.c new file mode 100644 index 00000000000..c9f8b6b1e8d --- /dev/null +++ b/gcc/testsuite/gcc.dg/pr58893.c @@ -0,0 +1,5 @@ +/* PR preprocessor/58893 */ +/* { dg-do compile } */ +/* { dg-options "-include pr58893-0.h -include pr58893-1.h -I${srcdir}/gcc.dg" } */ +/* { dg-error "pr58893-1.h: No such file or directory" "" { target *-*-* } 0 } */ +/* { dg-prune-output "compilation terminated" } */ diff --git a/libcpp/ChangeLog b/libcpp/ChangeLog index 463bb60e761..92999040a02 100644 --- a/libcpp/ChangeLog +++ b/libcpp/ChangeLog @@ -1,3 +1,9 @@ +2014-09-30 Bernd Edlinger + + PR preprocessor/58893 + * errors.c (cpp_diagnostic): Fix possible out of bounds access. + * files.c (_cpp_stack_include): Initialize src_loc for IT_CMDLINE. + 2014-09-24 Marek Polacek PR c/61405 diff --git a/libcpp/errors.c b/libcpp/errors.c index d1ca7a12ff4..bc857f0b81d 100644 --- a/libcpp/errors.c +++ b/libcpp/errors.c @@ -48,10 +48,7 @@ cpp_diagnostic (cpp_reader * pfile, int level, int reason, current run -- that is invalid. */ else if (pfile->cur_token == pfile->cur_run->base) { - if (pfile->cur_run->prev != NULL) - src_loc = pfile->cur_run->prev->limit->src_loc; - else - src_loc = 0; + src_loc = 0; } else { diff --git a/libcpp/files.c b/libcpp/files.c index a442783f1d1..00302fd774f 100644 --- a/libcpp/files.c +++ b/libcpp/files.c @@ -991,6 +991,18 @@ _cpp_stack_include (cpp_reader *pfile, const char *fname, int angle_brackets, _cpp_file *file; bool stacked; + /* For -include command-line flags we have type == IT_CMDLINE. + When the first -include file is processed we have the case, where + pfile->cur_token == pfile->cur_run->base, we are directly called up + by the front end. However in the case of the second -include file, + we are called from _cpp_lex_token -> _cpp_get_fresh_line -> + cpp_push_include, with pfile->cur_token != pfile->cur_run->base, + and pfile->cur_token[-1].src_loc not (yet) initialized. + However, when the include file cannot be found, we need src_loc to + be initialized to some safe value: 0 means UNKNOWN_LOCATION. */ + if (type == IT_CMDLINE && pfile->cur_token != pfile->cur_run->base) + pfile->cur_token[-1].src_loc = 0; + dir = search_path_head (pfile, fname, angle_brackets, type); if (!dir) return false; -- 2.30.2