[PATCH] mgag200: Fix a memory leak in mgag200fb_create()

2012-06-24 Thread Jesper Juhl
On Fri, 22 Jun 2012, devendra.aaru wrote: > On Fri, Jun 22, 2012 at 3:43 AM, Jesper Juhl wrote: > > First we allocate memory for 'sysram' with vmalloc() and subsequently > > we allocate for 'info' with framebuffer_alloc(). If the second > > allocation fails we return -ENOMEM, but neglect to vfree

Re: [PATCH] mgag200: Fix a memory leak in mgag200fb_create()

2012-06-24 Thread Jesper Juhl
On Fri, 22 Jun 2012, devendra.aaru wrote: > On Fri, Jun 22, 2012 at 3:43 AM, Jesper Juhl wrote: > > First we allocate memory for 'sysram' with vmalloc() and subsequently > > we allocate for 'info' with framebuffer_alloc(). If the second > > allocation fails we return -ENOMEM, but neglect to vfree

[PATCH] mgag200: Fix a memory leak in mgag200fb_create()

2012-06-22 Thread devendra.aaru
On Fri, Jun 22, 2012 at 3:43 AM, Jesper Juhl wrote: > First we allocate memory for 'sysram' with vmalloc() and subsequently > we allocate for 'info' with framebuffer_alloc(). If the second > allocation fails we return -ENOMEM, but neglect to vfree() the memory > we previously allocated for 'sysram

Re: [PATCH] mgag200: Fix a memory leak in mgag200fb_create()

2012-06-22 Thread devendra.aaru
On Fri, Jun 22, 2012 at 3:43 AM, Jesper Juhl wrote: > First we allocate memory for 'sysram' with vmalloc() and subsequently > we allocate for 'info' with framebuffer_alloc(). If the second > allocation fails we return -ENOMEM, but neglect to vfree() the memory > we previously allocated for 'sysram

[PATCH] mgag200: Fix a memory leak in mgag200fb_create()

2012-06-22 Thread Jesper Juhl
First we allocate memory for 'sysram' with vmalloc() and subsequently we allocate for 'info' with framebuffer_alloc(). If the second allocation fails we return -ENOMEM, but neglect to vfree() the memory we previously allocated for 'sysram', thus leaking it. Signed-off-by: Jesper Juhl --- drivers

[PATCH] mgag200: Fix a memory leak in mgag200fb_create()

2012-06-21 Thread Jesper Juhl
First we allocate memory for 'sysram' with vmalloc() and subsequently we allocate for 'info' with framebuffer_alloc(). If the second allocation fails we return -ENOMEM, but neglect to vfree() the memory we previously allocated for 'sysram', thus leaking it. Signed-off-by: Jesper Juhl --- drivers