diff options
| author | Harald Anlauf <anlauf@gmx.de> | 2021-05-05 15:25:50 +0200 |
|---|---|---|
| committer | Harald Anlauf <anlauf@gmx.de> | 2021-05-05 15:25:50 +0200 |
| commit | a8b79cc939d6786293f654c42a2d1b0ab040de0e (patch) | |
| tree | 03f0d8b2532c9e139e588d1fdd1ab1c8fe494560 /gcc/fortran/trans-expr.c | |
| parent | f3661f2d63fbc5fd30c24d22137691e16b0a0a17 (diff) | |
| download | gcc-a8b79cc939d6786293f654c42a2d1b0ab040de0e.zip gcc-a8b79cc939d6786293f654c42a2d1b0ab040de0e.tar.gz gcc-a8b79cc939d6786293f654c42a2d1b0ab040de0e.tar.bz2 | |
PR fortran/100274 - ICE in gfc_conv_procedure_call, at fortran/trans-expr.c:6131
When the check for the length of formal and actual character arguments
found a mismatch and emitted a warning, it would skip further checks
like that could lead to errors. Fix that by continuing the checking.
Also catch a NULL pointer dereference.
gcc/fortran/ChangeLog:
PR fortran/100274
* interface.c (gfc_compare_actual_formal): Continue checks after
emitting warning for argument length mismatch.
* trans-expr.c (gfc_conv_procedure_call): Check for NULL pointer
dereference.
gcc/testsuite/ChangeLog:
PR fortran/100274
* gfortran.dg/argument_checking_25.f90: New test.
Diffstat (limited to 'gcc/fortran/trans-expr.c')
| -rw-r--r-- | gcc/fortran/trans-expr.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/gcc/fortran/trans-expr.c b/gcc/fortran/trans-expr.c index b83b021..9389a45 100644 --- a/gcc/fortran/trans-expr.c +++ b/gcc/fortran/trans-expr.c @@ -6125,6 +6125,7 @@ gfc_conv_procedure_call (gfc_se * se, gfc_symbol * sym, bool add_clobber; add_clobber = fsym && fsym->attr.intent == INTENT_OUT && !fsym->attr.allocatable && !fsym->attr.pointer + && e->symtree && e->symtree->n.sym && !e->symtree->n.sym->attr.dimension && !e->symtree->n.sym->attr.pointer && !e->symtree->n.sym->attr.allocatable |