Re: [PATCH] drm/nouveau/ga102: Free resources on error in ga102_chan_new()

2021-09-21 Thread Karol Herbst
On Tue, Sep 21, 2021 at 3:22 PM Tim Gardner wrote: > > > > On 9/20/21 8:07 PM, Karol Herbst wrote: > > On Mon, Sep 20, 2021 at 8:17 PM Tim Gardner > > wrote: > >> > >> Coverity complains of a resource leak in ga102_chan_new(): > >> > >> CID 119637 (#7 of 7): Resource leak (RESOURCE_LEAK) > >> 13

Re: [PATCH] drm/nouveau/ga102: Free resources on error in ga102_chan_new()

2021-09-21 Thread Tim Gardner
On 9/20/21 8:07 PM, Karol Herbst wrote: On Mon, Sep 20, 2021 at 8:17 PM Tim Gardner wrote: Coverity complains of a resource leak in ga102_chan_new(): CID 119637 (#7 of 7): Resource leak (RESOURCE_LEAK) 13. leaked_storage: Variable chan going out of scope leaks the storage it points to. 19

Re: [PATCH] drm/nouveau/ga102: Free resources on error in ga102_chan_new()

2021-09-20 Thread Karol Herbst
On Mon, Sep 20, 2021 at 8:17 PM Tim Gardner wrote: > > Coverity complains of a resource leak in ga102_chan_new(): > > CID 119637 (#7 of 7): Resource leak (RESOURCE_LEAK) > 13. leaked_storage: Variable chan going out of scope leaks the storage it > points to. > 190return ret; > > F

[PATCH] drm/nouveau/ga102: Free resources on error in ga102_chan_new()

2021-09-20 Thread Tim Gardner
Coverity complains of a resource leak in ga102_chan_new(): CID 119637 (#7 of 7): Resource leak (RESOURCE_LEAK) 13. leaked_storage: Variable chan going out of scope leaks the storage it points to. 190return ret; Fix this by freeing 'chan' in the error path. Cc: Ben Skeggs Cc: Da