[PATCH libinput] Check realloc() return value before assigning.

Signed-off-by: Carlos Olmedo Escobar --- src/libinput.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/libinput.c b/src/libinput.c index bcd0dcd..e5a1ae4 100644 --- a/src/libinput.c +++ b/src/libinput.c @@ -2578,6 +2578,7 @@ libinput_post_event(struct libinput

Re: [PATCH v6 wayland-protocols] Add the tablet protocol

A couple of grammar nitpicks below: 2016-03-01 1:09 GMT+01:00, Peter Hutterer : > Signed-off-by: Peter Hutterer > Reviewed-by: Daniel Stone > --- > Changes to v5: > - remove leftover statement falsely claminig wp_tablet_tool.destroy requires > a remove event first > > Makefile.am

[PATCH wayland-protocols] Remove 'is'.

Signed-off-by: Carlos Olmedo Escobar --- unstable/pointer-constraints/pointer-constraints-unstable-v1.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/unstable/pointer-constraints/pointer-constraints-unstable-v1.xml b/unstable/pointer-constraints/pointer-constraints

Re: [PATCH weston v2 8/8] clients: add simple-dmabuf client

Hi, Sorry for not inlining my comment on the original mail. I don't have it. The line with "if (!window->buffers[1].buffer)" at the email "[PATCH weston v2 8/8] clients: add simple-dmabuf client" by Pekka maybe is wrong, I'm not 100% sure but perhaps it is meant to be if (!window->buffers[i].buff

[PATCH weston] Separate declarations from code.

Thanks Pekka for pointing that out. Signed-off-by: Carlos Olmedo Escobar --- ivi-shell/ivi-layout-transition.c | 21 +++-- 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/ivi-shell/ivi-layout-transition.c b/ivi-shell/ivi-layout-transition.c index 904e8d6..f691d35

Re: [PATCH libinput] test: fix Coverity complaints

Hi, Even though I'm quite unknown in this list, this patch is so trivial that I guess I can give you my: Reviewed-by: Carlos Olmedo Escobar 2015-02-18 22:28 GMT+01:00, Peter Hutterer : > seat_button_count > seat_key_count ... uninitialized variable > > t = zalloc > s = zall

libinput in Coverity

Hi, I was trying to check libinput in Coverity but it seems it's not there yet. Is there anybody who is supposed to register libinput there? Regards, ~Carlos. ___ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.or

Re: [PATCH weston] ivi-shell-transition: check create_layout_transition() return value.

--- >> From: wayland-devel >> [mailto:wayland-devel-boun...@lists.freedesktop.org] On Behalf Of >> Carlos Olmedo Escobar >> Sent: Sunday, January 18, 2015 3:43 AM >> To: wayland-devel@lists.freedesktop.org >> Cc: Carlos Olmedo Escobar >> Subject

[PATCH weston] zoom: Check the value of level before using it.

Check the value of level before dividing 1 by it. Signed-off-by: Carlos Olmedo Escobar --- src/zoom.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/zoom.c b/src/zoom.c index 7553849..f71d08c 100644 --- a/src/zoom.c +++ b/src/zoom.c @@ -120,12 +120,13

[PATCH weston] ivi-shell-transition: check create_layout_transition() return value.

Signed-off-by: Carlos Olmedo Escobar --- ivi-shell/ivi-layout-transition.c | 16 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/ivi-shell/ivi-layout-transition.c b/ivi-shell/ivi-layout-transition.c index 1e9ee23..904e8d6 100644 --- a/ivi-shell/ivi-layout

[wayland] Remove useless semicolon.

Signed-off-by: Carlos Olmedo Escobar --- src/wayland-server.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/wayland-server.c b/src/wayland-server.c index 270e5c7..c40d300 100644 --- a/src/wayland-server.c +++ b/src/wayland-server.c @@ -1118,7 +1118,7

[PATCH weston] compositor: Fix weston_subsurface_is_synchronized() return value.

Commit 280e7dd918f1717c7d677676384a9cd991097741 introduced a bug in the return value of weston_subsurface_is_synchronized(). Signed-off-by: Carlos Olmedo Escobar --- src/compositor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/compositor.c b/src/compositor.c index

[PATCH libinput v2] evdev-mt-touchpad: check calloc result

Check the value returned by calloc. Signed-off-by: Carlos Olmedo Escobar --- src/evdev-mt-touchpad.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/evdev-mt-touchpad.c b/src/evdev-mt-touchpad.c index 6355364..109441d 100644 --- a/src/evdev-mt-touchpad.c +++ b/src/evdev-mt-touchpad.c

[PATCH libinput v2] evdev-mt-touchpad: check calloc result

Check the value returned by calloc. --- src/evdev-mt-touchpad.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/evdev-mt-touchpad.c b/src/evdev-mt-touchpad.c index 6355364..109441d 100644 --- a/src/evdev-mt-touchpad.c +++ b/src/evdev-mt-touchpad.c @@ -721,6 +721,8 @@ tp_init_slots(struct

[PATCH libinput] evdev-mt-touchpad: check calloc result

Check the value returned by calloc. --- src/evdev-mt-touchpad.c | 5 + 1 file changed, 5 insertions(+) diff --git a/src/evdev-mt-touchpad.c b/src/evdev-mt-touchpad.c index bbbd8f3..9ec1682 100644 --- a/src/evdev-mt-touchpad.c +++ b/src/evdev-mt-touchpad.c @@ -25,6 +25,7 @@ #include #includ