diff options
| author | Leandro Ribeiro <leandrohr@riseup.net> | 2019-12-03 18:32:05 -0300 |
|---|---|---|
| committer | Daniel Stone <daniel@fooishbar.org> | 2019-12-11 19:36:57 +0000 |
| commit | 2a6cfef07eeadc265fffe9f623564fcfb896b78f (patch) | |
| tree | 0a9cee944b9ca9227edac752e9bcdf901956abb0 | |
| parent | 1b3ad0993baf1e87c403d11d23786892ea65a2e5 (diff) | |
screen-share: define variable type before using as function argument
Define the void pointer as pixman_region32_t instead of using it
directly.
Void pointers are used by generic API's so each user can define
a data type as needed. Not doing this is dangerous, since void
pointers implicitly cast from/to any pointer without giving
warnings. So we must define its type instead of using it directly
in a function call.
Signed-off-by: Leandro Ribeiro <leandrohr@riseup.net>
| -rw-r--r-- | compositor/screen-share.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/compositor/screen-share.c b/compositor/screen-share.c index a2b21594..c8256fac 100644 --- a/compositor/screen-share.c +++ b/compositor/screen-share.c @@ -819,6 +819,7 @@ shared_output_repainted(struct wl_listener *listener, void *data) struct shared_output *so = container_of(listener, struct shared_output, frame_listener); pixman_region32_t damage; + pixman_region32_t *current_damage = data; struct ss_shm_buffer *sb; int32_t x, y, width, height, stride; int i, nrects, do_yflip, y_orig; @@ -847,7 +848,7 @@ shared_output_repainted(struct wl_listener *listener, void *data) } else { /* Damage in output coordinates */ pixman_region32_init(&damage); - pixman_region32_intersect(&damage, &so->output->region, data); + pixman_region32_intersect(&damage, &so->output->region, current_damage); pixman_region32_translate(&damage, -so->output->x, -so->output->y); } |