[Mesa-dev] [PATCH 2/5] nv50, nvc0: fix potential resource leak in nvXX_create_texture_view()

2015-12-14 Thread Samuel Pitoiset
Spotted by Coverity. Signed-off-by: Samuel Pitoiset --- src/gallium/drivers/nouveau/nv50/nv50_tex.c | 3 ++- src/gallium/drivers/nouveau/nvc0/nvc0_tex.c | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/nouveau/nv50/nv50_tex.c

Re: [Mesa-dev] [PATCH 2/5] nv50, nvc0: fix potential resource leak in nvXX_create_texture_view()

2015-12-14 Thread Samuel Pitoiset
On 12/14/2015 04:01 PM, Ilia Mirkin wrote: Again, bogus. Can't get there. I'd take a patch that asserts though, or marks it unreachable, or just drops the default case entirely. Make it unreachable is fine by me. On Mon, Dec 14, 2015 at 5:14 AM, Samuel Pitoiset

Re: [Mesa-dev] [PATCH 2/5] nv50, nvc0: fix potential resource leak in nvXX_create_texture_view()

2015-12-14 Thread Ilia Mirkin
Again, bogus. Can't get there. I'd take a patch that asserts though, or marks it unreachable, or just drops the default case entirely. On Mon, Dec 14, 2015 at 5:14 AM, Samuel Pitoiset wrote: > Spotted by Coverity. > > Signed-off-by: Samuel Pitoiset