add ouput_list empty checking 74/100974/1
authorJunkyeong Kim <jk0430.kim@samsung.com>
Wed, 30 Nov 2016 04:38:55 +0000 (13:38 +0900)
committerJunkyeong Kim <jk0430.kim@samsung.com>
Wed, 30 Nov 2016 04:39:05 +0000 (13:39 +0900)
Change-Id: I84472e40bd79edff7c3c80d811bde08f283f9321
Signed-off-by: Junkyeong Kim <jk0430.kim@samsung.com>
src/tdm_drm_display.c

index 9ff29ee..ad0e6d6 100644 (file)
@@ -382,16 +382,16 @@ _tdm_drm_display_create_layer_list(tdm_drm_data *drm_data)
        tdm_drm_output_data *output_data = NULL;
        int i;
 
+       if (LIST_IS_EMPTY(&drm_data->output_list)) {
+               TDM_ERR("no output");
+               return TDM_ERROR_OPERATION_FAILED;
+       }
+
        /* The TDM drm backend only support one output. */
        LIST_FOR_EACH_ENTRY(output_data, &drm_data->output_list, link) {
                break;
        }
 
-       if (!output_data) {
-               TDM_ERR("no output");
-               return TDM_ERROR_OPERATION_FAILED;
-       }
-
        if (drm_data->plane_res->count_planes == 0) {
                TDM_ERR("no layer error");
                return TDM_ERROR_OPERATION_FAILED;
@@ -493,16 +493,16 @@ _tdm_drm_display_create_layer_list_type(tdm_drm_data *drm_data)
        tdm_error ret;
        int i;
 
+       if (LIST_IS_EMPTY(&drm_data->output_list)) {
+               TDM_ERR("no output");
+               return TDM_ERROR_OPERATION_FAILED;
+       }
+
        /* The TDM drm backend only support one output. */
        LIST_FOR_EACH_ENTRY(output_data, &drm_data->output_list, link) {
                break;
        }
 
-       if (!output_data) {
-               TDM_ERR("no output");
-               return TDM_ERROR_OPERATION_FAILED;
-       }
-
        ret = _tdm_drm_display_get_property(drm_data,
                                            drm_data->plane_res->planes[0],
                                            DRM_MODE_OBJECT_PLANE, "type", &type,