diff options
Diffstat (limited to 'bcmdhd/wifi_hal/link_layer_stats.cpp')
-rw-r--r-- | bcmdhd/wifi_hal/link_layer_stats.cpp | 84 |
1 files changed, 15 insertions, 69 deletions
diff --git a/bcmdhd/wifi_hal/link_layer_stats.cpp b/bcmdhd/wifi_hal/link_layer_stats.cpp index 3a922db..05ab438 100644 --- a/bcmdhd/wifi_hal/link_layer_stats.cpp +++ b/bcmdhd/wifi_hal/link_layer_stats.cpp @@ -39,7 +39,7 @@ #include <utils/Log.h> -#include "wifi_hal.h" +#include <hardware_legacy/wifi_hal.h> #include "common.h" #include "cpp_bindings.h" @@ -79,7 +79,7 @@ public: { } virtual int create() { - // ALOGI("Creating message to get link statistics; iface = %d", mIfaceInfo->id); + ALOGI("Creating message to get link statistics; iface = %d", mIfaceInfo->id); int ret = mMsg.create(GOOGLE_OUI, LSTATS_SUBCMD_GET_INFO); if (ret < 0) { @@ -99,7 +99,7 @@ protected: uint32_t total_size = 0, per_radio_size = 0, data_len = 0, rem_len = 0; int num_radios = 0, id = 0, subcmd = 0, len = 0; - // ALOGI("In GetLinkStatsCommand::handleResponse"); + ALOGI("In GetLinkStatsCommand::handleResponse"); if (reply.get_cmd() != NL80211_CMD_VENDOR) { ALOGD("Ignoring reply with cmd = %d", reply.get_cmd()); @@ -126,10 +126,16 @@ protected: } else { ALOGW("Ignoring invalid attribute type = %d, size = %d\n", it.get_type(), it.get_len()); + return NL_SKIP; } } if (num_radios) { + if (!data || !data_len) { + ALOGE("%s: null data\n", __func__); + return NL_SKIP; + } + rem_len = MAX_CMD_RESP_BUF_LEN; radioStatsBuf = (u8 *)malloc(MAX_CMD_RESP_BUF_LEN); if (!radioStatsBuf) { @@ -139,11 +145,6 @@ protected: memset(radioStatsBuf, 0, MAX_CMD_RESP_BUF_LEN); output = radioStatsBuf; - if (!data || !data_len) { - ALOGE("%s: null data\n", __func__); - return NL_SKIP; - } - data_ptr = (u8*)data; for (int i = 0; i < num_radios; i++) { rem_len -= per_radio_size; @@ -160,12 +161,18 @@ protected: convertToExternalRadioStatStructure((wifi_radio_stat*)data_ptr, &per_radio_size); if (!radio_stat_ptr || !per_radio_size) { + if (radio_stat_ptr) { + free(radio_stat_ptr); + radio_stat_ptr = NULL; + } ALOGE("No data for radio %d\n", i); continue; } memcpy(output, radio_stat_ptr, per_radio_size); output += per_radio_size; total_size += per_radio_size; + free(radio_stat_ptr); + radio_stat_ptr = NULL; } iface_stat = ((u8*)data + total_size); @@ -176,38 +183,8 @@ protected: } (*mHandler.on_link_stats_results)(id, (wifi_iface_stat *)iface_stat, num_radios, (wifi_radio_stat *)radioStatsBuf); - } else { - /* To be deprecated, adding it to keep it backward compatible */ - ALOGD("GetLinkStatCommand: zero radio case\n"); - data = reply.get_vendor_data(); - if (!data) { - ALOGE("Invalid vendor data received\n"); - return NL_SKIP; - } - - num_radios = 1; - data = reply.get_vendor_data(); - len = reply.get_vendor_data_len(); - if (!data || !len) { - ALOGE("Invalid vendor data received\n"); - return NL_SKIP; - } - radio_stat_ptr = - convertToExternalRadioStatStructureLegacy((wifi_radio_stat_internal *)data); - if (!radio_stat_ptr) { - ALOGE("Invalid stats pointer received\n"); - return NL_SKIP; - } - wifi_iface_stat *iface_stat = - (wifi_iface_stat *)((char *)&((wifi_radio_stat_internal *)data)->channels - + radio_stat_ptr->num_channels * sizeof(wifi_channel_stat)); - (*mHandler.on_link_stats_results)(id, iface_stat, num_radios, radio_stat_ptr); } exit: - if (radio_stat_ptr) { - free(radio_stat_ptr); - radio_stat_ptr = NULL; - } if (radioStatsBuf) { free(radioStatsBuf); radioStatsBuf = NULL; @@ -247,37 +224,6 @@ private: } return external_stat_ptr; } - - wifi_radio_stat *convertToExternalRadioStatStructureLegacy(wifi_radio_stat_internal *internal_stat_ptr) { - wifi_radio_stat *external_stat_ptr = NULL; - if (!internal_stat_ptr) { - ALOGE("Sta_ptr is null\n"); - } else { - uint32_t channel_size = internal_stat_ptr->num_channels * sizeof(wifi_channel_stat); - uint32_t total_size = sizeof(wifi_radio_stat) + channel_size; - external_stat_ptr = (wifi_radio_stat *)malloc(total_size); - if (external_stat_ptr) { - external_stat_ptr->radio = internal_stat_ptr->radio; - external_stat_ptr->on_time = internal_stat_ptr->on_time; - external_stat_ptr->tx_time = internal_stat_ptr->tx_time; - external_stat_ptr->rx_time = internal_stat_ptr->rx_time; - external_stat_ptr->tx_time_per_levels = NULL; - external_stat_ptr->num_tx_levels = 0; - external_stat_ptr->on_time_scan = internal_stat_ptr->on_time_scan; - external_stat_ptr->on_time_nbd = internal_stat_ptr->on_time_nbd; - external_stat_ptr->on_time_gscan = internal_stat_ptr->on_time_gscan; - external_stat_ptr->on_time_roam_scan = internal_stat_ptr->on_time_roam_scan; - external_stat_ptr->on_time_pno_scan = internal_stat_ptr->on_time_pno_scan; - external_stat_ptr->on_time_hs20 = internal_stat_ptr->on_time_hs20; - external_stat_ptr->num_channels = internal_stat_ptr->num_channels; - if (internal_stat_ptr->num_channels) { - memcpy(&(external_stat_ptr->channels), &(internal_stat_ptr->channels), - channel_size); - } - } - } - return external_stat_ptr; - } }; wifi_error wifi_get_link_stats(wifi_request_id id, |