This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: firewire: firedtv-avc.c: replace BUG with proper, error return
Author:  Hans Verkuil <[email protected]>
Date:    Fri Aug 4 12:59:41 2023 +0200

This resolves this smatch error:

drivers/media/firewire/firedtv-avc.c:602 avc_tuner_dsd() error: uninitialized 
symbol 'pos'.

Signed-off-by: Hans Verkuil <[email protected]>

 drivers/media/firewire/firedtv-avc.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

---

diff --git a/drivers/media/firewire/firedtv-avc.c 
b/drivers/media/firewire/firedtv-avc.c
index 71991f8638e6..a36c28412170 100644
--- a/drivers/media/firewire/firedtv-avc.c
+++ b/drivers/media/firewire/firedtv-avc.c
@@ -597,7 +597,8 @@ int avc_tuner_dsd(struct firedtv *fdtv,
        case FIREDTV_DVB_C: pos = avc_tuner_dsd_dvb_c(fdtv, p); break;
        case FIREDTV_DVB_T: pos = avc_tuner_dsd_dvb_t(fdtv, p); break;
        default:
-               BUG();
+               ret = -EIO;
+               goto unlock;
        }
        pad_operands(c, pos);
 
@@ -612,6 +613,7 @@ int avc_tuner_dsd(struct firedtv *fdtv,
        if (status)
                *status = r->operand[2];
 #endif
+unlock:
        mutex_unlock(&fdtv->avc_mutex);
 
        if (ret == 0)

_______________________________________________
linuxtv-commits mailing list
[email protected]
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to