Message ID | 5C79BC4A.2030103@gmx.de |
---|---|
State | New |
Headers | show |
Series | [PR,fortran/77583] - ICE in pp_quoted_string, at pretty-print.c:966 | expand |
On Sat, Mar 02, 2019 at 12:12:10AM +0100, Harald Anlauf wrote: > The attached patch (originally by Steve Kargl) fixes a NULL pointer > dereference that may occur when checking for a conflict. > > Regtested successfully. > > OK for trunk? Backport to active branches? > > > 2019-03-02 Harald Anlauf <anlauf@gmx.de> > Steve Kargl <sgk@troutmask.apl.washington.edu> Steven G. Kargl <kargl@gcc.gnu.org> ;-) I, of course, approve of the patch, but you might give others a chance to disagree.
I didn't see any disagreement, so committed to trunk (rev.269353) and "backported" to 7- and 8-branches. Thanks, Harald On 03/02/19 00:15, Steve Kargl wrote: > On Sat, Mar 02, 2019 at 12:12:10AM +0100, Harald Anlauf wrote: >> The attached patch (originally by Steve Kargl) fixes a NULL pointer >> dereference that may occur when checking for a conflict. >> >> Regtested successfully. >> >> OK for trunk? Backport to active branches? >> >> >> 2019-03-02 Harald Anlauf <anlauf@gmx.de> >> Steve Kargl <sgk@troutmask.apl.washington.edu> > > Steven G. Kargl <kargl@gcc.gnu.org> > > ;-) > > I, of course, approve of the patch, but you might give > others a chance to disagree. >
Index: gcc/fortran/symbol.c =================================================================== --- gcc/fortran/symbol.c (revision 269332) +++ gcc/fortran/symbol.c (working copy) @@ -525,7 +525,7 @@ /* The copying of procedure dummy arguments for module procedures in a submodule occur whilst the current state is COMP_CONTAINS. It is necessary, therefore, to let this through. */ - if (attr->dummy + if (name && attr->dummy && (attr->function || attr->subroutine) && gfc_current_state () == COMP_CONTAINS && !(gfc_new_block && gfc_new_block->abr_modproc_decl))