libweston: assert make/model in weston_output_enable()
authorPekka Paalanen <pekka.paalanen@collabora.co.uk>
Thu, 24 Aug 2017 13:08:49 +0000 (16:08 +0300)
committerPekka Paalanen <pekka.paalanen@collabora.co.uk>
Tue, 17 Apr 2018 12:19:57 +0000 (15:19 +0300)
Output make and model are not allowed to be NULL in the protocol, so
ensure they are not forgotten when enabling an output.

Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.co.uk>
Reviewed-by: Ian Ray <ian.ray@ge.com>
Reviewed-by: Daniel Stone <daniels@collabora.com>
Acked-by: Derek Foreman <derekf@osg.samsung.com>
libweston/compositor.c

index 0efd570..547e826 100644 (file)
@@ -5524,6 +5524,7 @@ weston_output_enable(struct weston_output *output)
 {
        struct weston_compositor *c = output->compositor;
        struct weston_output *iterator;
+       struct weston_head *head;
        int x = 0, y = 0;
 
        if (output->enabled) {
@@ -5538,6 +5539,11 @@ weston_output_enable(struct weston_output *output)
                return -1;
        }
 
+       wl_list_for_each(head, &output->head_list, output_link) {
+               assert(head->make);
+               assert(head->model);
+       }
+
        iterator = container_of(c->output_list.prev,
                                struct weston_output, link);