------- Additional Comments From nickc at redhat dot com  2008-02-20 15:56 
-------
Hi Diogo,

  You are correct, the dwarf2.c code does not clean up its memory usage very
well.  The patch you suggested is a start, but it does not go far enough.  For
example it does not free the varinfo and funcinfo structures that are also
allocated in scan_unit_for_symbols.  So I am going to apply the uploaded
variation on your patch, along with this changelog entry.

  I notice that I did not follow up on the bfd_realloc_or_free function idea
either, so I will do that as well.

Cheers
  Nick

bfd/ChangeLog
2008-02-20  Diogo de Carvalho Kraemer  <[EMAIL PROTECTED]>
            Nick Clifton  <[EMAIL PROTECTED]>

        PR 868
        * dwarf2.c (read_abbrevs): Free the abbreviation table if we run
        out of memory.
        (decode_line_info): Free the line_info_table before returning a
        failure result.
        (_bfd_dwarf2_cleanup_debug_info): Free the abbreviation table.
        Free the line table.  Free the function table.  Free the variable
        table.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=868

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.


_______________________________________________
bug-binutils mailing list
bug-binutils@gnu.org
http://lists.gnu.org/mailman/listinfo/bug-binutils

Reply via email to