------- Additional Comments From nickc at redhat dot com  2008-01-09 10:44 
-------
Hi Jakub, Hi Andreas,

  I agree that there is no reason why we should not fix the leaks, so I have
gone ahead and applied a slight variant of Jakub's patch (uploaded).  The
changes I made were:

  * Omitted the change to list_matching_formats().  There was no need for it
    and any decent compiler will produce the same code either way.

  * Moved the architecture walking variable ('arch') in the set_endianess() 
    functions inside the if-statement.

  * Added a ChangeLog entry.

Cheers
  Nick

binutils/ChangeLog
2008-01-09  Jakub Zawadzki  <[EMAIL PROTECTED]>

        PR binutils/55326
        * bucomm.c (list_supported_architectures): Free architecture list
        after use.
        * windres.c (set_endianess): Likewise.
        * windmc.c (set_endianess): Likewise.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


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

------- 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