Fix REVERSE_INULL 09/303209/2 accepted/tizen/unified/20231228.165650 accepted/tizen/unified/riscv/20240103.054640
authorJaechul Lee <jcsing.lee@samsung.com>
Thu, 21 Dec 2023 08:02:06 +0000 (17:02 +0900)
committerJaechul Lee <jcsing.lee@samsung.com>
Thu, 21 Dec 2023 08:08:46 +0000 (17:08 +0900)
[Version] 15.0.72
[Issue Type] Coverity

Change-Id: I93c16701edf55db3e3d8de8f2eb12aeb9bb3b2dd
Signed-off-by: Jaechul Lee <jcsing.lee@samsung.com>
packaging/pulseaudio-modules-tizen.spec
src/preprocessor/module-tizenaudio-preprocessor.c

index e93c4f6..3c9e605 100644 (file)
@@ -2,7 +2,7 @@
 
 Name:             pulseaudio-modules-tizen
 Summary:          Pulseaudio modules for Tizen
-Version:          15.0.71
+Version:          15.0.72
 Release:          0
 Group:            Multimedia/Audio
 License:          LGPL-2.1+
index f8527a0..c96ddd1 100644 (file)
@@ -413,6 +413,7 @@ static int preprocess(pa_source_output *o, pa_memchunk *chunk, pa_memchunk *ochu
     pa_assert(o->thread_info.processor_holder);
 
     holder = (pa_processor_holder *)o->thread_info.processor_holder;
+    pa_assert(holder);
 
     /* chunk must contain resampled sound pcm */
     pa_processor_holder_push_data(holder, chunk);
@@ -533,11 +534,10 @@ static int process_msg(
             pa_processor_reference *reference = NULL;
             bool bypass = !!offset;
 
+            pa_assert(holder);
+
             reference = pa_processor_holder_get_connected_processor_reference(holder);
-            if (!holder || !reference) {
-                pa_log_error("holder(%p), reference(%p) aren't set", holder, reference);
-                break;
-            }
+            pa_assert(reference);
 
             pa_processor_reference_set_bypass(reference, bypass);