diff options
| author | Bhargav Shah <c_bhargv@qti.qualcomm.com> | 2015-06-18 16:39:26 +0530 |
|---|---|---|
| committer | Anjaneedevi Kapparapu <akappa@codeaurora.org> | 2015-07-01 16:54:41 +0530 |
| commit | 679f1882f50024d0c76f58c6257fb22148fda3d9 (patch) | |
| tree | c33c4a03c32eba7d32b4a59e54f5379204bd5250 | |
| parent | ca31034efbc64868e83f3590f14876f9d9d23caf (diff) | |
qcacld-2.0: Check the return status of sscanf for TDLSSCAN
Currently in TDLSSCAN ioctl, return value of sscanf is not checked.
Update code to check the return value of sscanf
Change-Id: I94a62186f168d948d8e1cee0d018ca7fe7bbd48f
CRs-Fixed: 856380
| -rwxr-xr-x | CORE/HDD/src/wlan_hdd_main.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/CORE/HDD/src/wlan_hdd_main.c b/CORE/HDD/src/wlan_hdd_main.c index 59e2deaa2d02..5e39560c5a04 100755 --- a/CORE/HDD/src/wlan_hdd_main.c +++ b/CORE/HDD/src/wlan_hdd_main.c @@ -6007,7 +6007,13 @@ static int hdd_driver_command(hdd_adapter_t *pAdapter, int set_value; /* Move pointer to point the string */ value += 8; - sscanf(value, "%d", &set_value); + ret = sscanf(value, "%d", &set_value); + if (ret != 1) { + VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_ERROR, + "Wrong value is given for tdls_scan_type"); + ret = -EINVAL; + goto exit; + } hddLog(LOG1, FL("Tdls scan type val: %d"), set_value); ret = hdd_set_tdls_scan_type(pHddCtx, set_value); |