From 858dcee5f102fc7273e4ab1665e5230929a3c958 Mon Sep 17 00:00:00 2001 From: Alexander Preisinger Date: Thu, 19 Sep 2013 17:32:22 +0200 Subject: wayland/common: exit properly on fd errors Before this commit there was just an error message, but the file descriptor was still open. Now we close the file descriptor and prevent it from calling endlessly. Also a CLOSE_WIN event is sent which closes the window eventually if the action of CLOSE_WIN is set to quit or quit_watch_later. --- video/out/wayland_common.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/video/out/wayland_common.c b/video/out/wayland_common.c index 533b8c056d..109e2dc4b0 100644 --- a/video/out/wayland_common.c +++ b/video/out/wayland_common.c @@ -805,8 +805,12 @@ static int vo_wayland_check_events (struct vo *vo) * when pausing no input events get queued so we have to check if there * are events to read from the file descriptor through poll */ if (poll(&fd, 1, 0) > 0) { - if (fd.revents & POLLERR || fd.revents & POLLHUP) - MP_ERR(wl, "error occurred on the display fd\n"); + if (fd.revents & POLLERR || fd.revents & POLLHUP) { + MP_FATAL(wl, "error occurred on the display fd: " + "closing file descriptor\n"); + close(wl->display.display_fd); + mp_input_put_key(vo->input_ctx, MP_KEY_CLOSE_WIN); + } if (fd.revents & POLLIN) wl_display_dispatch(dp); if (fd.revents & POLLOUT) -- cgit v1.2.3