Re: [Mesa-dev] [PATCH mesa] egl/drm: remove temporary fd variable

2017-05-31 Thread Emil Velikov
On 31 May 2017 at 17:55, Eric Engestrom wrote: > In all codepaths, this var ends up assigned to the struct, except one: > a cleanup codepath, where the `close()` was removed, leading to fd leaks. > Remove the temp fd and assign to the struct field directly instead. > >

[Mesa-dev] [PATCH mesa] egl/drm: remove temporary fd variable

2017-05-31 Thread Eric Engestrom
In all codepaths, this var ends up assigned to the struct, except one: a cleanup codepath, where the `close()` was removed, leading to fd leaks. Remove the temp fd and assign to the struct field directly instead. CovID: 1213930 Fixes: 7ec07beedf486d8ab901 ("egl/drm: make use of the