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 89d8757..d24a56e 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 bb86007..187ebf1 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 746eb74..618c803 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 f8601ac..9741901 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 948e8f0..41bc90e 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;