diff options
| author | Derek Foreman <derekf@osg.samsung.com> | 2015-05-11 15:40:11 -0500 |
|---|---|---|
| committer | Daniel Stone <daniels@collabora.com> | 2015-08-06 16:28:30 +0100 |
| commit | f9318d145262444e2b671b40122fec9cab39b609 (patch) | |
| tree | b0595e3f5de23db355dea0c5c567cb65183df631 /src/data-device.c | |
| parent | 2ccd9a2d6247d26975a8a846e9eb7a283c84a1ba (diff) | |
| download | weston-f9318d145262444e2b671b40122fec9cab39b609.tar.gz | |
input: add a weston_pointer_clear_focus() helper function
Valgrind has shown that in at least one place (default_grab_pointer_focus)
we're testing uninitialized values coming out of weston_compositor_pick_view.
This is happening when default_grab_pointer_focus is called when there is
nothing on the view list, and during the first repaint when only the black
surface with no input region exists.
This patch adds a function to clear pointer focus and also set the sx,sy
co-ordinates to a sentinel value we shouldn't compute with.
Assertions are added to make sure any time pointer focus is set to NULL
these values are used.
weston_compositor_pick_view() now returns these values too.
Now the values are always initialized, even when no view exists, and
they're initialized in such a way that actually doing computation
with them should fail in an obvious way, but we can compare them
safely for equality.
Signed-off-by: Derek Foreman <derekf@osg.samsung.com>
Reviewed-by: Daniel Stone <daniels@collabora.com>
Diffstat (limited to 'src/data-device.c')
| -rw-r--r-- | src/data-device.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/data-device.c b/src/data-device.c index 89ffce49..1612091a 100644 --- a/src/data-device.c +++ b/src/data-device.c @@ -591,8 +591,7 @@ weston_pointer_start_drag(struct weston_pointer *pointer, &drag->base.data_source_listener); } - weston_pointer_set_focus(pointer, NULL, - wl_fixed_from_int(0), wl_fixed_from_int(0)); + weston_pointer_clear_focus(pointer); weston_pointer_start_grab(pointer, &drag->grab); return 0; |