move va_copy for unexpected return without va_end

[Version] 0.0.11
[Profile] Common
[Issue Type] Bug fix
[Dependency module] N/A
[Test] [M(T) - Boot=(OK), sdb=(OK), Home=(OK), Touch=(OK), Version=tizen-3.0-mobile_20161130.2]

Signed-off-by: SeokHoon Lee <andy.shlee@samsung.com>
Change-Id: I7999c4279d29823ab0404226da5ce148e4e19583

diff --git a/packaging/libmm-streamrecorder.spec b/packaging/libmm-streamrecorder.spec
index f850b96..f63eaf5 100644 (file)
--- a/packaging/libmm-streamrecorder.spec
+++ b/packaging/libmm-streamrecorder.spec
@@ -1,6 +1,6 @@
 Name:       libmm-streamrecorder
 Summary:    Media Stream Recorder library
-Version:    0.0.10
+Version:    0.0.11
 Release:    0
 Group:      Multimedia/Other
 License:    Apache-2.0

diff --git a/src/mm_streamrecorder_attribute.c b/src/mm_streamrecorder_attribute.c
index 6db79d1..e908d0f 100644 (file)
--- a/src/mm_streamrecorder_attribute.c
+++ b/src/mm_streamrecorder_attribute.c
@@ -864,15 +864,15 @@ int _mmstreamrecorder_set_attributes(MMHandleType handle, char **err_attr_name,
 
        mmf_return_val_if_fail(handle, MM_ERROR_STREAMRECORDER_INVALID_ARGUMENT);
 
-       /* copy var_args to keep original var_args */
-       va_copy(var_args_copy, var_args);
-
        attrs = MMF_STREAMRECORDER_ATTRS(handle);
        if (!attrs) {
                _mmstreamrec_dbg_err("handle 0x%x, attrs is NULL, attr name [%s]", handle, attribute_name);
                return MM_ERROR_STREAMRECORDER_NOT_INITIALIZED;
        }
 
+       /* copy var_args to keep original var_args */
+       va_copy(var_args_copy, var_args);
+
        ret = __mmstreamrecorder_check_valid_pair(handle, err_attr_name, attribute_name, var_args);
 
        _mmstreamrec_dbg_err("__mmstreamrecorder_check_valid_pair handle 0x%x, attr name [%s] , ret = %d", handle, attribute_name, ret);