diff options
| author | Jasper St. Pierre <jstpierre@mecheye.net> | 2014-08-07 09:55:49 -0400 |
|---|---|---|
| committer | Pekka Paalanen <pekka.paalanen@collabora.co.uk> | 2014-08-21 13:51:07 +0300 |
| commit | 78d80474073b6145f6f03dd4928c3a4daf9c6b9c (patch) | |
| tree | dde53d203a299e046bf00ece8a544457c4186940 /tests/queue-test.c | |
| parent | tests: test the wl_display_roundtrip_queue() function (diff) | |
| download | wayland-78d80474073b6145f6f03dd4928c3a4daf9c6b9c.tar wayland-78d80474073b6145f6f03dd4928c3a4daf9c6b9c.tar.gz wayland-78d80474073b6145f6f03dd4928c3a4daf9c6b9c.tar.bz2 wayland-78d80474073b6145f6f03dd4928c3a4daf9c6b9c.tar.lz wayland-78d80474073b6145f6f03dd4928c3a4daf9c6b9c.tar.xz wayland-78d80474073b6145f6f03dd4928c3a4daf9c6b9c.tar.zst wayland-78d80474073b6145f6f03dd4928c3a4daf9c6b9c.zip | |
server: Don't expose wl_display as a global
The idea here was that once upon a time, clients could rebind wl_display
to a higher version, so we offered the ability to rebind it
here. However, this is particularly broken. The existing bind
implementation actually still hardcodes version numbers, and it leaks
previous resources, overwriting the existing one.
The newly bound resource *also* won't have any listeners attached by the
client, meaning that the error and delete_id events won't get delivered
correctly. Unless the client poked into libwayland internals, it also
can't possibly set up these handlers correctly either, so the client
will sustain errors and leak all deleted globals.
Since this never worked correctly in the first place, we can feel safe
removing it.
Acked-by: Jason Ekstrand <jason@jlekstrand.net>
Reviewed-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Diffstat (limited to 'tests/queue-test.c')
0 files changed, 0 insertions, 0 deletions
