From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2071) id 109283858D28; Sun, 5 Mar 2023 19:35:42 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 109283858D28 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1678044942; bh=FTN3xltRGxaxffXqNMZRHmKh/1oMMH9NbiPQ6jIlPgY=; h=From:To:Subject:Date:From; b=vX8WfrCllysdVfS/jYN4D+waT6O6BXjaImDCqDiMwz00cESBnztQhmg6HMWsRtlcr fwQXq0QFVZ7zQMsmE7LlUaVWmtnROQIW0Not5of/O62uN5UCsvG5DJYg21OpszuRBs /gag5M3DaTMLu1tWR1Lqi+oIPDeRLqjVUERhYNBM= MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="utf-8" From: Harald Anlauf To: gcc-cvs@gcc.gnu.org Subject: [gcc r11-10559] Fortran: reject invalid CHARACTER length of derived type components [PR96024] X-Act-Checkin: gcc X-Git-Author: Harald Anlauf X-Git-Refname: refs/heads/releases/gcc-11 X-Git-Oldrev: 8412f78621f6dcb7cee23dda49afb28299d8083b X-Git-Newrev: 7db643a17e8c862b4c758fd69d62a45c6de43d38 Message-Id: <20230305193542.109283858D28@sourceware.org> Date: Sun, 5 Mar 2023 19:35:42 +0000 (GMT) List-Id: https://gcc.gnu.org/g:7db643a17e8c862b4c758fd69d62a45c6de43d38 commit r11-10559-g7db643a17e8c862b4c758fd69d62a45c6de43d38 Author: Harald Anlauf Date: Tue Feb 21 22:06:33 2023 +0100 Fortran: reject invalid CHARACTER length of derived type components [PR96024] gcc/fortran/ChangeLog: PR fortran/96024 * resolve.c (resolve_component): The type of a CHARACTER length expression must be INTEGER. gcc/testsuite/ChangeLog: PR fortran/96024 * gfortran.dg/pr96024.f90: New test. (cherry picked from commit 31303c9b5bab200754cdb7ef8cd91ae4918f3018) Diff: --- gcc/fortran/resolve.c | 13 +++++++++++++ gcc/testsuite/gfortran.dg/pr96024.f90 | 11 +++++++++++ 2 files changed, 24 insertions(+) diff --git a/gcc/fortran/resolve.c b/gcc/fortran/resolve.c index 29125694e1e..18e766494f2 100644 --- a/gcc/fortran/resolve.c +++ b/gcc/fortran/resolve.c @@ -14747,6 +14747,19 @@ resolve_component (gfc_component *c, gfc_symbol *sym) c->ts.u.cl->length ? &c->ts.u.cl->length->where : &c->loc); return false; } + + if (c->ts.u.cl->length && c->ts.u.cl->length->ts.type != BT_INTEGER) + { + if (!c->ts.u.cl->length->error) + { + gfc_error ("Character length expression of component %qs at %L " + "must be of INTEGER type, found %s", + c->name, &c->ts.u.cl->length->where, + gfc_basic_typename (c->ts.u.cl->length->ts.type)); + c->ts.u.cl->length->error = 1; + } + return false; + } } if (c->ts.type == BT_CHARACTER && c->ts.deferred diff --git a/gcc/testsuite/gfortran.dg/pr96024.f90 b/gcc/testsuite/gfortran.dg/pr96024.f90 new file mode 100644 index 00000000000..2c914a997f2 --- /dev/null +++ b/gcc/testsuite/gfortran.dg/pr96024.f90 @@ -0,0 +1,11 @@ +! { dg-do compile } +! PR fortran/96024 - ICE in mio_name_expr_t +! Contributed by G.Steinmetz + +module m + implicit none + type t + character(char(1)) :: a ! { dg-error "must be of INTEGER type" } + end type + type(t) :: z = t('a') +end