Re: [Mesa-dev] [PATCH] winsys/radeon: don't leak the fd when it is 0

2015-07-29 Thread Mario Kleiner
On 07/29/2015 09:05 PM, Alex Deucher wrote: On Wed, Jul 29, 2015 at 2:55 PM, Mario Kleiner wrote: On 07/29/2015 08:50 PM, Alex Deucher wrote: On Wed, Jul 29, 2015 at 2:48 PM, Mario Kleiner wrote: On 07/29/2015 05:46 PM, Alex Deucher wrote: On Wed, Jul 29, 2015 at 10:44 AM, Emil Veli

Re: [Mesa-dev] [PATCH] winsys/radeon: don't leak the fd when it is 0

2015-07-29 Thread Alex Deucher
On Wed, Jul 29, 2015 at 2:55 PM, Mario Kleiner wrote: > On 07/29/2015 08:50 PM, Alex Deucher wrote: >> >> On Wed, Jul 29, 2015 at 2:48 PM, Mario Kleiner >> wrote: >>> >>> >>> >>> On 07/29/2015 05:46 PM, Alex Deucher wrote: On Wed, Jul 29, 2015 at 10:44 AM, Emil Velikov w

Re: [Mesa-dev] [PATCH] winsys/radeon: don't leak the fd when it is 0

2015-07-29 Thread Mario Kleiner
On 07/29/2015 08:50 PM, Alex Deucher wrote: On Wed, Jul 29, 2015 at 2:48 PM, Mario Kleiner wrote: On 07/29/2015 05:46 PM, Alex Deucher wrote: On Wed, Jul 29, 2015 at 10:44 AM, Emil Velikov wrote: Earlier commit added an extra dup(fd) to fix a ZaphodHeads issue. Although it did not consid

Re: [Mesa-dev] [PATCH] winsys/radeon: don't leak the fd when it is 0

2015-07-29 Thread Alex Deucher
On Wed, Jul 29, 2015 at 2:48 PM, Mario Kleiner wrote: > > > On 07/29/2015 05:46 PM, Alex Deucher wrote: >> >> On Wed, Jul 29, 2015 at 10:44 AM, Emil Velikov >> wrote: >>> >>> Earlier commit added an extra dup(fd) to fix a ZaphodHeads issue. >>> Although it did not consider the (very unlikely) cas

Re: [Mesa-dev] [PATCH] winsys/radeon: don't leak the fd when it is 0

2015-07-29 Thread Mario Kleiner
On 07/29/2015 05:46 PM, Alex Deucher wrote: On Wed, Jul 29, 2015 at 10:44 AM, Emil Velikov wrote: Earlier commit added an extra dup(fd) to fix a ZaphodHeads issue. Although it did not consider the (very unlikely) case where we might end up with the valid fd == 0. Fixes: 28dda47ae4d(winsys/ra

Re: [Mesa-dev] [PATCH] winsys/radeon: don't leak the fd when it is 0

2015-07-29 Thread Alex Deucher
On Wed, Jul 29, 2015 at 10:44 AM, Emil Velikov wrote: > Earlier commit added an extra dup(fd) to fix a ZaphodHeads issue. > Although it did not consider the (very unlikely) case where we might end > up with the valid fd == 0. > > Fixes: 28dda47ae4d(winsys/radeon: Use dup fd as key in drm-winsys ha

[Mesa-dev] [PATCH] winsys/radeon: don't leak the fd when it is 0

2015-07-29 Thread Emil Velikov
Earlier commit added an extra dup(fd) to fix a ZaphodHeads issue. Although it did not consider the (very unlikely) case where we might end up with the valid fd == 0. Fixes: 28dda47ae4d(winsys/radeon: Use dup fd as key in drm-winsys hash table to fix ZaphodHeads.) Cc: Mario Kleiner Signed-off-by: