diff options
| author | Mukul Sharma <mukul@qti.qualcomm.com> | 2015-10-27 23:42:45 +0530 |
|---|---|---|
| committer | Anjaneedevi Kapparapu <akappa@codeaurora.org> | 2015-10-28 16:03:59 +0530 |
| commit | 6feb2faf80a05940618aa2eef2b62e4e2e54f148 (patch) | |
| tree | 5ad24db1f975a1cc0b69bdb11938784bf05b9a8c | |
| parent | 8bd73c3452ab22ba9bdbaac5ab12de2ed25fcb9d (diff) | |
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.
Change-Id: Ida0133304b00627d01ef7f85f5b15ed9d404d443
CRs-Fixed: 930549
| -rw-r--r-- | CORE/HDD/src/wlan_hdd_oemdata.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/CORE/HDD/src/wlan_hdd_oemdata.c b/CORE/HDD/src/wlan_hdd_oemdata.c index 17e36891a995..1aef25740130 100644 --- a/CORE/HDD/src/wlan_hdd_oemdata.c +++ b/CORE/HDD/src/wlan_hdd_oemdata.c @@ -200,6 +200,12 @@ static int __iw_set_oem_data_req(struct net_device *dev, ENTER(); + 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) { VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_FATAL, |