execute memcpy only success case

Change-Id: Ib730cb1dba433bf708743766680bb566ad91e112
Signed-off-by: Junkyeong Kim <jk0430.kim@samsung.com>

diff --git a/src/tdm_hwc.c b/src/tdm_hwc.c
index 0c498732faf0d22d90be957ff8666e0f92ace09c..11a7d5dac302733f4fb70c0d1f4b9e1518fdb172 100644 (file)
--- a/src/tdm_hwc.c
+++ b/src/tdm_hwc.c
@@ -1049,7 +1049,8 @@ tdm_hwc_get_property(tdm_hwc *hwc, uint32_t id, tdm_value *value)
        if (private_module->use_hal_tdm) {
                hal_tdm_value hvalue;
                ret = (tdm_error)hal_tdm_hwc_get_property((hal_tdm_hwc *)private_hwc->hwc_backend, id, &hvalue);
-               memcpy(value->ptr, &hvalue.ptr, sizeof(tdm_value));
+               if (ret == TDM_ERROR_NONE)
+                       memcpy(&value->ptr, &hvalue.ptr, sizeof(tdm_value));
        } else {
                if (!func_hwc->hwc_get_property) {
                        /* LCOV_EXCL_START */

diff --git a/src/tdm_hwc_window.c b/src/tdm_hwc_window.c
index 157cefab2b70a9bc525917ee59137caacff03eac..074131d2a3304d87d4ae60eef9cbabb05c996b90 100644 (file)
--- a/src/tdm_hwc_window.c
+++ b/src/tdm_hwc_window.c
@@ -468,7 +468,8 @@ tdm_hwc_window_get_property(tdm_hwc_window *hwc_window, unsigned int id, tdm_val
        if (private_module->use_hal_tdm) {
                hal_tdm_value hvalue;
                ret = hal_tdm_hwc_window_get_property((hal_tdm_hwc_window *)private_hwc_window->hwc_window_backend, id, &hvalue);
-               memcpy(value->ptr, &hvalue.ptr, sizeof(tdm_value));
+               if (ret == TDM_ERROR_NONE)
+                       memcpy(&value->ptr, &hvalue.ptr, sizeof(tdm_value));
        } else {
                if (!func_hwc_window->hwc_window_get_property) {
                        /* LCOV_EXCL_START */

diff --git a/src/tdm_output.c b/src/tdm_output.c
index bd2afa76baec206862e2ac8350d7c241b8bd96c0..9232de960a1a582073ceb271206b4ead574ed01c 100644 (file)
--- a/src/tdm_output.c
+++ b/src/tdm_output.c
@@ -950,7 +950,8 @@ tdm_output_get_property(tdm_output *output, unsigned int id, tdm_value *value)
        if (private_module->use_hal_tdm) {
                hal_tdm_value hvalue;
                ret = (tdm_error)hal_tdm_output_get_property(private_output->output_backend, id, &hvalue);
-               memcpy(value->ptr, &hvalue.ptr, sizeof(tdm_value));
+               if (ret == TDM_ERROR_NONE)
+                       memcpy(&value->ptr, &hvalue.ptr, sizeof(tdm_value));
        } else {
                func_output = &private_module->func_output;