summaryrefslogtreecommitdiffstats
path: root/drivers/infiniband
diff options
context:
space:
mode:
authorLiu Yuan <tailai.ly@taobao.com>2011-04-01 06:23:49 +0000
committerRoland Dreier <roland@purestorage.com>2011-05-24 10:06:25 -0700
commit52f81dbaf1378faf64c3ecea5129cebf826ef126 (patch)
treecd7be0e965a12f0da8ad42f08414df2b6beebf3c /drivers/infiniband
parentc3af0980ce01dfb8e946990c3ae4c3727b1e9f26 (diff)
downloadop-kernel-dev-52f81dbaf1378faf64c3ecea5129cebf826ef126.zip
op-kernel-dev-52f81dbaf1378faf64c3ecea5129cebf826ef126.tar.gz
RDMA/nes: Add a check for strict_strtoul()
It should check if strict_strtoul() succeeds before using 'wqm_quanta_value'. Signed-off-by: Liu Yuan <tailai.ly@taobao.com> [ Convert to kstrtoul() directly while we're here. - Roland ] Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'drivers/infiniband')
-rw-r--r--drivers/infiniband/hw/nes/nes.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/nes/nes.c b/drivers/infiniband/hw/nes/nes.c
index 13de119..2d668c6 100644
--- a/drivers/infiniband/hw/nes/nes.c
+++ b/drivers/infiniband/hw/nes/nes.c
@@ -1138,7 +1138,9 @@ static ssize_t nes_store_wqm_quanta(struct device_driver *ddp,
u32 i = 0;
struct nes_device *nesdev;
- strict_strtoul(buf, 0, &wqm_quanta_value);
+ if (kstrtoul(buf, 0, &wqm_quanta_value) < 0)
+ return -EINVAL;
+
list_for_each_entry(nesdev, &nes_dev_list, list) {
if (i == ee_flsh_adapter) {
nesdev->nesadapter->wqm_quanta = wqm_quanta_value;
OpenPOWER on IntegriCloud