When a thread is sleeping, waiting until another thread read
from the display, it always returns 0. Even when an error
occured. In documentation stands:
"return 0 on success or -1 on error. In case of error errno will
be set accordingly"
So this is a fix for this.
Along with the read_events, fix a test so that it now complies
with this behaviour (and we have this tested)
Signed-off-by: Marek Chalupa <mchqwerty@gmail.com>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
while (display->read_serial == serial)
pthread_cond_wait(&display->reader_cond,
&display->mutex);
+
+ if (display->last_error) {
+ errno = display->last_error;
+ return -1;
+ }
}
return 0;
* thread should be woken up or it will block indefinitely.
*/
c->display_stopped = 1;
- assert(wl_display_read_events(c->wl_display) == 0);
+ assert(wl_display_read_events(c->wl_display) == -1);
- wl_display_dispatch_pending(c->wl_display);
+ assert(wl_display_dispatch_pending(c->wl_display) == -1);
assert(wl_display_get_error(c->wl_display));
pthread_exit(NULL);