media: dvb_ca_en50221: sanity check slot number from userspace
authorColin Ian King <colin.king@canonical.com>
Wed, 20 Sep 2017 22:19:59 +0000 (18:19 -0400)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Thu, 14 Dec 2017 14:51:17 +0000 (09:51 -0500)
Currently a user can pass in an unsanitized slot number which
will lead to and out of range index into ca->slot_info. Fix this
by checking that the slot number is no more than the allowed
maximum number of slots. Seems that this bug has been in the driver
forever.

Detected by CoverityScan, CID#139381 ("Untrusted pointer read")

Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Jasmin Jessich <jasmin@anw.at>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/dvb-core/dvb_ca_en50221.c

index d48b61eb01f4cdc7f11913208a6beef0136a7bff..a3b2754e71247af038bbdd36b7dd585bd4ad10ab 100644 (file)
@@ -1473,6 +1473,9 @@ static ssize_t dvb_ca_en50221_io_write(struct file *file,
                return -EFAULT;
        buf += 2;
        count -= 2;
+
+       if (slot >= ca->slot_count)
+               return -EINVAL;
        sl = &ca->slot_info[slot];
 
        /* check if the slot is actually running */