[PATCH 1/1] Resubmit mtd: mtdoops: fix for a potential memory leak in mtdoops_notify_remove

2013-08-12 Thread Nilanjan Roychowdhury
we are allocating cxt->oops_page_used using vmalloc in mtdoops_notify_add for every mtd_info addition but not freeing it in mtdoops_notify_remove. Also care is taken so that we do not free the same pointer again in module remove. Signed-off-by: Nilanjan Roychowdhury --- drivers/mtd/mtdoop

Re: [PATCH 1/1] mtd: mtdoops: fix for a potential memory leak in mtdoops_notify_remove

2013-08-12 Thread Nilanjan Roychowdhury
On Mon, Aug 12, 2013 at 10:56 PM, Ezequiel Garcia wrote: > > On Sun, Aug 11, 2013 at 01:11:48PM -0700, Nilanjan Roychowdhury wrote: > > we are allocating cxt->oops_page_used using vmalloc in mtdoops_notify_add > > for > > every mtd_info addition but not freeing

Re: [PATCH 1/1] mtd: mtdoops: fix for a potential memory leak in mtdoops_notify_remove

2013-08-12 Thread Nilanjan Roychowdhury
On Mon, Aug 12, 2013 at 10:56 PM, Ezequiel Garcia ezequiel.gar...@free-electrons.com wrote: On Sun, Aug 11, 2013 at 01:11:48PM -0700, Nilanjan Roychowdhury wrote: we are allocating cxt-oops_page_used using vmalloc in mtdoops_notify_add for every mtd_info addition but not freeing

[PATCH 1/1] Resubmit mtd: mtdoops: fix for a potential memory leak in mtdoops_notify_remove

2013-08-12 Thread Nilanjan Roychowdhury
we are allocating cxt-oops_page_used using vmalloc in mtdoops_notify_add for every mtd_info addition but not freeing it in mtdoops_notify_remove. Also care is taken so that we do not free the same pointer again in module remove. Signed-off-by: Nilanjan Roychowdhury nilanjan.roychowdh

[PATCH 1/1] mtd: mtdoops: fix for a potential memory leak in mtdoops_notify_remove

2013-08-11 Thread Nilanjan Roychowdhury
we are allocating cxt->oops_page_used using vmalloc in mtdoops_notify_add for every mtd_info addition but not freeing it in mtdoops_notify_remove Signed-off-by: Nilanjan Roychowdhury --- drivers/mtd/mtdoops.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mtd/mtdoops.c b/driv

[PATCH 1/1] mtd: mtdoops: fix for a potential memory leak in mtdoops_notify_remove

2013-08-11 Thread Nilanjan Roychowdhury
we are allocating cxt-oops_page_used using vmalloc in mtdoops_notify_add for every mtd_info addition but not freeing it in mtdoops_notify_remove Signed-off-by: Nilanjan Roychowdhury nilanjan.roychowdh...@gmail.com --- drivers/mtd/mtdoops.c |1 + 1 file changed, 1 insertion(+) diff --git