From: Jaechul Lee <jcsing.lee@samsung.com>
Date: Thu, 21 Dec 2023 08:02:06 +0000 (+0900)
Subject: Fix REVERSE_INULL
X-Git-Tag: accepted/tizen/unified/20231228.165650^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b50cc5866fb7e3567340d19a22d491f66aed6abd;p=platform%2Fcore%2Fmultimedia%2Fpulseaudio-modules-tizen.git

Fix REVERSE_INULL

[Version] 15.0.72
[Issue Type] Coverity

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

diff --git a/packaging/pulseaudio-modules-tizen.spec b/packaging/pulseaudio-modules-tizen.spec
index e93c4f6..3c9e605 100644
--- 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.71
+Version:          15.0.72
 Release:          0
 Group:            Multimedia/Audio
 License:          LGPL-2.1+
diff --git a/src/preprocessor/module-tizenaudio-preprocessor.c b/src/preprocessor/module-tizenaudio-preprocessor.c
index f8527a0..c96ddd1 100644
--- a/src/preprocessor/module-tizenaudio-preprocessor.c
+++ b/src/preprocessor/module-tizenaudio-preprocessor.c
@@ -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);