re PR fortran/70870 (Segmentation violation in gfc_assign_data_value)

2018-05-10  Steven G. Kargl  <kargl@gcc.gnu.org>

	PR fortran/70870
	* data.c (gfc_assign_data_value): Check that a data object does
	not also have default initialization.

2018-05-10  Steven G. Kargl  <kargl@gcc.gnu.org>

	PR fortran/70870
	* gfortran.dg/pr70870_1.f90: New test.

From-SVN: r260138

diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog
index c41804ea4c7504252d4f7bfe58ad1a83a9a136c3..aedcb2d96084e497c6f4ab92f375e9b9124f5bf1 100644 (file)
--- a/gcc/fortran/ChangeLog
+++ b/gcc/fortran/ChangeLog
@@ -1,3 +1,9 @@
+2018-05-10  Steven G. Kargl  <kargl@gcc.gnu.org>
+
+       PR fortran/70870
+       * data.c (gfc_assign_data_value): Check that a data object does
+       not also have default initialization.
+
 2018-05-10  Marek Polacek  <polacek@redhat.com>
 
        PR fortran/85735

diff --git a/gcc/fortran/data.c b/gcc/fortran/data.c
index fdd7052e6a4a2edd7c1b041545b65ec04e638d11..780a5ad9564ad69effa338ebeaabbef8f02ee34e 100644 (file)
--- a/gcc/fortran/data.c
+++ b/gcc/fortran/data.c
@@ -491,6 +491,15 @@ gfc_assign_data_value (gfc_expr *lvalue, gfc_expr *rvalue, mpz_t index,
     }
   else
     {
+      if (lvalue->ts.type == BT_DERIVED
+         && gfc_has_default_initializer (lvalue->ts.u.derived))
+       {
+         gfc_error ("Nonpointer object %qs with default initialization "
+                    "shall not appear in a DATA statement at %L", 
+                    symbol->name, &lvalue->where);
+         return false;
+       }
+
       /* Overwriting an existing initializer is non-standard but usually only
         provokes a warning from other compilers.  */
       if (init != NULL)

diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog
index 6d906667a217b01f35410e24c04568ae11f9b624..64014c4858544f1eed58d090f5e42f74477909ea 100644 (file)
--- a/gcc/testsuite/ChangeLog
+++ b/gcc/testsuite/ChangeLog
@@ -1,3 +1,8 @@
+2018-05-10  Steven G. Kargl  <kargl@gcc.gnu.org>
+
+       PR fortran/70870
+       * gfortran.dg/pr70870_1.f90: New test.
+
 2018-05-10  Uros Bizjak  <ubizjak@gmail.com>
 
        * gcc.target/i386/xgetsetbv.c: Check also variable arguments.

diff --git a/gcc/testsuite/gfortran.dg/pr70870_1.f90 b/gcc/testsuite/gfortran.dg/pr70870_1.f90
new file mode 100644 (file)
index 0000000..0f9584a
--- /dev/null
+++ b/gcc/testsuite/gfortran.dg/pr70870_1.f90
@@ -0,0 +1,9 @@
+! { dg-do compile }
+! PR fortran/70870
+! Contributed by Vittorio Zecca <zeccav at gmail dot com >
+      type t
+       integer :: g=0   ! default initialization
+      end type
+      type(t) :: v2
+      data v2/t(2)/     ! { dg-error "default initialization shall not" }
+      end