msm: pcie: Add proper check before accessing variables
Add proper check before accessing variables. Change-Id: Idb4aec6e276d7c82a4311be733fcb13b875ae6c8 Signed-off-by: Rama Krishna Phani A <rphani@codeaurora.org>
This commit is contained in:
parent
a337c17ceb
commit
51df7d76f7
1 changed files with 15 additions and 1 deletions
|
@ -1,4 +1,4 @@
|
|||
/* Copyright (c) 2014-2017, The Linux Foundation. All rights reserved.
|
||||
/* Copyright (c) 2014-2018, The Linux Foundation. All rights reserved.
|
||||
*
|
||||
* This program is free software; you can redistribute it and/or modify
|
||||
* it under the terms of the GNU General Public License version 2 and
|
||||
|
@ -2421,6 +2421,13 @@ static void msm_pcie_sel_debug_testcase(struct msm_pcie_dev_t *dev,
|
|||
break;
|
||||
}
|
||||
|
||||
if (((base_sel - 1) >= MSM_PCIE_MAX_RES) ||
|
||||
(!dev->res[base_sel - 1].resource)) {
|
||||
PCIE_DBG_FS(dev, "PCIe: RC%d Resource does not exist\n",
|
||||
dev->rc_idx);
|
||||
break;
|
||||
}
|
||||
|
||||
PCIE_DBG_FS(dev,
|
||||
"base: %s: 0x%p\nwr_offset: 0x%x\nwr_mask: 0x%x\nwr_value: 0x%x\n",
|
||||
dev->res[base_sel - 1].name,
|
||||
|
@ -2440,6 +2447,13 @@ static void msm_pcie_sel_debug_testcase(struct msm_pcie_dev_t *dev,
|
|||
|
||||
break;
|
||||
case 13: /* dump all registers of base_sel */
|
||||
if (((base_sel - 1) >= MSM_PCIE_MAX_RES) ||
|
||||
(!dev->res[base_sel - 1].resource)) {
|
||||
PCIE_DBG_FS(dev, "PCIe: RC%d Resource does not exist\n",
|
||||
dev->rc_idx);
|
||||
break;
|
||||
}
|
||||
|
||||
if (!base_sel) {
|
||||
PCIE_DBG_FS(dev, "Invalid base_sel: 0x%x\n", base_sel);
|
||||
break;
|
||||
|
|
Loading…
Add table
Reference in a new issue