diff options
| author | Paul Thomas <pault@gcc.gnu.org> | 2008-02-05 11:16:33 +0000 |
|---|---|---|
| committer | Paul Thomas <pault@gcc.gnu.org> | 2008-02-05 11:16:33 +0000 |
| commit | 3969f765e38d4c32b58f87b8256f0690ce48ba34 (patch) | |
| tree | 76d03a98e8e40b4bc08cfd374a29a641ed449b26 /gcc/fortran/data.c | |
| parent | 3467b230ffd4f43407c6643d8990f545e06aa32f (diff) | |
| download | gcc-3969f765e38d4c32b58f87b8256f0690ce48ba34.zip gcc-3969f765e38d4c32b58f87b8256f0690ce48ba34.tar.gz gcc-3969f765e38d4c32b58f87b8256f0690ce48ba34.tar.bz2 | |
re PR fortran/32315 (DATA with implied-do: Bounds checks missing [regression vs. g77])
2008-02-05 Paul Thomas <pault@gcc.gnu.org>
PR fortran/32315
* data.c (gfc_assign_data_value): Add bounds check for array
references.
2008-02-05 Paul Thomas <pault@gcc.gnu.org>
PR fortran/32315
* gfortran.dg/data_bounds_1.f90: New test.
From-SVN: r132113
Diffstat (limited to 'gcc/fortran/data.c')
| -rw-r--r-- | gcc/fortran/data.c | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/gcc/fortran/data.c b/gcc/fortran/data.c index 41dc85f..10fcd48 100644 --- a/gcc/fortran/data.c +++ b/gcc/fortran/data.c @@ -311,6 +311,27 @@ gfc_assign_data_value (gfc_expr *lvalue, gfc_expr *rvalue, mpz_t index) else mpz_set (offset, index); + /* Check the bounds. */ + if (mpz_cmp_si (offset, 0) < 0) + { + gfc_error ("Data element below array lower bound at %L", + &lvalue->where); + return FAILURE; + } + else + { + mpz_t size; + if (spec_size (ref->u.ar.as, &size) == SUCCESS + && mpz_cmp (offset, size) >= 0) + { + mpz_clear (size); + gfc_error ("Data element above array upper bound at %L", + &lvalue->where); + return FAILURE; + } + mpz_clear (size); + } + /* Splay tree containing offset and gfc_constructor. */ spt = expr->con_by_offset; |