On Wed, Mar 20, 2019 at 2:26 PM Chia-I Wu <olva...@gmail.com> wrote:
> Reviewed-by: Chia-I Wu <olva...@gmail.com>
>
Anything else to need for merging this? I think this is a straightforward
leaking fix.
>
> On Mon, Mar 18, 2019 at 4:40 PM Lepton Wu <lep...@chromium.org> wrote:
>
>> This fd was create in virgl_drm_screen_create and should be closed
>> in virgl_drm_screen_destroy.
>>
>> Signed-off-by: Lepton Wu <lep...@chromium.org>
>> ---
>> src/gallium/winsys/virgl/drm/virgl_drm_winsys.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/src/gallium/winsys/virgl/drm/virgl_drm_winsys.c
>> b/src/gallium/winsys/virgl/drm/virgl_drm_winsys.c
>> index 01811a0e997..5501fe3ed48 100644
>> --- a/src/gallium/winsys/virgl/drm/virgl_drm_winsys.c
>> +++ b/src/gallium/winsys/virgl/drm/virgl_drm_winsys.c
>> @@ -973,6 +973,7 @@ virgl_drm_screen_destroy(struct pipe_screen *pscreen)
>> if (destroy) {
>> int fd = virgl_drm_winsys(screen->vws)->fd;
>> util_hash_table_remove(fd_tab, intptr_to_pointer(fd));
>> + close(fd);
>> }
>> mtx_unlock(&virgl_screen_mutex);
>>
>> --
>> 2.21.0.225.g810b269d1ac-goog
>>
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev@lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
