diff options
| author | CNSS_WLAN Service <cnssbldsw@qualcomm.com> | 2021-04-02 06:56:28 -0700 |
|---|---|---|
| committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2021-04-02 06:56:28 -0700 |
| commit | 68d06000bf4028987dcbfc1fcdadc8174a2de6bc (patch) | |
| tree | 34281d2a64adb3aa15354b2e19212e3286eb3079 | |
| parent | 717f58316141d70b369cd6c974ff1a6e7c395f42 (diff) | |
| parent | e025a1ceadee69a7abda5a91face873eba191299 (diff) | |
Merge "wpa_supplicant_8_lib: Remove unnecessary initialization and malloc" into wlan-aosp.lnx.6.0
| -rw-r--r-- | qcwcn/wpa_supplicant_8_lib/driver_cmd_nl80211.c | 16 |
1 files changed, 5 insertions, 11 deletions
diff --git a/qcwcn/wpa_supplicant_8_lib/driver_cmd_nl80211.c b/qcwcn/wpa_supplicant_8_lib/driver_cmd_nl80211.c index a09a938..04b4ad1 100644 --- a/qcwcn/wpa_supplicant_8_lib/driver_cmd_nl80211.c +++ b/qcwcn/wpa_supplicant_8_lib/driver_cmd_nl80211.c @@ -529,24 +529,18 @@ parse_beacon_ies: } static int parse_get_feature_info(struct resp_info *info, struct nlattr *vendata, - int datalen) + int datalen) { struct nlattr *tb_vendor[NUM_QCA_WLAN_VENDOR_FEATURES + 1]; struct nlattr *attr; - char *result = NULL; nla_parse(tb_vendor, NUM_QCA_WLAN_VENDOR_FEATURES, vendata, datalen, NULL); attr = tb_vendor[QCA_WLAN_VENDOR_ATTR_FEATURE_FLAGS]; if (attr) { - int length = snprintf( NULL, 0, "%d", nla_get_u32(attr)); - result = (char *)malloc(length + 1); - if (result != NULL) { - memset(result, 0, length + 1); - snprintf(result, length + 1, "%d", nla_get_u32(attr)); - snprintf(info->reply_buf, info->reply_buf_len, - "%s", result); - wpa_printf(MSG_DEBUG, "%s: driver supported feature info = %s", __func__, result); - } + snprintf(info->reply_buf, info->reply_buf_len, "%u", + nla_get_u32(attr)); + wpa_printf(MSG_DEBUG, "%s: driver supported feature info = %s", + __func__, info->reply_buf); } else { snprintf(info->reply_buf, info->reply_buf_len, "FAIL"); return -1; |