media: tegra: camera: sanity-check ioctl parameter
Greg Hackmann [Fri, 19 Feb 2016 21:33:31 +0000 (13:33 -0800)]
Several places in the camera stack can hit integer overflows or cause
bad allocations if userspace passes in a bogus sizeofvalue parameter.
Protect against this by using appropriately-sized integer types, adding
range checks, replacing array-allocation calls with kcalloc(), and
checking for allocations returning ZERO_SIZE_PTR.

For one specific ioctl (PCLLK_IOCTL_UPDATE) sizeofvalue = 0 is fine,
since when that happens the subdrivers won't actually touch the returned
allocation.  In fact the existing userspace camera driver makes calls
like these and expects them to succeed!  Handle this special case by
adding a __camera_get_params variant that optionally treats zero-sized
inputs as valid.

(back ported from Nexus N9 project)

Bug 1832830

Change-Id: Ie3250d8a4b814de5820fa0190b4cbd1af3ca4b3f
Reported-by: Jianqiang Zhao <zhaojianqiang1@gmail.com>
Signed-off-by: Greg Hackmann <ghackmann@google.com>
Reviewed-on: http://git-master/r/1271367
Reviewed-by: Automatic_Commit_Validation_User
GVS: Gerrit_Virtual_Submit
Reviewed-by: Frank Chen <frankc@nvidia.com>
Tested-by: Frank Chen <frankc@nvidia.com>
Reviewed-by: Jihoon Bang <jbang@nvidia.com>
Reviewed-by: Winnie Hsu <whsu@nvidia.com>

drivers/media/platform/tegra/cam_dev/imx135.c
drivers/media/platform/tegra/cam_dev/of_camera.c
drivers/media/platform/tegra/cam_dev/virtual.c
drivers/media/platform/tegra/camera.c
include/media/camera.h

index eaa0856..7dd3e9b 100644 (file)
@@ -4,7 +4,7 @@
  * the virtual PCL driver to handle some special features (hardware resources,
  * sequences, etc.).
  *
- * Copyright (c) 2013, NVIDIA CORPORATION.  All rights reserved.
+ * Copyright (c) 2013-2016, NVIDIA CORPORATION.  All rights reserved.
  *
  * This program is free software; you can redistribute it and/or modify it
  * under the terms and conditions of the GNU General Public License,
@@ -42,11 +42,11 @@ struct imx135_info {
 };
 
 static int imx135_update(
-       struct camera_device *cdev, struct cam_update *upd, int num)
+       struct camera_device *cdev, struct cam_update *upd, u32 num)
 {
        /* struct imx135_info *info = dev_get_drvdata(cdev->dev); */
        int err = 0;
-       int idx;
+       u32 idx;
 
        dev_dbg(cdev->dev, "%s %d\n", __func__, num);
        mutex_lock(&cdev->mutex);
index 460ca1a..f293b9a 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * debugfs.c
  *
- * Copyright (c) 2013-2014, NVIDIA CORPORATION.  All rights reserved.
+ * Copyright (c) 2013-2016, NVIDIA CORPORATION.  All rights reserved.
 
  * This program is free software; you can redistribute it and/or modify it
  * under the terms and conditions of the GNU General Public License,
@@ -222,7 +222,7 @@ int of_camera_get_property(struct camera_info *cam, unsigned long arg)
        }
 
        /* sanity check */
-       if (!param.sizeofvalue) {
+       if (!param.sizeofvalue || param.sizeofvalue > INT_MAX) {
                dev_err(cam->dev, "%s invalid property name length %d\n",
                        __func__, param.sizeofvalue);
                return -EBADF;
index f08ad15..d25becc 100644 (file)
@@ -41,10 +41,10 @@ struct chip_config {
 };
 
 static int virtual_update(
-       struct camera_device *cdev, struct cam_update *upd, int num)
+       struct camera_device *cdev, struct cam_update *upd, u32 num)
 {
        int err = 0;
-       int idx;
+       u32 idx;
 
        dev_dbg(cdev->dev, "%s %d\n", __func__, num);
        mutex_lock(&cdev->mutex);
index 2542f42..f54a3c0 100644 (file)
@@ -1,7 +1,7 @@
 /*
  * camera.c - generic camera device driver
  *
- * Copyright (c) 2013-2014, NVIDIA CORPORATION.  All rights reserved.
+ * Copyright (c) 2013-2016, NVIDIA CORPORATION.  All rights reserved.
  *
  * Contributors:
  *     Charlie Huang <chahuang@nvidia.com>
@@ -131,12 +131,12 @@ int camera_copy_user_params(unsigned long arg, struct nvc_param *prm)
 }
 #endif
 
-int camera_get_params(
+int __camera_get_params(
        struct camera_info *cam, unsigned long arg, int u_size,
-       struct nvc_param *prm, void **data)
+       struct nvc_param *prm, void **data, bool zero_size_ok)
 {
        void *buf;
-       unsigned size;
+       size_t size;
 
 #ifdef CONFIG_COMPAT
        memset(prm, 0, sizeof(*prm));
@@ -156,9 +156,14 @@ int camera_get_params(
        if (!data)
                return 0;
 
+       if (zero_size_ok && prm->sizeofvalue == 0) {
+               *data = ZERO_SIZE_PTR;
+               return 0;
+       }
+
        size = prm->sizeofvalue * u_size;
-       buf = kzalloc(size, GFP_KERNEL);
-       if (!buf) {
+       buf = kcalloc(prm->sizeofvalue, u_size, GFP_KERNEL);
+       if (ZERO_OR_NULL_PTR(buf)) {
                dev_err(cam->dev, "%s allocate memory failed!\n", __func__);
                return -ENOMEM;
        }
@@ -231,7 +236,7 @@ static int camera_seq_wr(struct camera_info *cam, unsigned long arg)
        }
 
        p_i2c_table = devm_kzalloc(cdev->dev, params.sizeofvalue, GFP_KERNEL);
-       if (p_i2c_table == NULL) {
+       if (ZERO_OR_NULL_PTR(p_i2c_table)) {
                dev_err(cam->dev, "%s devm_kzalloc err line %d\n",
                        __func__, __LINE__);
                return -ENOMEM;
@@ -586,7 +591,8 @@ static int camera_update(struct camera_info *cam, unsigned long arg)
                return err;
        }
 
-       err = camera_get_params(cam, arg, sizeof(*upd), &param, (void **)&upd);
+       err = __camera_get_params(cam, arg, sizeof(*upd), &param, (void **)&upd,
+                       true);
        if (err)
                return err;
 
index 5cbac56..7528b9a 100644 (file)
@@ -337,7 +337,7 @@ struct camera_chip {
        int     (*power_off)(struct camera_device *cdev);
        int     (*shutdown)(struct camera_device *cdev);
        int     (*update)(struct camera_device *cdev,
-                       struct cam_update *upd, int num);
+                       struct cam_update *upd, u32 num);
 };
 
 struct camera_sync_dev {
@@ -386,8 +386,14 @@ struct camera_platform_info {
 };
 
 /* common functions */
-int camera_get_params(
-       struct camera_info *, unsigned long, int, struct nvc_param *, void **);
+int __camera_get_params(
+       struct camera_info *, unsigned long, int, struct nvc_param *, void **,
+       bool);
+static inline int camera_get_params(struct camera_info *cam, unsigned long arg,
+               int u_size, struct nvc_param *prm, void **data)
+{
+       return __camera_get_params(cam, arg, u_size, prm, data, false);
+}
 int camera_copy_user_params(unsigned long, struct nvc_param *);
 
 int virtual_device_add(struct device *, unsigned long);