From: Jes Sorensen <Jes.Sorensen@redhat.com>
To: Dan Carpenter <dan.carpenter@oracle.com>
Cc: Larry Finger <Larry.Finger@lwfinger.net>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-wireless@vger.kernel.org, devel@driverdev.osuosl.org,
kernel-janitors@vger.kernel.org
Subject: Re: [patch] staging: rtl8723au: incorrect use of ether_addr_copy()
Date: Wed, 08 Oct 2014 15:59:33 +0200 [thread overview]
Message-ID: <wrfj4mvezmpm.fsf@redhat.com> (raw)
In-Reply-To: <20141008104014.GA23096@mwanda> (Dan Carpenter's message of "Wed, 8 Oct 2014 13:40:14 +0300")
Dan Carpenter <dan.carpenter@oracle.com> writes:
> The return from myid() isn't aligned correctly for ether_addr_copy().
>
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Sorry, this makes no sense, just fix it properly!
drivers/staging/rtl8723au/include/rtw_eeprom.h:
struct eeprom_priv {
u8 bautoload_fail_flag;
u8 bloadfile_fail_flag;
u8 bloadmac_fail_flag;
/* u8 bempty; */
/* u8 sys_config; */
u8 mac_addr[6]; /* PermanentAddress */
/* u8 config0; */
Move mac_addr[6] to the top of the struct and be done with it.
NACK
Jes
> diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
> index 3eb77de..c8f7890 100644
> --- a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
> +++ b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
> @@ -2402,7 +2402,7 @@ void issue_beacon23a(struct rtw_adapter *padapter, int timeout_ms)
> mgmt->seq_ctrl = 0;
>
> ether_addr_copy(mgmt->da, bc_addr);
> - ether_addr_copy(mgmt->sa, myid(&padapter->eeprompriv));
> + memcpy(mgmt->sa, myid(&padapter->eeprompriv), ETH_ALEN);
> ether_addr_copy(mgmt->bssid, get_my_bssid23a(cur_network));
>
> /* timestamp will be inserted by hardware */
> @@ -2556,7 +2556,7 @@ static void issue_probersp(struct rtw_adapter *padapter, unsigned char *da,
> cpu_to_le16(IEEE80211_FTYPE_MGMT | IEEE80211_STYPE_PROBE_RESP);
>
> ether_addr_copy(mgmt->da, da);
> - ether_addr_copy(mgmt->sa, mac);
> + memcpy(mgmt->sa, mac, ETH_ALEN);
> ether_addr_copy(mgmt->bssid, bssid);
>
> mgmt->seq_ctrl = cpu_to_le16(IEEE80211_SN_TO_SEQ(pmlmeext->mgnt_seq));
> @@ -2718,7 +2718,7 @@ static int _issue_probereq(struct rtw_adapter *padapter,
> ether_addr_copy(pwlanhdr->addr3, bc_addr);
> }
>
> - ether_addr_copy(pwlanhdr->addr2, mac);
> + memcpy(pwlanhdr->addr2, mac, ETH_ALEN);
>
> pwlanhdr->seq_ctrl =
> cpu_to_le16(IEEE80211_SN_TO_SEQ(pmlmeext->mgnt_seq));
> @@ -2863,8 +2863,8 @@ static void issue_auth(struct rtw_adapter *padapter, struct sta_info *psta,
> #ifdef CONFIG_8723AU_AP_MODE
> unsigned short val16;
> ether_addr_copy(mgmt->da, psta->hwaddr);
> - ether_addr_copy(mgmt->sa, myid(&padapter->eeprompriv));
> - ether_addr_copy(mgmt->bssid, myid(&padapter->eeprompriv));
> + memcpy(mgmt->sa, myid(&padapter->eeprompriv), ETH_ALEN);
> + memcpy(mgmt->bssid, myid(&padapter->eeprompriv), ETH_ALEN);
>
> /* setting auth algo number */
> val16 = (u16)psta->authalg;
> @@ -2895,7 +2895,7 @@ static void issue_auth(struct rtw_adapter *padapter, struct sta_info *psta,
> struct ieee80211_mgmt *iv_mgmt;
>
> ether_addr_copy(mgmt->da, get_my_bssid23a(&pmlmeinfo->network));
> - ether_addr_copy(mgmt->sa, myid(&padapter->eeprompriv));
> + memcpy(mgmt->sa, myid(&padapter->eeprompriv), ETH_ALEN);
> ether_addr_copy(mgmt->bssid,
> get_my_bssid23a(&pmlmeinfo->network));
>
> @@ -3006,7 +3006,7 @@ static void issue_assocrsp(struct rtw_adapter *padapter, unsigned short status,
> mgmt->frame_control = cpu_to_le16(IEEE80211_FTYPE_MGMT | pkt_type);
>
> ether_addr_copy(mgmt->da, pstat->hwaddr);
> - ether_addr_copy(mgmt->sa, myid(&padapter->eeprompriv));
> + memcpy(mgmt->sa, myid(&padapter->eeprompriv), ETH_ALEN);
> ether_addr_copy(mgmt->bssid, get_my_bssid23a(&pmlmeinfo->network));
>
> mgmt->seq_ctrl = cpu_to_le16(IEEE80211_SN_TO_SEQ(pmlmeext->mgnt_seq));
> @@ -3129,7 +3129,7 @@ static void issue_assocreq(struct rtw_adapter *padapter)
> cpu_to_le16(IEEE80211_FTYPE_MGMT | IEEE80211_STYPE_ASSOC_REQ);
>
> ether_addr_copy(mgmt->da, get_my_bssid23a(&pmlmeinfo->network));
> - ether_addr_copy(mgmt->sa, myid(&padapter->eeprompriv));
> + memcpy(mgmt->sa, myid(&padapter->eeprompriv), ETH_ALEN);
> ether_addr_copy(mgmt->bssid, get_my_bssid23a(&pmlmeinfo->network));
>
> mgmt->seq_ctrl = cpu_to_le16(IEEE80211_SN_TO_SEQ(pmlmeext->mgnt_seq));
> @@ -3400,7 +3400,7 @@ static int _issue_nulldata23a(struct rtw_adapter *padapter, unsigned char *da,
> pwlanhdr->frame_control |= cpu_to_le16(IEEE80211_FCTL_PM);
>
> ether_addr_copy(pwlanhdr->addr1, da);
> - ether_addr_copy(pwlanhdr->addr2, myid(&padapter->eeprompriv));
> + memcpy(pwlanhdr->addr2, myid(&padapter->eeprompriv), ETH_ALEN);
> ether_addr_copy(pwlanhdr->addr3, get_my_bssid23a(&pmlmeinfo->network));
>
> pwlanhdr->seq_ctrl =
> @@ -3528,7 +3528,7 @@ static int _issue_qos_nulldata23a(struct rtw_adapter *padapter,
> pwlanhdr->qos_ctrl |= cpu_to_le16(IEEE80211_QOS_CTL_EOSP);
>
> ether_addr_copy(pwlanhdr->addr1, da);
> - ether_addr_copy(pwlanhdr->addr2, myid(&padapter->eeprompriv));
> + memcpy(pwlanhdr->addr2, myid(&padapter->eeprompriv), ETH_ALEN);
> ether_addr_copy(pwlanhdr->addr3, get_my_bssid23a(&pmlmeinfo->network));
>
> pwlanhdr->seq_ctrl =
> @@ -3633,7 +3633,7 @@ static int _issue_deauth(struct rtw_adapter *padapter, unsigned char *da,
> cpu_to_le16(IEEE80211_FTYPE_MGMT | IEEE80211_STYPE_DEAUTH);
>
> ether_addr_copy(mgmt->da, da);
> - ether_addr_copy(mgmt->sa, myid(&padapter->eeprompriv));
> + memcpy(mgmt->sa, myid(&padapter->eeprompriv), ETH_ALEN);
> ether_addr_copy(mgmt->bssid, get_my_bssid23a(&pmlmeinfo->network));
>
> mgmt->seq_ctrl = cpu_to_le16(IEEE80211_SN_TO_SEQ(pmlmeext->mgnt_seq));
> @@ -3737,7 +3737,7 @@ void issue_action_spct_ch_switch23a(struct rtw_adapter *padapter,
> cpu_to_le16(IEEE80211_FTYPE_MGMT | IEEE80211_STYPE_ACTION);
>
> ether_addr_copy(mgmt->da, ra); /* RA */
> - ether_addr_copy(mgmt->sa, myid(&padapter->eeprompriv)); /* TA */
> + memcpy(mgmt->sa, myid(&padapter->eeprompriv), ETH_ALEN); /* TA */
> ether_addr_copy(mgmt->bssid, ra); /* DA = RA */
>
> mgmt->seq_ctrl = cpu_to_le16(IEEE80211_SN_TO_SEQ(pmlmeext->mgnt_seq));
> @@ -3799,7 +3799,7 @@ void issue_action_BA23a(struct rtw_adapter *padapter,
> cpu_to_le16(IEEE80211_FTYPE_MGMT | IEEE80211_STYPE_ACTION);
>
> ether_addr_copy(mgmt->da, raddr);
> - ether_addr_copy(mgmt->sa, myid(&padapter->eeprompriv));
> + memcpy(mgmt->sa, myid(&padapter->eeprompriv), ETH_ALEN);
> ether_addr_copy(mgmt->bssid, get_my_bssid23a(&pmlmeinfo->network));
>
> mgmt->seq_ctrl = cpu_to_le16(IEEE80211_SN_TO_SEQ(pmlmeext->mgnt_seq));
> diff --git a/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c b/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c
> index 271c33d..976ec2c 100644
> --- a/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c
> +++ b/drivers/staging/rtl8723au/hal/rtl8723a_cmd.c
> @@ -219,7 +219,7 @@ ConstructBeacon(struct rtw_adapter *padapter, u8 *pframe, u32 *pLength)
> cpu_to_le16(IEEE80211_FTYPE_MGMT | IEEE80211_STYPE_BEACON);
>
> ether_addr_copy(mgmt->da, bc_addr);
> - ether_addr_copy(mgmt->sa, myid(&padapter->eeprompriv));
> + memcpy(mgmt->sa, myid(&padapter->eeprompriv), ETH_ALEN);
> ether_addr_copy(mgmt->bssid, get_my_bssid23a(cur_network));
>
> /* A Beacon frame shouldn't have fragment bits set */
> diff --git a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c b/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
> index bd6953a..cb81e64 100644
> --- a/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
> +++ b/drivers/staging/rtl8723au/os_dep/ioctl_cfg80211.c
> @@ -2414,7 +2414,7 @@ void rtw_cfg80211_indicate_sta_disassoc(struct rtw_adapter *padapter,
> mgmt.frame_control =
> cpu_to_le16(IEEE80211_FTYPE_MGMT | IEEE80211_STYPE_DEAUTH);
>
> - ether_addr_copy(mgmt.da, myid(&padapter->eeprompriv));
> + memcpy(mgmt.da, myid(&padapter->eeprompriv), ETH_ALEN);
> ether_addr_copy(mgmt.sa, da);
> ether_addr_copy(mgmt.bssid, get_my_bssid23a(&pmlmeinfo->network));
>
next prev parent reply other threads:[~2014-10-08 14:20 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-08 10:40 [patch] staging: rtl8723au: incorrect use of ether_addr_copy() Dan Carpenter
2014-10-08 12:26 ` Joe Perches
2014-10-08 12:46 ` Dan Carpenter
2014-10-08 12:50 ` Julia Lawall
2014-10-08 13:17 ` Dan Carpenter
2014-10-08 14:22 ` Joe Perches
2014-10-08 14:33 ` Jes Sorensen
2014-10-08 14:51 ` Joe Perches
2014-10-08 15:09 ` Jes Sorensen
2014-10-08 13:59 ` Jes Sorensen [this message]
2014-10-08 14:24 ` Dan Carpenter
2014-10-08 14:32 ` Jes Sorensen
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=wrfj4mvezmpm.fsf@redhat.com \
--to=jes.sorensen@redhat.com \
--cc=Larry.Finger@lwfinger.net \
--cc=dan.carpenter@oracle.com \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@linuxfoundation.org \
--cc=kernel-janitors@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).