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

Subject: media: pci: cx18: if cx == NULL, then don't use it.
Author:  Hans Verkuil <hverkuil-ci...@xs4all.nl>
Date:    Fri Oct 13 09:43:55 2023 +0200

If the cx NULL pointer check is true, then it reports
an error with CX18_ERR, but that uses cx. Replace it with
pr_err.

Reported by smatch:

drivers/media/pci/cx18/cx18-mailbox.c:834 cx18_vapi() error: we previously 
assumed 'cx' could be null (see line 833)

Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/pci/cx18/cx18-mailbox.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

---

diff --git a/drivers/media/pci/cx18/cx18-mailbox.c 
b/drivers/media/pci/cx18/cx18-mailbox.c
index 3b283f3c6726..a6457c23d18c 100644
--- a/drivers/media/pci/cx18/cx18-mailbox.c
+++ b/drivers/media/pci/cx18/cx18-mailbox.c
@@ -831,7 +831,7 @@ int cx18_vapi(struct cx18 *cx, u32 cmd, int args, ...)
        int i;
 
        if (cx == NULL) {
-               CX18_ERR("cx == NULL (cmd=%x)\n", cmd);
+               pr_err("cx == NULL (cmd=%x)\n", cmd);
                return 0;
        }
        if (args > MAX_MB_ARGUMENTS) {

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to