From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 2071) id 0B38F38582AB; Sun, 5 Mar 2023 19:36:49 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 0B38F38582AB DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1678045009; bh=dvRysB+JruwhxOupJVQ8Wd16B3NiJ1L7zCfxxKmlfEI=; h=From:To:Subject:Date:From; b=jEuHeBHwSxOSKdbnDJdgMbGVteTnoOuU1Q0utDneJ+K3A78HSkPeykRmtvFuxhr3n nk0ey6fEev6BfEXcJz5x+DPWegxykccrZpJ8FQ8oPB2nQ5fvB9l0ua2H4ZhGZ9bTDX XDxdxe22lNOrOQdeGNg33FQDSCT9Y//y4Mawuec4= 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 r10-11240] 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-10 X-Git-Oldrev: 9db1287c8ced5425f6ef9d26b05a3eb9cbcc4b8d X-Git-Newrev: 22031c5868bbb434b51ea4010ccae0cf8c890d6b Message-Id: <20230305193649.0B38F38582AB@sourceware.org> Date: Sun, 5 Mar 2023 19:36:49 +0000 (GMT) List-Id: https://gcc.gnu.org/g:22031c5868bbb434b51ea4010ccae0cf8c890d6b commit r10-11240-g22031c5868bbb434b51ea4010ccae0cf8c890d6b 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 3163f63c996..288970deca0 100644 --- a/gcc/fortran/resolve.c +++ b/gcc/fortran/resolve.c @@ -14643,6 +14643,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