vblank: check return value

Change-Id: I465d4528be887606e8113cdbd16f6da860dc5907

diff --git a/src/tdm_vblank.c b/src/tdm_vblank.c
index dddb423..3b55944 100644 (file)
--- a/src/tdm_vblank.c
+++ b/src/tdm_vblank.c
@@ -384,8 +384,8 @@ _tdm_vblank_update_output_info(tdm_private_vblank *private_vblank)
 
        if (connection == TDM_OUTPUT_CONN_STATUS_MODE_SETTED) {
                const tdm_output_mode *mode = NULL;
-               tdm_output_get_mode(output, &mode);
-               if (mode)
+               ret = tdm_output_get_mode(output, &mode);
+               if (ret == TDM_ERROR_NONE && mode)
                        vrefresh = mode->vrefresh;
                else
                        VWR("mode not setted!!!");
@@ -456,6 +456,7 @@ EXTERN tdm_error
 tdm_vblank_set_client_vblank_fps(unsigned int pid, const char *name, unsigned int fps)
 {
        tdm_private_vblank *v = NULL;
+       tdm_error ret = TDM_ERROR_NONE;
 
        TDM_RETURN_VAL_IF_FAIL(pid > 0, TDM_ERROR_INVALID_PARAMETER);
        TDM_RETURN_VAL_IF_FAIL(fps > 0, TDM_ERROR_INVALID_PARAMETER);
@@ -484,9 +485,11 @@ tdm_vblank_set_client_vblank_fps(unsigned int pid, const char *name, unsigned in
                                continue;
                }
 
-               tdm_vblank_set_fps(v, fps);
-
-               TDM_INFO("(pid:%u) '%s' fps changed: %d", pid, v->name, fps);
+               ret = tdm_vblank_set_fps(v, fps);
+               if (ret == TDM_ERROR_NONE)
+                       TDM_INFO("(pid:%u) '%s' fps changed: %d", pid, v->name, fps);
+               else
+                       TDM_ERR("(pid:%u) '%s' fps changing failed: %d", pid, v->name, fps);
        }
        pthread_mutex_unlock(&valid_list_lock);
 
@@ -497,6 +500,7 @@ EXTERN tdm_error
 tdm_vblank_set_client_ignore_global_fps(unsigned int pid, const char *name, unsigned int ignore)
 {
        tdm_private_vblank *v = NULL;
+       tdm_error ret = TDM_ERROR_NONE;
 
        TDM_RETURN_VAL_IF_FAIL(pid > 0, TDM_ERROR_INVALID_PARAMETER);
 
@@ -524,9 +528,11 @@ tdm_vblank_set_client_ignore_global_fps(unsigned int pid, const char *name, unsi
                                continue;
                }
 
-               tdm_vblank_ignore_global_fps(v, ignore);
-
-               TDM_INFO("(pid:%u) '%s' ignore changed: %u", pid, v->name, ignore);
+               ret = tdm_vblank_ignore_global_fps(v, ignore);
+               if (ret == TDM_ERROR_NONE)
+                       TDM_INFO("(pid:%u) '%s' ignore changed: %u", pid, v->name, ignore);
+               else
+                       TDM_ERR("(pid:%u) '%s' ignore changing failed: %u", pid, v->name, ignore);
        }
        pthread_mutex_unlock(&valid_list_lock);