media: pwc-ctl: pChoose can't be NULL
authorMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Mon, 25 Mar 2019 20:20:47 +0000 (16:20 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Mon, 25 Mar 2019 22:02:27 +0000 (18:02 -0400)
The way the code works, compression will be a valid value (less or equal to 3)
on both set_video_mode_foo() calls at the beginning of the while() loop.

So, the value for pChoose can't be NULL.

Solves those smatch warnings:

drivers/media/usb/pwc/pwc-ctrl.c: drivers/media/usb/pwc/pwc-ctrl.c:252 set_video_mode_Timon() warn: variable dereferenced before check 'pChoose' (see line 248)
drivers/media/usb/pwc/pwc-ctrl.c: drivers/media/usb/pwc/pwc-ctrl.c:302 set_video_mode_Kiara() warn: variable dereferenced before check 'pChoose' (see line 298)

and simplifies the code a little bit.

Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/usb/pwc/pwc-ctrl.c

index 655cef39eb3dcdc40ac66f5274824328cfabf6b0..b681a184ef87c729a0204932602a7764a9bde32c 100644 (file)
@@ -242,14 +242,14 @@ static int set_video_mode_Timon(struct pwc_device *pdev, int size, int pixfmt,
        fps = (frames / 5) - 1;
 
        /* Find a supported framerate with progressively higher compression */
-       pChoose = NULL;
-       while (*compression <= 3) {
+       do {
                pChoose = &Timon_table[size][fps][*compression];
                if (pChoose->alternate != 0)
                        break;
                (*compression)++;
-       }
-       if (pChoose == NULL || pChoose->alternate == 0)
+       } while (*compression <= 3);
+
+       if (pChoose->alternate == 0)
                return -ENOENT; /* Not supported. */
 
        if (send_to_cam)
@@ -279,7 +279,7 @@ static int set_video_mode_Timon(struct pwc_device *pdev, int size, int pixfmt,
 static int set_video_mode_Kiara(struct pwc_device *pdev, int size, int pixfmt,
                                int frames, int *compression, int send_to_cam)
 {
-       const struct Kiara_table_entry *pChoose = NULL;
+       const struct Kiara_table_entry *pChoose;
        int fps, ret = 0;
 
        if (size >= PSZ_MAX || *compression < 0 || *compression > 3)
@@ -293,13 +293,14 @@ static int set_video_mode_Kiara(struct pwc_device *pdev, int size, int pixfmt,
        fps = (frames / 5) - 1;
 
        /* Find a supported framerate with progressively higher compression */
-       while (*compression <= 3) {
+       do {
                pChoose = &Kiara_table[size][fps][*compression];
                if (pChoose->alternate != 0)
                        break;
                (*compression)++;
-       }
-       if (pChoose == NULL || pChoose->alternate == 0)
+       } while (*compression <= 3);
+
+       if (pChoose->alternate == 0)
                return -ENOENT; /* Not supported. */
 
        /* Firmware bug: video endpoint is 5, but commands are sent to endpoint 4 */