staging: rtl8723au: collect_bss_info23a(): Eliminate smatch warning
authorJes Sorensen <Jes.Sorensen@redhat.com>
Sat, 26 Apr 2014 16:54:51 +0000 (18:54 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 26 Apr 2014 17:16:12 +0000 (10:16 -0700)
Make length an unsigned int. This should avoid Smatch warning that
'length' can go negative.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723au/core/rtw_mlme_ext.c

index 21d112093fe1319db1f2b3cbd9c802c40b2642c2..e49170b3bb7a5d0b6910858010421e0dec86dfbf 100644 (file)
@@ -4469,18 +4469,18 @@ u8 collect_bss_info23a(struct rtw_adapter *padapter,
                       struct recv_frame *precv_frame,
                       struct wlan_bssid_ex *bssid)
 {
-       int i, length;
+       int i;
        const u8 *p;
        struct sk_buff *skb = precv_frame->pkt;
        struct ieee80211_mgmt *mgmt = (struct ieee80211_mgmt *) skb->data;
-       int packet_len = skb->len;
+       unsigned int length;
        u8 ie_offset;
        struct registry_priv *pregistrypriv = &padapter->registrypriv;
        struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
        struct mlme_ext_info *pmlmeinfo = &pmlmeext->mlmext_info;
        u16 capab_info;
 
-       length = packet_len - sizeof(struct ieee80211_hdr_3addr);
+       length = skb->len - sizeof(struct ieee80211_hdr_3addr);
 
        if (length > MAX_IE_SZ) {
                /* DBG_8723A("IE too long for survey event\n"); */