[multiprocess RFA] Fix memory leak of focus_command

[multiprocess RFA] Fix memory leak of focus_command

[teawater]

[-- Attachment #1: Type: text/plain, Size: 215 bytes --]

Hi Stan,

I found that focus_command doesn't free current_itset before overwrite
it. So I write a patch for it.

2008-11-27  Hui Zhu  <teawater@gmail.com>

	* infcmd.c (focus_command): Fix memory leak.

Thanks,
Hui

[-- Attachment #2: fix_focus_command_memory_leak.txt --]
[-- Type: text/plain, Size: 231 bytes --]

--- gdb.orig/infcmd.c
+++ gdb/infcmd.c
@@ -219,6 +219,8 @@
       return;
     }
 
+  if (current_itset)
+    xfree (current_itset);
   current_itset = itset;
 
   /* For now, set a current exec from the first element of the focus

Re: [multiprocess RFA] Fix memory leak of focus_command

[Michael Snyder]

teawater wrote:
> Hi Stan,
> 
> I found that focus_command doesn't free current_itset before overwrite
> it. So I write a patch for it.

Look at you, my former protege, off working in other people's branches!
I'm so proud...   ;-)

Re: [multiprocess RFA] Fix memory leak of focus_command

[Michael Snyder]

Michael Snyder wrote:
> teawater wrote:
>> Hi Stan,
>>
>> I found that focus_command doesn't free current_itset before overwrite
>> it. So I write a patch for it.
> 
> Look at you, my former protege, off working in other people's branches!
> I'm so proud...   ;-)

Gracelessly I confess that this was supposed to go out privately.  ;-O