diff --git a/libgfortran/ChangeLog b/libgfortran/ChangeLog index 1cbdaf393fb33eb69cb6e1ab71877ac09fa0315c..476a41921f396f185bf430208b9be579db1f9fc2 100644 --- a/libgfortran/ChangeLog +++ b/libgfortran/ChangeLog @@ -1,3 +1,13 @@ +2020-05-14 Thomas Koenig <tkoenig@gcc.gnu.org> + + PR libfortran/95119 + * io/close.c (close_status): Add CLOSE_INVALID. + (st_close): Return early on invalid STATUS parameter. + +2020-05-14 H.J. Lu <hongjiu.lu@intel.com> + + * configure: Regenerated. + 2020-05-14 H.J. Lu <hongjiu.lu@intel.com> * configure: Regenerated. diff --git a/libgfortran/io/close.c b/libgfortran/io/close.c index 8aaa00393e70197b92ec24ea712bc05a8ebbe658..17e064ba4e8d1ac05bac1963f03abf788c5dca6c 100644 --- a/libgfortran/io/close.c +++ b/libgfortran/io/close.c @@ -31,7 +31,7 @@ see the files COPYING3 and COPYING.RUNTIME respectively. If not, see #endif typedef enum -{ CLOSE_DELETE, CLOSE_KEEP, CLOSE_UNSPECIFIED } +{ CLOSE_INVALID = - 1, CLOSE_DELETE, CLOSE_KEEP, CLOSE_UNSPECIFIED } close_status; static const st_option status_opt[] = { @@ -61,6 +61,12 @@ st_close (st_parameter_close *clp) find_option (&clp->common, clp->status, clp->status_len, status_opt, "Bad STATUS parameter in CLOSE statement"); + if (status == CLOSE_INVALID) + { + library_end (); + return; + } + u = find_unit (clp->common.unit); if (ASYNC_IO && u && u->au) diff --git a/libgomp/ChangeLog b/libgomp/ChangeLog index 8a66266d7c6fd50e661777093ab6cf6bfe7ec4fc..3339ffb3b2221df0f7106129eeca079ae4871154 100644 --- a/libgomp/ChangeLog +++ b/libgomp/ChangeLog @@ -1,3 +1,16 @@ +2020-05-14 Thomas Koenig <tkoenig@gcc.gnu.org> + + PR libfortran/95119 + * testsuite/libgomp.fortran/close_errors_1.f90: New test. + +2020-05-14 H.J. Lu <hongjiu.lu@intel.com> + + * configure: Regenerated. + +2020-05-14 Jakub Jelinek <jakub@redhat.com> + + * testsuite/libgomp.c-c++-common/target-40.c: New test. + 2020-05-14 H.J. Lu <hongjiu.lu@intel.com> * configure: Regenerated. diff --git a/libgomp/testsuite/libgomp.fortran/close_errors_1.f90 b/libgomp/testsuite/libgomp.fortran/close_errors_1.f90 new file mode 100644 index 0000000000000000000000000000000000000000..6edb7da4116f7e47db6ea45af872d2846c95b17a --- /dev/null +++ b/libgomp/testsuite/libgomp.fortran/close_errors_1.f90 @@ -0,0 +1,19 @@ +! { dg-do run } +! PR 95115 - this used to hang with -pthread. Original test case by +! Bill Long. + +program test + character(len=16) my_status + character(len=1000) :: iomsg + open (unit=10, file='test.dat') + print *,42 + write (10, *) 'weird' + rewind (10) + read (10, *) my_status + close (10) + open (unit=10, file='test.dat') + close (unit=10, status=my_status, iostat=ios, iomsg=iomsg) + if (ios == 0) stop 1 + if (iomsg /= "Bad STATUS parameter in CLOSE statement") stop 2 + close (10, status='delete') +end program test