Fix null check after deref
[platform/core/uifw/libtdm.git] / src / tdm_capture.c
index 92e306a..3d36ed4 100644 (file)
@@ -49,240 +49,386 @@ SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
     TDM_RETURN_VAL_IF_FAIL(capture != NULL, TDM_ERROR_INVALID_PARAMETER); \
     private_capture = (tdm_private_capture*)capture; \
     private_display = private_capture->private_display; \
-    func_capture = private_capture->func_capture
+    func_capture = &private_display->func_capture
 
-static void
-_tdm_caputre_cb_done(tdm_capture *capture_backend, tbm_surface_h buffer, void *user_data)
+static tdm_error
+_tdm_capture_check_if_exist(tdm_private_capture *private_capture,
+                            tbm_surface_h buffer)
 {
-    tdm_private_capture *private_capture = user_data;
-    tdm_private_display *private_display = private_capture->private_display;
-    int lock_after_cb_done = 0;
-    int ret;
-
-    ret = pthread_mutex_trylock(&private_display->lock);
-    if (ret == 0)
-        pthread_mutex_unlock(&private_display->lock);
-    else  if (ret == EBUSY)
-    {
-        pthread_mutex_unlock(&private_display->lock);
-        lock_after_cb_done = 1;
-    }
-
-    tdm_buffer_unref_backend(buffer);
-
-    if (lock_after_cb_done)
-        pthread_mutex_lock(&private_display->lock);
+       tdm_buffer_info *buf_info = NULL;
+
+       LIST_FOR_EACH_ENTRY(buf_info, &private_capture->buffer_list, link) {
+               if (buf_info->buffer == buffer) {
+                       TDM_ERR("%p attached twice", buffer);
+                       return TDM_ERROR_BAD_REQUEST;
+               }
+       }
+
+       LIST_FOR_EACH_ENTRY(buf_info, &private_capture->pending_buffer_list, link) {
+               if (buf_info->buffer == buffer) {
+                       TDM_ERR("%p attached twice", buffer);
+                       return TDM_ERROR_BAD_REQUEST;
+               }
+       }
+
+       return TDM_ERROR_NONE;
 }
 
-INTERN tdm_private_capture*
-tdm_capture_create_output_internal(tdm_private_output *private_output, tdm_error *error)
+INTERN void
+tdm_capture_cb_done(tdm_capture *capture_backend, tbm_surface_h buffer,
+                    void *user_data)
+{
+       tdm_private_capture *private_capture = user_data;
+       tdm_private_display *private_display = private_capture->private_display;
+       tdm_buffer_info *buf_info;
+       tbm_surface_h first_entry;
+
+       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
+
+       if (private_capture->owner_tid != syscall(SYS_gettid)) {
+               tdm_thread_cb_capture_done capture_done;
+               tdm_error ret;
+
+               capture_done.base.type = TDM_THREAD_CB_PP_DONE;
+               capture_done.base.length = sizeof capture_done;
+               capture_done.capture_stamp = private_capture->stamp;
+               capture_done.buffer = buffer;
+               capture_done.user_data = user_data;
+
+               ret = tdm_thread_send_cb(private_display->private_loop, &capture_done.base);
+               TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+
+               return;
+       }
+
+       if (private_capture->owner_tid != syscall(SYS_gettid))
+               TDM_NEVER_GET_HERE();
+
+       if (tdm_debug_buffer)
+               TDM_INFO("capture(%p) done: %p", private_capture, buffer);
+
+       first_entry = tdm_buffer_list_get_first_entry(&private_capture->buffer_list);
+       if (first_entry != buffer)
+               TDM_ERR("%p is skipped", first_entry);
+
+       if ((buf_info = tdm_buffer_get_info(buffer)))
+               LIST_DEL(&buf_info->link);
+
+       _pthread_mutex_unlock(&private_display->lock);
+       tdm_buffer_unref_backend(buffer);
+       _pthread_mutex_lock(&private_display->lock);
+}
+
+INTERN tdm_private_capture *
+tdm_capture_find_stamp(tdm_private_display *private_display, unsigned long stamp)
 {
-    tdm_private_display *private_display;
-    tdm_func_display *func_display;
-    tdm_func_capture *func_capture;
-    tdm_private_capture *private_capture = NULL;
-    tdm_capture *capture_backend = NULL;
-    tdm_error ret = TDM_ERROR_NONE;
-
-    private_display = private_output->private_display;
-    func_display = &private_display->func_display;
-    func_capture = &private_display->func_capture;
-
-    if (!(private_display->capabilities & TDM_DISPLAY_CAPABILITY_CAPTURE))
-    {
-        TDM_ERR("no capture capability");
-        if (error)
-            *error = TDM_ERROR_BAD_REQUEST;
-        return NULL;
-    }
-
-    capture_backend = func_display->output_create_capture(private_output->output_backend, &ret);
-    if (ret != TDM_ERROR_NONE)
-    {
-        if (error)
-            *error = ret;
-        return NULL;
-    }
-
-    private_capture = calloc(1, sizeof(tdm_private_capture));
-    if (!private_capture)
-    {
-        TDM_ERR("failed: alloc memory");
-        func_capture->capture_destroy(capture_backend);
-        if (error)
-            *error = TDM_ERROR_OUT_OF_MEMORY;
-        return NULL;
-    }
-
-    ret = func_capture->capture_set_done_handler(capture_backend, _tdm_caputre_cb_done, private_capture);
-    if (ret != TDM_ERROR_NONE)
-    {
-        TDM_ERR("set capture_done_handler failed");
-        func_capture->capture_destroy(capture_backend);
-        if (error)
-            *error = ret;
-        return NULL;
-    }
-
-    LIST_ADD(&private_capture->link, &private_output->capture_list);
-    private_capture->func_capture = func_capture;
-    private_capture->target = TDM_CAPTURE_TARGET_OUTPUT;
-    private_capture->private_display = private_display;
-    private_capture->private_output = private_output;
-    private_capture->private_layer = NULL;
-    private_capture->capture_backend = capture_backend;
-
-    if (error)
-        *error = TDM_ERROR_NONE;
-
-    return private_capture;
+       tdm_private_capture *private_capture = NULL;
+
+       TDM_RETURN_VAL_IF_FAIL(TDM_MUTEX_IS_LOCKED(), NULL);
+
+       LIST_FOR_EACH_ENTRY(private_capture, &private_display->capture_list, link) {
+               if (private_capture->stamp == stamp)
+                       return private_capture;
+       }
+
+       return NULL;
 }
 
