[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

[mikael at gcc dot gnu dot org]

--- Comment #8 from mikael at gcc dot gnu dot org  2010-07-03 10:40 ---
Fixed, no need for backport. 


-- 

mikael at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED
   Target Milestone|--- |4.6.0


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44662

[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

[mikael at gcc dot gnu dot org]

--- Comment #6 from mikael at gcc dot gnu dot org  2010-07-02 15:56 ---
(In reply to comment #5)
 If you need someone to apply, test and commit , let me know.  I will have time
 and its pretty quick on my quad machine
 
No thanks, I will need a commit tree anyway. 
It's currently bootstrapping so it's a matter of hours now. 


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44662

[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

[mikael at gcc dot gnu dot org]

--- Comment #7 from mikael at gcc dot gnu dot org  2010-07-02 20:01 ---
Subject: Bug 44662

Author: mikael
Date: Fri Jul  2 20:01:05 2010
New Revision: 161739

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161739
Log:
2010-07-02  Mikael Morin  mik...@gcc.gnu.org

PR fortran/44662
* decl.c (match_procedure_in_type): Clear structure before using.
(gfc_match_generic): Ditto.


Modified:
trunk/gcc/fortran/ChangeLog
trunk/gcc/fortran/decl.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44662

[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

[mikael at gcc dot gnu dot org]

--- Comment #4 from mikael at gcc dot gnu dot org  2010-07-01 17:05 ---
PR44746 is a probable duplicate. 
I should hurry to commit before some others show up. 

I need to 
 * finish what I'm doing
 * retrieve my commit key from... well, where it is
 * find some space to save my work
 * actually save it
 * exercise my memory on that f*** pass phrase
 * checkout a fresh tree
 * bootstrap
 * test again, just to be sure
 * finally commit

I will report if I'm still stuck at step 2 next week, so that, would someone
have extra time, he could loose it in fixing this PR.


-- 

mikael at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |mikael at gcc dot gnu dot
   |dot org |org
 Status|UNCONFIRMED |ASSIGNED
 Ever Confirmed|0   |1
   Last reconfirmed|-00-00 00:00:00 |2010-07-01 17:05:25
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44662

[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

[jvdelisle at gcc dot gnu dot org]

--- Comment #5 from jvdelisle at gcc dot gnu dot org  2010-07-01 19:10 
---
If you need someone to apply, test and commit , let me know.  I will have time
and its pretty quick on my quad machine


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44662

[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

[dominiq at lps dot ens dot fr]

--- Comment #3 from dominiq at lps dot ens dot fr  2010-06-28 08:48 ---
This is likely the cause of recent failures for gfortran on ppc, see

http://gcc.gnu.org/ml/gcc-testresults/2010-06/msg02876.html
http://gcc.gnu.org/ml/gcc-testresults/2010-06/msg02871.html

The patch in comment #2 fixes the regression for
gfortran.dg/typebound_call_4.f03 on powerpc-apple-darwin9 without new failure.
Is not the patch qualifying for obvious?


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44662

[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

[mikael at gcc dot gnu dot org]

--- Comment #1 from mikael at gcc dot gnu dot org  2010-06-25 09:45 ---
It is fixed by the patch at:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43829#c13


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44662

[Bug fortran/44662] unitialized memory on testcases abstract_type_6.f03 and typebound_call_4.f03

[mikael at gcc dot gnu dot org]

--- Comment #2 from mikael at gcc dot gnu dot org  2010-06-25 11:29 ---
(In reply to comment #1)
 It is fixed by the patch at:
 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43829#c13
 

relevant hunk:

diff --git a/decl.c b/decl.c
index aa7a266..840c32c 100644
--- a/decl.c
+++ b/decl.c
@@ -7622,8 +7630,8 @@ match_procedure_in_type (void)
 }

   /* Construct the data structure.  */
+  memset (tb, 0, sizeof (tb));
   tb.where = gfc_current_locus;
-  tb.is_generic = 0;

   /* Match binding attributes.  */
   m = match_binding_attributes (tb, false, false);


-- 

mikael at gcc dot gnu dot org changed:

   What|Removed |Added

   Keywords||patch


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=44662