Re: [PATCH] drm/gma500: fix double freeing

2015-10-05 Thread Patrik Jakobsson
On Fri, Oct 2, 2015 at 5:56 PM, Sudip Mukherjee wrote: > On Thu, Oct 01, 2015 at 07:07:33PM +0200, Patrik Jakobsson wrote: >> On Wed, Sep 30, 2015 at 8:12 AM, Sudip Mukherjee >> wrote: >> > On Tue, Sep 29, 2015 at 03:20:35PM +0200, Patrik Jakobsson wrote: >> >> On Thu, Sep 24, 2015 at 5:57 PM, Su

Re: [PATCH] drm/gma500: fix double freeing

2015-10-02 Thread Sudip Mukherjee
On Thu, Oct 01, 2015 at 07:07:33PM +0200, Patrik Jakobsson wrote: > On Wed, Sep 30, 2015 at 8:12 AM, Sudip Mukherjee > wrote: > > On Tue, Sep 29, 2015 at 03:20:35PM +0200, Patrik Jakobsson wrote: > >> On Thu, Sep 24, 2015 at 5:57 PM, Sudip Mukherjee > >> wrote: > >> > On Wed, Sep 09, 2015 at 06:2

Re: [PATCH] drm/gma500: fix double freeing

2015-10-01 Thread Patrik Jakobsson
On Wed, Sep 30, 2015 at 8:12 AM, Sudip Mukherjee wrote: > On Tue, Sep 29, 2015 at 03:20:35PM +0200, Patrik Jakobsson wrote: >> On Thu, Sep 24, 2015 at 5:57 PM, Sudip Mukherjee >> wrote: >> > On Wed, Sep 09, 2015 at 06:20:40PM +0530, Sudip Mukherjee wrote: >> >> If backing->stolen is true then we

Re: [PATCH] drm/gma500: fix double freeing

2015-09-29 Thread Sudip Mukherjee
On Tue, Sep 29, 2015 at 03:20:35PM +0200, Patrik Jakobsson wrote: > On Thu, Sep 24, 2015 at 5:57 PM, Sudip Mukherjee > wrote: > > On Wed, Sep 09, 2015 at 06:20:40PM +0530, Sudip Mukherjee wrote: > >> If backing->stolen is true then we were freeing backing by calling > >> psb_gtt_free_range() but w

Re: [PATCH] drm/gma500: fix double freeing

2015-09-29 Thread Patrik Jakobsson
On Thu, Sep 24, 2015 at 5:57 PM, Sudip Mukherjee wrote: > On Wed, Sep 09, 2015 at 06:20:40PM +0530, Sudip Mukherjee wrote: >> If backing->stolen is true then we were freeing backing by calling >> psb_gtt_free_range() but we called it again after unlocking the mutex. >> Lets make it NULL after free

Re: [PATCH] drm/gma500: fix double freeing

2015-09-24 Thread Sudip Mukherjee
On Wed, Sep 09, 2015 at 06:20:40PM +0530, Sudip Mukherjee wrote: > If backing->stolen is true then we were freeing backing by calling > psb_gtt_free_range() but we called it again after unlocking the mutex. > Lets make it NULL after freeing in psb_gtt_free_range() and check for > NULL before callin

[PATCH] drm/gma500: fix double freeing

2015-09-09 Thread Sudip Mukherjee
If backing->stolen is true then we were freeing backing by calling psb_gtt_free_range() but we called it again after unlocking the mutex. Lets make it NULL after freeing in psb_gtt_free_range() and check for NULL before calling the function for the second time. Signed-off-by: Sudip Mukherjee ---