[Bug tree-optimization/26147] New: another memory leak

[Bug tree-optimization/26147] New: another memory leak

[uttamp at us dot ibm dot com]

After running coverity tool I discovered following memory leaks.

1) in gcc/opts.c: memory never freed allocated by 'new_option' pointer.

2) in gcc/df-core.c: Memory never freed allocated by 'postorder' pointer.

3) in tree-data-ref.c: memory never freed allocated by dist_v and init_v
pointers.

I'll submit the patches to fix these leaks.


-- 
           Summary: another memory leak
           Product: gcc
           Version: 4.2.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: tree-optimization
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: uttamp at us dot ibm dot com


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

[Bug other/26147] another memory leak

[pinskia at gcc dot gnu dot org]

------- Comment #1 from pinskia at gcc dot gnu dot org  2006-02-07 02:02 -------
Confirmed.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
          Component|tree-optimization           |other
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2006-02-07 02:02:23
               date|                            |


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

[Bug other/26147] another memory leak

[dje at gcc dot gnu dot org]

------- Comment #2 from dje at gcc dot gnu dot org  2006-02-15 19:33 -------
Subject: Bug 26147

Author: dje
Date: Wed Feb 15 19:33:33 2006
New Revision: 111114

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=111114
Log:
2006-02-15  Uttam Pawar  <uttamp@us.ibm.com>

        PR rtl-optimization/26184
        * modulo-sched.c (generate_reg_moves): Free bitmap vector
        uses_of_defs.
        * modulo-sched.c (sms_schedule): Free g_arr pointer.
        * modulo-sched.c (sms_schedule_by_order): Free bitmap pointers
        must_precede, must_follow and tobe_scheduled.

        PR other/26147
        * opts.c (common_handle_option): Free new_option pointer.
        * df-core.c (df_analyze): Free postorder pointer.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/df-core.c
    trunk/gcc/modulo-sched.c
    trunk/gcc/opts.c


-- 


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

[Bug other/26147] another memory leak

[pinskia at gcc dot gnu dot org]

------- Comment #3 from pinskia at gcc dot gnu dot org  2006-02-15 23:09 -------
Fixed.

dist_v and init_v are GC pointers so they don't really need to be freed
manually.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
           Keywords|                            |memory-hog
         Resolution|                            |FIXED
   Target Milestone|---                         |4.2.0


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

[Bug other/26147] another memory leak

[uttamp at us dot ibm dot com]

------- Comment #4 from uttamp at us dot ibm dot com  2006-02-16 00:35 -------
Subject: Re:  another memory leak


> 
> ------- Comment #3 from pinskia at gcc dot gnu dot org  2006-02-15 23:09 -------
> Fixed.
> 
> dist_v and init_v are GC pointers so they don't really need to be freed
> manually.

That's correct. I realized it after submitting the bug report.


-- 


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