[Bug gas/30861] Memory leak

[nickc at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=30861

Nick Clifton  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #3 from Nick Clifton  ---
Patch applied.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug gas/30861] Memory leak

[cvs-commit at gcc dot gnu.org]

https://sourceware.org/bugzilla/show_bug.cgi?id=30861

--- Comment #2 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Nick Clifton :

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=dc63d5682e48ab64724a2da2ee803cd13bf5f0c0

commit dc63d5682e48ab64724a2da2ee803cd13bf5f0c0
Author: Nick Clifton 
Date:   Mon Oct 2 16:23:14 2023 +0100

Fix memory leak in RiscV assembler.

  PR 30861
  * config/tc-riscv.c (riscv_insert_uleb128_fixes): Release duplicated
memory.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Bug gas/30861] Memory leak

[nickc at redhat dot com]

https://sourceware.org/bugzilla/show_bug.cgi?id=30861

Nick Clifton  changed:

   What|Removed |Added

 CC||nickc at redhat dot com

--- Comment #1 from Nick Clifton  ---
Hi Jacob,

  Thanks for the bug report.   In general it is not always safe to free an
expression that has been passed to the fix_new_expr() function, as parts of the
expr structure might be referenced by symbols created during the call.  In the
particular case of the riscv_insert_uleb128_fixes() function, it should be safe
however, as the operation is set to O_symbol, which does not result in further
symbols being created.

  I will check in the obvious patch to add a free statement.

Cheers
  Nick

-- 
You are receiving this mail because:
You are on the CC list for the bug.