[PATCH 1/2] compositor-drm: Ensure strings from EDID blobs are null terminated

2015-08-28 Thread Bryce Harrington
strncpy only adds null terminating bytes if the source string is smaller than the destination string. Since this function relies on the string being null terminated when checking its contents, we better make sure there is at least a \0 as the last character. Signed-off-by: Bryce Harrington ---

[PATCH 2/2] Ensure strncpy results are null terminated.

2015-08-28 Thread Bryce Harrington
Signed-off-by: Bryce Harrington --- clients/window.c | 1 + src/compositor-fbdev.c | 1 + 2 files changed, 2 insertions(+) diff --git a/clients/window.c b/clients/window.c index 0e73f5b..614b320 100644 --- a/clients/window.c +++ b/clients/window.c @@ -5886,6 +5886,7 @@ keysym_modifiers_add

[PATCH 4/5] ivi-shell: remove is_surface_in_layer API

2015-08-28 Thread Ucan, Emre (ADITG/SW1)
This internal API is redundant, because a surface is allowed to be only on one layer. Signed-off-by: Emre Ucan --- ivi-shell/ivi-layout.c | 23 +-- 1 file changed, 1 insertion(+), 22 deletions(-) diff --git a/ivi-shell/ivi-layout.c b/ivi-shell/ivi-layout.c index 47c044a..

[PATCH 5/5] ivi-shell: introduce get_weston_view

2015-08-28 Thread Ucan, Emre (ADITG/SW1)
The internal API "get_weston_view" is introduced, which returns the weston_view of the given ivi_layout_surface. The API returns a NULL pointer, if the ivi_layout_surface does not have any weston_view. The weston_view is required in many places of ivi-shell implementation. Therefore, this API wil

[PATCH 2/5] ivi-shell: remove struct link_layer

2015-08-28 Thread Ucan, Emre (ADITG/SW1)
link_layer's sole purpose is to link a surface to multiple layers, if the surface should be shown in multiple layers. This can be only achieved, if the surface has multiple weston_views with different transformation matrices. Current implementation assumes in many places that a ivi_surface has o

[PATCH 1/5] ivi-shell: remove struct link_screen

2015-08-28 Thread Ucan, Emre (ADITG/SW1)
link_screen's sole purpose is to link a layer to multiple screens, if the layer should be shown in multiple screens. This can be only achieved, if surfaces of the layer have multiple weston_views dedicated to the different screens. Current implementation assumes in many places that a ivi_surface

[PATCH 3/5] ivi-shell: remove is_layer_in_screen API

2015-08-28 Thread Ucan, Emre (ADITG/SW1)
This internal API is redundant, because a layer is allowed to be only on one screen. Signed-off-by: Emre Ucan --- ivi-shell/ivi-layout.c | 19 +-- 1 file changed, 1 insertion(+), 18 deletions(-) diff --git a/ivi-shell/ivi-layout.c b/ivi-shell/ivi-layout.c index 2a33b3e..47c04