-INTERN tdm_private_capture*
-tdm_capture_create_layer_internal(tdm_private_layer *private_layer, tdm_error *error)
+INTERN tdm_private_capture *
+tdm_capture_create_output_internal(tdm_private_output *private_output,
+                                   tdm_error *error)
 {
-    tdm_private_display *private_display;
-    tdm_private_output *private_output;
-    tdm_func_display *func_display;
-    tdm_func_capture *func_capture;
-    tdm_private_capture *private_capture = NULL;
-    tdm_capture *capture_backend = NULL;
-    tdm_error ret = TDM_ERROR_NONE;
-
-    private_output = private_layer->private_output;
-    private_display = private_output->private_display;
-    func_display = &private_display->func_display;
-    func_capture = &private_display->func_capture;
-
-    if (!(private_display->capabilities & TDM_DISPLAY_CAPABILITY_CAPTURE))
-    {
-        TDM_ERR("no capture capability");
-        if (error)
-            *error = TDM_ERROR_BAD_REQUEST;
-        return NULL;
-    }
-
-    capture_backend = func_display->layer_create_capture(private_layer->layer_backend, &ret);
-    if (ret != TDM_ERROR_NONE)
-        return NULL;
-
-    private_capture = calloc(1, sizeof(tdm_private_capture));
-    if (!private_capture)
-    {
-        TDM_ERR("failed: alloc memory");
-        func_capture->capture_destroy(capture_backend);
-        if (error)
-            *error = TDM_ERROR_OUT_OF_MEMORY;
-        return NULL;
-    }
-
-    LIST_ADD(&private_capture->link, &private_output->capture_list);
-    private_capture->target = TDM_CAPTURE_TARGET_LAYER;
-    private_capture->func_capture = func_capture;
-    private_capture->private_display = private_display;
-    private_capture->private_output = private_output;
-    private_capture->private_layer = private_layer;
-    private_capture->capture_backend = capture_backend;
-
-    if (error)
-        *error = TDM_ERROR_NONE;
-
-    return private_capture;
+       tdm_private_display *private_display = private_output->private_display;
+       tdm_func_output *func_output = &private_display->func_output;
+       tdm_func_capture *func_capture = &private_display->func_capture;
+       tdm_private_capture *private_capture = NULL;
+       tdm_capture *capture_backend = NULL;
+       tdm_error ret = TDM_ERROR_NONE;
+
+       TDM_RETURN_VAL_IF_FAIL(TDM_MUTEX_IS_LOCKED(), NULL);
+
+       if (!(private_display->capabilities & TDM_DISPLAY_CAPABILITY_CAPTURE)) {
+               TDM_ERR("no capture capability");
+               if (error)
+                       *error = TDM_ERROR_NO_CAPABILITY;
+               return NULL;
+       }
+
+       capture_backend = func_output->output_create_capture(
+                                 private_output->output_backend, &ret);
+       if (ret != TDM_ERROR_NONE) {
+               if (error)
+                       *error = ret;
+               return NULL;
+       }
+
+       private_capture = calloc(1, sizeof(tdm_private_capture));
+       if (!private_capture) {
+               TDM_ERR("failed: alloc memory");
+               func_capture->capture_destroy(capture_backend);
+               if (error)
+                       *error = TDM_ERROR_OUT_OF_MEMORY;
+               return NULL;
+       }
+
+       ret = func_capture->capture_set_done_handler(capture_backend,
+                       tdm_capture_cb_done, private_capture);
+       if (ret != TDM_ERROR_NONE) {
+               TDM_ERR("capture(%p) set capture_done_handler failed", private_capture);
+               func_capture->capture_destroy(capture_backend);
+               if (error)
+                       *error = ret;
+               return NULL;
+       }
+
+       private_capture->stamp = tdm_helper_get_time_in_millis();
+       while (tdm_capture_find_stamp(private_display, private_capture->stamp))
+               private_capture->stamp++;
+
+       LIST_ADD(&private_capture->link, &private_output->capture_list);
+       LIST_ADD(&private_capture->display_link, &private_display->capture_list);
+
+       private_capture->target = TDM_CAPTURE_TARGET_OUTPUT;
+       private_capture->private_display = private_display;
+       private_capture->private_output = private_output;
+       private_capture->private_layer = NULL;
+       private_capture->capture_backend = capture_backend;
+       private_capture->owner_tid = syscall(SYS_gettid);
+
+       LIST_INITHEAD(&private_capture->pending_buffer_list);
+       LIST_INITHEAD(&private_capture->buffer_list);
+
+       if (error)
+               *error = TDM_ERROR_NONE;
+
+       return private_capture;
+}
+
+INTERN tdm_private_capture *
+tdm_capture_create_layer_internal(tdm_private_layer *private_layer,
+                                  tdm_error *error)
+{
+       tdm_private_output *private_output = private_layer->private_output;
+       tdm_private_display *private_display = private_output->private_display;
+       tdm_func_layer *func_layer = &private_display->func_layer;
+       tdm_func_capture *func_capture = &private_display->func_capture;
+       tdm_private_capture *private_capture = NULL;
+       tdm_capture *capture_backend = NULL;
+       tdm_error ret = TDM_ERROR_NONE;
+
+       TDM_RETURN_VAL_IF_FAIL(TDM_MUTEX_IS_LOCKED(), NULL);
+
+       if (!(private_display->capabilities & TDM_DISPLAY_CAPABILITY_CAPTURE)) {
+               TDM_ERR("no capture capability");
+               if (error)
+                       *error = TDM_ERROR_NO_CAPABILITY;
+               return NULL;
+       }
+
+       capture_backend = func_layer->layer_create_capture(private_layer->layer_backend,
+                         &ret);
+       if (ret != TDM_ERROR_NONE)
+               return NULL;
+
+       private_capture = calloc(1, sizeof(tdm_private_capture));
+       if (!private_capture) {
+               TDM_ERR("failed: alloc memory");
+               func_capture->capture_destroy(capture_backend);
+               if (error)
+                       *error = TDM_ERROR_OUT_OF_MEMORY;
+               return NULL;
+       }
+
+       private_capture->stamp = tdm_helper_get_time_in_millis();
+       while (tdm_capture_find_stamp(private_display, private_capture->stamp))
+               private_capture->stamp++;
+
+       LIST_ADD(&private_capture->link, &private_layer->capture_list);
+       LIST_ADD(&private_capture->display_link, &private_display->capture_list);
+
+       private_capture->target = TDM_CAPTURE_TARGET_LAYER;
+       private_capture->private_display = private_display;
+       private_capture->private_output = private_output;
+       private_capture->private_layer = private_layer;
+       private_capture->capture_backend = capture_backend;
+       private_capture->owner_tid = syscall(SYS_gettid);
+
+       LIST_INITHEAD(&private_capture->pending_buffer_list);
+       LIST_INITHEAD(&private_capture->buffer_list);
+
+       if (error)
+               *error = TDM_ERROR_NONE;
+
+       return private_capture;
 }
 
 INTERN void
 tdm_capture_destroy_internal(tdm_private_capture *private_capture)
 {
-    tdm_func_capture *func_capture;
+       tdm_private_display *private_display;
+       tdm_func_capture *func_capture;
+       tdm_buffer_info *b = NULL, *bb = NULL;
+
+       TDM_RETURN_IF_FAIL(TDM_MUTEX_IS_LOCKED());
+
+       if (!private_capture)
+               return;
+
+        private_display = private_capture->private_display;
 
-    if (!private_capture)
-        return;
+       LIST_DEL(&private_capture->link);
+       LIST_DEL(&private_capture->display_link);
 
-    LIST_DEL(&private_capture->link);
+       func_capture = &private_capture->private_display->func_capture;
+       func_capture->capture_destroy(private_capture->capture_backend);
 
-    func_capture = private_capture->func_capture;
-    func_capture->capture_destroy(private_capture->capture_backend);
+       if (!LIST_IS_EMPTY(&private_capture->pending_buffer_list)) {
+               TDM_WRN("capture(%p) not finished:", private_capture);
+               tdm_buffer_list_dump(&private_capture->pending_buffer_list);
 
-    free(private_capture);
+               LIST_FOR_EACH_ENTRY_SAFE(b, bb, &private_capture->pending_buffer_list, link) {
+                       LIST_DEL(&b->link);
+                       _pthread_mutex_unlock(&private_display->lock);
+                       tdm_buffer_unref_backend(b->buffer);
+                       _pthread_mutex_lock(&private_display->lock);
+               }
+       }
+
+       if (!LIST_IS_EMPTY(&private_capture->buffer_list)) {
+               TDM_WRN("capture(%p) not finished:", private_capture);
+               tdm_buffer_list_dump(&private_capture->buffer_list);
+
+               LIST_FOR_EACH_ENTRY_SAFE(b, bb, &private_capture->buffer_list, link) {
+                       LIST_DEL(&b->link);
+                       _pthread_mutex_unlock(&private_display->lock);
+                       tdm_buffer_unref_backend(b->buffer);
+                       _pthread_mutex_lock(&private_display->lock);
+               }
+       }
+
+       private_capture->stamp = 0;
+       free(private_capture);
 }
 
 EXTERN void
 tdm_capture_destroy(tdm_capture *capture)
 {
-    tdm_private_capture *private_capture = capture;
-    tdm_private_display *private_display;
+       tdm_private_capture *private_capture = capture;
+       tdm_private_display *private_display;
 
-    if (!private_capture)
-        return;
+       if (!private_capture)
+               return;
 
-    private_display = private_capture->private_display;
+       private_display = private_capture->private_display;
 
-    pthread_mutex_lock(&private_display->lock);
-    tdm_capture_destroy_internal(private_capture);
-    pthread_mutex_unlock(&private_display->lock);
+       _pthread_mutex_lock(&private_display->lock);
+       tdm_capture_destroy_internal(private_capture);
+       _pthread_mutex_unlock(&private_display->lock);
 }
 
 EXTERN tdm_error
 tdm_capture_set_info(tdm_capture *capture, tdm_info_capture *info)
 {
-    CAPTURE_FUNC_ENTRY();
+       CAPTURE_FUNC_ENTRY();
 
-    TDM_RETURN_VAL_IF_FAIL(info != NULL, TDM_ERROR_INVALID_PARAMETER);
+       TDM_RETURN_VAL_IF_FAIL(info != NULL, TDM_ERROR_INVALID_PARAMETER);
 
-    pthread_mutex_lock(&private_display->lock);
+       _pthread_mutex_lock(&private_display->lock);
 
-    if (!func_capture->capture_set_info)
-    {
-        pthread_mutex_unlock(&private_display->lock);
-        return TDM_ERROR_NONE;
-    }
+       if (!func_capture->capture_set_info) {
+               _pthread_mutex_unlock(&private_display->lock);
+               TDM_DBG("failed: not implemented!!");
+               return TDM_ERROR_NOT_IMPLEMENTED;
+       }
 
-    ret = func_capture->capture_set_info(private_capture->capture_backend, info);
+       ret = func_capture->capture_set_info(private_capture->capture_backend, info);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 
-    pthread_mutex_unlock(&private_display->lock);
+       _pthread_mutex_unlock(&private_display->lock);
 
-    return ret;
+       return ret;
 }
 
 EXTERN tdm_error
 tdm_capture_attach(tdm_capture *capture, tbm_surface_h buffer)
 {
-    CAPTURE_FUNC_ENTRY();
+       CAPTURE_FUNC_ENTRY();
 
-    TDM_RETURN_VAL_IF_FAIL(buffer != NULL, TDM_ERROR_INVALID_PARAMETER);
+       TDM_RETURN_VAL_IF_FAIL(buffer != NULL, TDM_ERROR_INVALID_PARAMETER);
 
-    pthread_mutex_lock(&private_display->lock);
+       _pthread_mutex_lock(&private_display->lock);
 
-    if (!func_capture->capture_attach)
-    {
-        pthread_mutex_unlock(&private_display->lock);
-        return TDM_ERROR_NONE;
-    }
+       if (!func_capture->capture_attach) {
+               _pthread_mutex_unlock(&private_display->lock);
+               TDM_DBG("failed: not implemented!!");
+               return TDM_ERROR_NOT_IMPLEMENTED;
+       }
 
-    tdm_buffer_ref_backend(buffer);
-    ret = func_capture->capture_attach(private_capture->capture_backend, buffer);
+       ret = _tdm_capture_check_if_exist(private_capture, buffer);
+       if (ret != TDM_ERROR_NONE) {
+               _pthread_mutex_unlock(&private_display->lock);
+               return ret;
+       }
 
-    pthread_mutex_unlock(&private_display->lock);
+       ret = func_capture->capture_attach(private_capture->capture_backend, buffer);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
 
-    return ret;
-}
+       if (ret == TDM_ERROR_NONE) {
+               tdm_buffer_info *buf_info;
 
-EXTERN tdm_error
-tdm_capture_commit(tdm_capture *capture)
-{
-    CAPTURE_FUNC_ENTRY();
-
-    pthread_mutex_lock(&private_display->lock);
+               if ((buf_info = tdm_buffer_get_info(buffer)))
+                       LIST_ADDTAIL(&buf_info->link, &private_capture->pending_buffer_list);
 
-    if (!func_capture->capture_commit)
-    {
-        pthread_mutex_unlock(&private_display->lock);
-        return TDM_ERROR_NONE;
-    }
+               if (tdm_debug_buffer) {
+                       TDM_INFO("capture(%p) attached:", private_capture);
+                       tdm_buffer_list_dump(&private_capture->buffer_list);
+               }
+       }
 
-    ret = func_capture->capture_commit(private_capture->capture_backend);
+       _pthread_mutex_unlock(&private_display->lock);
 
-    pthread_mutex_unlock(&private_display->lock);
+       return ret;
+}
 
