tizen-sink/source: fix svace defects

This commit fixes warnings as follows

NO_CAST.INTEGER_OVERFLOW
SEC_INVALID_ASSIGNMENT_WITH_UNSIGNED_VALUES

[Version] 15.0.32
[Issue Type] Svace

Change-Id: I901662abb6efdc5171d58d4042e5e4140a69c1d0
Signed-off-by: Jaechul Lee <jcsing.lee@samsung.com>

diff --git a/packaging/pulseaudio-modules-tizen.spec b/packaging/pulseaudio-modules-tizen.spec
index 89d875778fe5714f9971e561a11691d69aef9af7..d24a56ef97978b238d800da8b021b3ede7796ec6 100644 (file)
--- a/packaging/pulseaudio-modules-tizen.spec
+++ b/packaging/pulseaudio-modules-tizen.spec
@@ -2,7 +2,7 @@
 
 Name:             pulseaudio-modules-tizen
 Summary:          Pulseaudio modules for Tizen
-Version:          15.0.31
+Version:          15.0.32
 Release:          0
 Group:            Multimedia/Audio
 License:          LGPL-2.1+

diff --git a/src/module-tizenaudio-sink.c b/src/module-tizenaudio-sink.c
index bb86007e710d08a34c125595cc9c3b7332509fff..187ebf11e23256364627357c0dee1bb79fc59ee6 100644 (file)
--- a/src/module-tizenaudio-sink.c
+++ b/src/module-tizenaudio-sink.c
@@ -635,7 +635,7 @@ int pa__init(pa_module*m) {
     pa_proplist_sets(data.proplist, PA_PROP_DEVICE_API, "tizen");
 
     frame_size = pa_frame_size(&ss);
-    buffer_size = u->frag_size * u->nfrags;
+    buffer_size = (size_t)(u->frag_size * u->nfrags);
     buffer_frames = buffer_size / frame_size;
     period_frames = u->frag_size / frame_size;
     pa_proplist_setf(data.proplist, PA_PROP_DEVICE_BUFFERING_BUFFER_SIZE, "%zu", buffer_frames * frame_size);

diff --git a/src/module-tizenaudio-source.c b/src/module-tizenaudio-source.c
index 746eb74960d4899652804b6096af61b45aa8a683..618c8038ac8d169ec3bfe7edf484bfef03135849 100644 (file)
--- a/src/module-tizenaudio-source.c
+++ b/src/module-tizenaudio-source.c
@@ -536,7 +536,7 @@ int pa__init(pa_module*m) {
     pa_proplist_sets(data.proplist, PA_PROP_DEVICE_API, "tizen");
 
     frame_size = pa_frame_size(&ss);
-    buffer_size = u->frag_size * u->nfrags;
+    buffer_size = (size_t)(u->frag_size * u->nfrags);
     buffer_frames = buffer_size / frame_size;
     period_frames = u->frag_size / frame_size;
     pa_proplist_setf(data.proplist, PA_PROP_DEVICE_BUFFERING_BUFFER_SIZE, "%zu", buffer_frames * frame_size);

diff --git a/src/tizenaudio-sink2.c b/src/tizenaudio-sink2.c
index f8601acc41d999894ea2de04fdf8b899211bf594..97419017ed6f2493f5e82cb42d374c175a1389aa 100644 (file)
--- a/src/tizenaudio-sink2.c
+++ b/src/tizenaudio-sink2.c
@@ -459,7 +459,7 @@ pa_sink *pa_tizenaudio_sink2_new(pa_module *m,
 
     param.direction = DIRECTION_OUT;
     param.ss = ss;
-    param.period_size = u->frag_size / pa_frame_size(&ss);
+    param.period_size = (uint32_t)(u->frag_size / pa_frame_size(&ss));
     param.periods = u->nfrags;
 
     pa_log_info("Try to open device with rate(%d), channels(%d), format(%s), period_size(%u), periods(%u)",
@@ -504,7 +504,7 @@ pa_sink *pa_tizenaudio_sink2_new(pa_module *m,
         }
 
         frame_size = pa_frame_size(&param.ss);
-        u->frag_size = period_size * frame_size;
+        u->frag_size = (uint32_t)(period_size * frame_size);
          /* TODO: can't trust periods value from hal */
         u->nfrags = param.periods;
 

diff --git a/src/tizenaudio-source2.c b/src/tizenaudio-source2.c
index 948e8f0cb66b12aaba09a130fb4789e2363dd6a7..41bc90e80dd4053754206bd16b9bd1773b65768c 100644 (file)
--- a/src/tizenaudio-source2.c
+++ b/src/tizenaudio-source2.c
@@ -461,7 +461,7 @@ pa_source *pa_tizenaudio_source2_new(pa_module *m,
 
     param.direction = DIRECTION_IN;
     param.ss = ss;
-    param.period_size = u->frag_size / pa_frame_size(&ss);
+    param.period_size = (uint32_t)(u->frag_size / pa_frame_size(&ss));
     param.periods = u->nfrags;
 
     pa_log_info("Try to open device with rate(%d), channels(%d), format(%s), period_size(%u), periods(%u)",
@@ -506,7 +506,7 @@ pa_source *pa_tizenaudio_source2_new(pa_module *m,
         }
 
         frame_size = pa_frame_size(&param.ss);
-        u->frag_size = period_size * frame_size;
+        u->frag_size = (uint32_t)(period_size * frame_size);
          /* TODO: can't trust periods value from hal */
         u->nfrags = param.periods;