[Bug fortran/67588] module.c heap use after free

2015-09-17 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67588 --- Comment #3 from Paul Thomas --- Author: pault Date: Thu Sep 17 09:09:34 2015 New Revision: 227855 URL: https://gcc.gnu.org/viewcvs?rev=227855=gcc=rev Log: 2015-09-17 Paul Thomas PR fortran/52846 PR

[Bug fortran/67588] module.c heap use after free

2015-09-17 Thread zeccav at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67588 --- Comment #5 from Vittorio Zecca --- I believe that use_list = module_list; at line module.c:805 is useless and can be expunged.

[Bug fortran/67588] module.c heap use after free

2015-09-17 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67588 --- Comment #6 from Paul Thomas --- (In reply to Vittorio Zecca from comment #5) > I believe that > > use_list = module_list; > > at line module.c:805 is useless and can be expunged. I don't think so Up to that point, use_list is set to

[Bug fortran/67588] module.c heap use after free

2015-09-17 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67588 Paul Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug fortran/67588] module.c heap use after free

2015-09-17 Thread zeccav at gmail dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67588 --- Comment #7 from Vittorio Zecca --- use_list is local to the function gfc_match_submodule and it is already reinitialized in the for statement at line 806. So there is a duplication. On return at line 812 it is then lost. Maybe optimization

[Bug fortran/67588] module.c heap use after free

2015-09-16 Thread pault at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67588 Paul Thomas changed: What|Removed |Added Assignee|unassigned at gcc dot gnu.org |pault at gcc dot gnu.org ---

[Bug fortran/67588] module.c heap use after free

2015-09-15 Thread kargl at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67588 kargl at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |NEW Last reconfirmed|