diff --git a/core/hdd/src/wlan_hdd_btc_chain_mode.c b/core/hdd/src/wlan_hdd_btc_chain_mode.c index 3c30728fadb3..13ba5005fd8d 100644 --- a/core/hdd/src/wlan_hdd_btc_chain_mode.c +++ b/core/hdd/src/wlan_hdd_btc_chain_mode.c @@ -45,10 +45,8 @@ wlan_hdd_btc_chain_mode_handler(struct wlan_objmgr_vdev *vdev) } vdev_id = wlan_vdev_get_id(vdev); - if (wlan_hdd_validate_vdev_id(vdev_id)) { - hdd_err("Invalid vdev id: %d", vdev_id); + if (wlan_hdd_validate_vdev_id(vdev_id)) return QDF_STATUS_E_INVAL; - } psoc = wlan_vdev_get_psoc(vdev); if (!psoc) { diff --git a/core/hdd/src/wlan_hdd_cfg80211.c b/core/hdd/src/wlan_hdd_cfg80211.c index b0fbbea54ca1..ae4ef51c658a 100644 --- a/core/hdd/src/wlan_hdd_cfg80211.c +++ b/core/hdd/src/wlan_hdd_cfg80211.c @@ -22341,10 +22341,8 @@ __wlan_hdd_cfg80211_update_owe_info(struct wiphy *wiphy, return -EINVAL; } - if (wlan_hdd_validate_vdev_id(adapter->vdev_id)) { - hdd_err("invalid vdev id: %d", adapter->vdev_id); + if (wlan_hdd_validate_vdev_id(adapter->vdev_id)) return -EINVAL; - } hdd_debug("owe_status %d", owe_info->status); diff --git a/core/hdd/src/wlan_hdd_hostapd.c b/core/hdd/src/wlan_hdd_hostapd.c index 0bb008d442a8..1d70fbd987ae 100644 --- a/core/hdd/src/wlan_hdd_hostapd.c +++ b/core/hdd/src/wlan_hdd_hostapd.c @@ -5944,10 +5944,8 @@ static int __wlan_hdd_cfg80211_stop_ap(struct wiphy *wiphy, goto exit; } - if (wlan_hdd_validate_vdev_id(adapter->vdev_id)) { - hdd_err("vdev is invalid. Hence return"); + if (wlan_hdd_validate_vdev_id(adapter->vdev_id)) goto exit; - } qdf_mtrace(QDF_MODULE_ID_HDD, QDF_MODULE_ID_HDD, TRACE_CODE_HDD_CFG80211_STOP_AP, diff --git a/core/hdd/src/wlan_hdd_rssi_monitor.c b/core/hdd/src/wlan_hdd_rssi_monitor.c index 9d0bd3f2caa8..5fbc39745466 100644 --- a/core/hdd/src/wlan_hdd_rssi_monitor.c +++ b/core/hdd/src/wlan_hdd_rssi_monitor.c @@ -77,10 +77,8 @@ __wlan_hdd_cfg80211_monitor_rssi(struct wiphy *wiphy, hdd_enter_dev(dev); - if (wlan_hdd_validate_vdev_id(adapter->vdev_id)) { - hdd_err("invalid session id: %d", adapter->vdev_id); + if (wlan_hdd_validate_vdev_id(adapter->vdev_id)) return -EINVAL; - } ret = wlan_hdd_validate_context(hdd_ctx); if (ret)