Re: [PATCH] kprobes: fix a memory leak in functionpre_handler_kretprobe()

[Greg KH]

On Wed, Jan 11, 2012 at 09:42:33PM +0530, Ananth N Mavinakayanahalli wrote:
> On Wed, Jan 11, 2012 at 11:21:51PM +0800, Jiang Liu wrote:
> > From: Jiang Liu <liuj97@xxxxxxxxx>
> > 
> > In function pre_handler_kretprobe(), the allocated kretprobe_instance object
> > will be leaked if the entry_handler callback returns non-zero. This may cause
> > all the preallocated kretprobe_instance objects exhausted. This issue could be
> > reproduced by changing samples/kprobes/kretprobe_example.c to probe 
> > "mutex_unlock". And the fix is straight forward, just put the allocated 
> > kretprobe_instance object back onto the free_instances list. 
> > 
> > Signed-off-by: Jiang Liu <jiang.liu@xxxxxxxxxx>
> > Acked-by: Jim Keniston <jkenisto@xxxxxxxxxx>
> > Acked-by: Ananth N Mavinakayanahalli <ananth@xxxxxxxxxx>
> 
> Thank you Jiang. I think the correct ID for stable is now
> stable@xxxxxxxxxxxxxxxx

<formletter>

This is not the correct way to submit patches for inclusion in the
stable kernel tree.  Please read Documentation/stable_kernel_rules.txt
for how to do this properly.

</formletter>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/