-    return ret;
+EXTERN tdm_error
+tdm_capture_commit(tdm_capture *capture)
+{
+       tdm_buffer_info *b = NULL, *bb = NULL;
+       tdm_private_output *private_output;
+
+       CAPTURE_FUNC_ENTRY();
+
+       _pthread_mutex_lock(&private_display->lock);
+
+       private_output = private_capture->private_output;
+       if (private_output->current_dpms_value > TDM_OUTPUT_DPMS_ON) {
+               TDM_ERR("output(%d) dpms: %s", private_output->pipe,
+                       dpms_str(private_output->current_dpms_value));
+               _pthread_mutex_unlock(&private_display->lock);
+               return TDM_ERROR_BAD_REQUEST;
+       }
+
+       if (!func_capture->capture_commit) {
+               _pthread_mutex_unlock(&private_display->lock);
+               TDM_DBG("failed: not implemented!!");
+               return TDM_ERROR_NOT_IMPLEMENTED;
+       }
+
+       ret = func_capture->capture_commit(private_capture->capture_backend);
+       TDM_WARNING_IF_FAIL(ret == TDM_ERROR_NONE);
+
+       if (ret == TDM_ERROR_NONE) {
+               LIST_FOR_EACH_ENTRY_SAFE(b, bb, &private_capture->pending_buffer_list, link) {
+                       LIST_DEL(&b->link);
+                       tdm_buffer_ref_backend(b->buffer);
+                       LIST_ADDTAIL(&b->link, &private_capture->buffer_list);
+               }
+       } else {
+               LIST_FOR_EACH_ENTRY_SAFE(b, bb, &private_capture->pending_buffer_list, link)
+                       LIST_DEL(&b->link);
+       }
+
+       _pthread_mutex_unlock(&private_display->lock);
+
+       return ret;
 }