V4L/DVB (8773): cx18: Fix cx18_find_handle() and add error checking

cx18: Fix cx18_find_handle() and add error checking.  cx18_find_handle() did
not find a good task handle and would use the invalid task handle under common
conditions.  Added a define for the invalid task handle and added error checking
as well.

Signed-off-by: Andy Walls <awalls@radix.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>

1 files changed, 30 insertions, 13 deletions

diff --git a/drivers/media/video/cx18/cx18-fileops.c b/drivers/media/video/cx18/cx18-fileops.c
index eb0144f95562..5f9089907544 100644
--- a/drivers/media/video/cx18/cx18-fileops.c
+++ b/drivers/media/video/cx18/cx18-fileops.c
@@ -132,6 +132,7 @@ static void cx18_dualwatch(struct cx18 *cx)
         u16 new_stereo_mode;
         const u16 stereo_mask = 0x0300;
         const u16 dual = 0x0200;
+        u32 h;
 
         new_stereo_mode = cx->params.audio_properties & stereo_mask;
         memset(&vt, 0, sizeof(vt));
@@ -143,13 +144,21 @@ static void cx18_dualwatch(struct cx18 *cx)
         if (new_stereo_mode == cx->dualwatch_stereo_mode)
                 return;
 
-        new_bitmap = new_stereo_mode | (cx->params.audio_properties & ~stereo_mask);
+        new_bitmap = new_stereo_mode
+                        | (cx->params.audio_properties & ~stereo_mask);
 
-        CX18_DEBUG_INFO("dualwatch: change stereo flag from 0x%x to 0x%x. new audio_bitmask=0x%ux\n",
-                           cx->dualwatch_stereo_mode, new_stereo_mode, new_bitmap);
+        CX18_DEBUG_INFO("dualwatch: change stereo flag from 0x%x to 0x%x. "
+                        "new audio_bitmask=0x%ux\n",
+                        cx->dualwatch_stereo_mode, new_stereo_mode, new_bitmap);
 
-        if (cx18_vapi(cx, CX18_CPU_SET_AUDIO_PARAMETERS, 2,
-                                cx18_find_handle(cx), new_bitmap) == 0) {
+        h = cx18_find_handle(cx);
+        if (h == CX18_INVALID_TASK_HANDLE) {
+                CX18_DEBUG_INFO("dualwatch: can't find valid task handle\n");
+                return;
+        }
+
+        if (cx18_vapi(cx,
+                      CX18_CPU_SET_AUDIO_PARAMETERS, 2, h, new_bitmap) == 0) {
                 cx->dualwatch_stereo_mode = new_stereo_mode;
                 return;
         }
@@ -695,20 +704,28 @@ int cx18_v4l2_open(struct inode *inode, struct file *filp)
 
 void cx18_mute(struct cx18 *cx)
 {
-        if (atomic_read(&cx->ana_capturing))
-                cx18_vapi(cx, CX18_CPU_SET_AUDIO_MUTE, 2,
-                                cx18_find_handle(cx), 1);
+        u32 h;
+        if (atomic_read(&cx->ana_capturing)) {
+                h = cx18_find_handle(cx);
+                if (h != CX18_INVALID_TASK_HANDLE)
+                        cx18_vapi(cx, CX18_CPU_SET_AUDIO_MUTE, 2, h, 1);
+                else
+                        CX18_ERR("Can't find valid task handle for mute\n");
+        }
         CX18_DEBUG_INFO("Mute\n");
 }
 
 void cx18_unmute(struct cx18 *cx)
 {
+        u32 h;
         if (atomic_read(&cx->ana_capturing)) {
-                cx18_msleep_timeout(100, 0);
-                cx18_vapi(cx, CX18_CPU_SET_MISC_PARAMETERS, 2,
-                                cx18_find_handle(cx), 12);
-                cx18_vapi(cx, CX18_CPU_SET_AUDIO_MUTE, 2,
-                                cx18_find_handle(cx), 0);
+                h = cx18_find_handle(cx);
+                if (h != CX18_INVALID_TASK_HANDLE) {
+                        cx18_msleep_timeout(100, 0);
+                        cx18_vapi(cx, CX18_CPU_SET_MISC_PARAMETERS, 2, h, 12);
+                        cx18_vapi(cx, CX18_CPU_SET_AUDIO_MUTE, 2, h, 0);
+                } else
+                        CX18_ERR("Can't find valid task handle for unmute\n");
         }
         CX18_DEBUG_INFO("Unmute\n");
 }