Skip to content
Snippets Groups Projects
Commit d4b451bd authored by Amarnath Hullur Subramanyam's avatar Amarnath Hullur Subramanyam Committed by Vineeta Srivastava
Browse files

wlan:Check priviledge permission before processing SET_OEM_DATA_REQ IOCTL


Kernel assumes all SET IOCTL commands are assigned with even
numbers. But in our WLAN driver, some SET IOCTLS are assigned with
odd numbers. This leads kernel fail to check, for some SET IOCTLs,
whether user has the right permission to do SET operation.
Hence, in driver, before processing SET_OEM_DATA_REQ IOCTLs, making
sure user task has right permission to process the command.

CRs-Fixed: 930549
Git-commit: 6feb2faf80a05940618aa2eef2b62e4e2e54f148
Bug: 25344453
Signed-off-by: default avatarAmarnath Hullur Subramanyam <amarnath@codeaurora.org>
parent 255dd931
Branches
Tags
No related merge requests found
...@@ -189,6 +189,12 @@ int iw_set_oem_data_req( ...@@ -189,6 +189,12 @@ int iw_set_oem_data_req(
hdd_adapter_t *pAdapter = (netdev_priv(dev)); hdd_adapter_t *pAdapter = (netdev_priv(dev));
hdd_wext_state_t *pwextBuf = WLAN_HDD_GET_WEXT_STATE_PTR(pAdapter); hdd_wext_state_t *pwextBuf = WLAN_HDD_GET_WEXT_STATE_PTR(pAdapter);
if (!capable(CAP_NET_ADMIN)) {
VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_ERROR,
FL("permission check failed"));
return -EPERM;
}
if ((WLAN_HDD_GET_CTX(pAdapter))->isLogpInProgress) if ((WLAN_HDD_GET_CTX(pAdapter))->isLogpInProgress)
{ {
VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_FATAL, VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_FATAL,
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment