diff options
| author | Harald Anlauf <anlauf@gmx.de> | 2022-09-20 22:23:43 +0200 |
|---|---|---|
| committer | Harald Anlauf <anlauf@gmx.de> | 2022-09-29 20:35:45 +0200 |
| commit | 2e81790f45c9b7ced4d8c70c3122404854aa5be5 (patch) | |
| tree | ace1b84b740f22d3cf91cfa3b81700b9ea84d2a8 /gcc | |
| parent | f1454a75c9a165c0ba30833f23f799044fd7a7fe (diff) | |
| download | gcc-2e81790f45c9b7ced4d8c70c3122404854aa5be5.zip gcc-2e81790f45c9b7ced4d8c70c3122404854aa5be5.tar.gz gcc-2e81790f45c9b7ced4d8c70c3122404854aa5be5.tar.bz2 | |
Fortran: NULL pointer dereference in invalid simplification [PR106985]
gcc/fortran/ChangeLog:
PR fortran/106985
* expr.cc (gfc_simplify_expr): Avoid NULL pointer dereference.
gcc/testsuite/ChangeLog:
PR fortran/106985
* gfortran.dg/pr106985.f90: New test.
(cherry picked from commit 8dbb15bc2d019488240c1e69d93121b0347ac092)
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/fortran/expr.cc | 3 | ||||
| -rw-r--r-- | gcc/testsuite/gfortran.dg/pr106985.f90 | 8 |
2 files changed, 10 insertions, 1 deletions
diff --git a/gcc/fortran/expr.cc b/gcc/fortran/expr.cc index be94c18..290ddf3 100644 --- a/gcc/fortran/expr.cc +++ b/gcc/fortran/expr.cc @@ -2287,7 +2287,8 @@ gfc_simplify_expr (gfc_expr *p, int type) initialization expression, or we want a subsection. */ if (p->symtree->n.sym->attr.flavor == FL_PARAMETER && (gfc_init_expr_flag || p->ref - || p->symtree->n.sym->value->expr_type != EXPR_ARRAY)) + || (p->symtree->n.sym->value + && p->symtree->n.sym->value->expr_type != EXPR_ARRAY))) { if (!simplify_parameter_variable (p, type)) return false; diff --git a/gcc/testsuite/gfortran.dg/pr106985.f90 b/gcc/testsuite/gfortran.dg/pr106985.f90 new file mode 100644 index 0000000..f4ed925 --- /dev/null +++ b/gcc/testsuite/gfortran.dg/pr106985.f90 @@ -0,0 +1,8 @@ +! { dg-do compile } +! PR fortran/106985 - ICE in gfc_simplify_expr +! Contributed by G.Steinmetz + +program p + integer, parameter :: a(2) = 1 + integer, parameter :: b = a(2) + b ! { dg-error "before its definition is complete" } +end |