All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH -next] staging: r8188eu: use 'is_zero_ether_addr' to identify an empty address
@ 2022-07-04 12:45 ` Zhang Zekun
  0 siblings, 0 replies; 8+ messages in thread
From: Zhang Zekun @ 2022-07-04 12:45 UTC (permalink / raw)
  To: Larry.Finger, phil, paskripkin, gregkh, martin, straube.linux
  Cc: linux-media, linux-rockchip, linux-staging, linux-kernel, xuqiang36

Use 'is_zero_ether_addr' to identify an empty ethernet address, intead
of using 'memcpy' directly.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Zhang Zekun <zhangzekun11@huawei.com>
---
 drivers/staging/r8188eu/core/rtw_mlme_ext.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/r8188eu/core/rtw_mlme_ext.c b/drivers/staging/r8188eu/core/rtw_mlme_ext.c
index cce0575e93b7..365f5b2786cf 100644
--- a/drivers/staging/r8188eu/core/rtw_mlme_ext.c
+++ b/drivers/staging/r8188eu/core/rtw_mlme_ext.c
@@ -3513,7 +3513,6 @@ static unsigned int on_action_public_p2p(struct recv_frame *precv_frame)
 	u32	p2p_ielen;
 	struct	wifidirect_info	*pwdinfo = &padapter->wdinfo;
 	u8	result = P2P_STATUS_SUCCESS;
-	u8	empty_addr[ETH_ALEN] = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 };
 
 	frame_body = (unsigned char *)(pframe + sizeof(struct ieee80211_hdr_3addr));
 
@@ -3551,7 +3550,7 @@ static unsigned int on_action_public_p2p(struct recv_frame *precv_frame)
 
 		/*	Commented by Kurt 20120113 */
 		/*	Get peer_dev_addr here if peer doesn't issue prov_disc frame. */
-		if (!memcmp(pwdinfo->rx_prov_disc_info.peerDevAddr, empty_addr, ETH_ALEN))
+		if (is_zero_ether_addr(pwdinfo->rx_prov_disc_info.peerDevAddr))
 			memcpy(pwdinfo->rx_prov_disc_info.peerDevAddr, GetAddr2Ptr(pframe), ETH_ALEN);
 
 		result = process_p2p_group_negotation_req(pwdinfo, frame_body, len);
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread
* [PATCH -next] staging: r8188eu: use 'is_zero_ether_addr' to identify an empty address
@ 2022-07-04 12:45 ` Zhang Zekun
  0 siblings, 0 replies; 8+ messages in thread
From: Zhang Zekun @ 2022-07-04 12:45 UTC (permalink / raw)
  To: Larry.Finger, phil, paskripkin, gregkh, martin, straube.linux
  Cc: linux-media, linux-rockchip, linux-staging, linux-kernel, xuqiang36

Use 'is_zero_ether_addr' to identify an empty ethernet address, intead
of using 'memcpy' directly.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Zhang Zekun <zhangzekun11@huawei.com>
---
 drivers/staging/r8188eu/core/rtw_mlme_ext.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/r8188eu/core/rtw_mlme_ext.c b/drivers/staging/r8188eu/core/rtw_mlme_ext.c
index cce0575e93b7..365f5b2786cf 100644
--- a/drivers/staging/r8188eu/core/rtw_mlme_ext.c
+++ b/drivers/staging/r8188eu/core/rtw_mlme_ext.c
@@ -3513,7 +3513,6 @@ static unsigned int on_action_public_p2p(struct recv_frame *precv_frame)
 	u32	p2p_ielen;
 	struct	wifidirect_info	*pwdinfo = &padapter->wdinfo;
 	u8	result = P2P_STATUS_SUCCESS;
-	u8	empty_addr[ETH_ALEN] = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 };
 
 	frame_body = (unsigned char *)(pframe + sizeof(struct ieee80211_hdr_3addr));
 
@@ -3551,7 +3550,7 @@ static unsigned int on_action_public_p2p(struct recv_frame *precv_frame)
 
 		/*	Commented by Kurt 20120113 */
 		/*	Get peer_dev_addr here if peer doesn't issue prov_disc frame. */
-		if (!memcmp(pwdinfo->rx_prov_disc_info.peerDevAddr, empty_addr, ETH_ALEN))
+		if (is_zero_ether_addr(pwdinfo->rx_prov_disc_info.peerDevAddr))
 			memcpy(pwdinfo->rx_prov_disc_info.peerDevAddr, GetAddr2Ptr(pframe), ETH_ALEN);
 
 		result = process_p2p_group_negotation_req(pwdinfo, frame_body, len);
-- 
2.17.1


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 8+ messages in thread
* Re: [PATCH -next] staging: r8188eu: use 'is_zero_ether_addr' to identify an empty address
  2022-07-04 13:27   ` Greg KH
@ 2022-07-04 13:38     ` Greg KH
  -1 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2022-07-04 13:38 UTC (permalink / raw)
  To: Zhang Zekun
  Cc: Larry.Finger, phil, paskripkin, martin, straube.linux,
	linux-media, linux-rockchip, linux-staging, linux-kernel,
	xuqiang36

On Mon, Jul 04, 2022 at 03:27:01PM +0200, Greg KH wrote:
> On Mon, Jul 04, 2022 at 12:31:40PM +0000, Zhang Zekun wrote:
> > Use 'is_zero_ether_addr' to identify an empty ethernet address, intead
> > of using 'memcpy' directly.
> > 
> > Reported-by: Hulk Robot <hulkci@huawei.com>
> 
> I am now just going to ignore all patch submissions with this line in it
> based on a total lack of responses by the developers using it.  See
> https://lore.kernel.org/r/Yr7DQJTPrSWTOa0c@kroah.com for why.

And now just ignore patches from your domain, see:
	https://lore.kernel.org/r/YsLq5vXtJgLWCqqz@kroah.com
for why (patches sent totally ignoring previous requests.)

Please work with the open source developers in your company to fix this
broken process, as it is causing more problems for me than it is worth.

greg k-h

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH -next] staging: r8188eu: use 'is_zero_ether_addr' to identify an empty address
@ 2022-07-04 13:38     ` Greg KH
  0 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2022-07-04 13:38 UTC (permalink / raw)
  To: Zhang Zekun
  Cc: Larry.Finger, phil, paskripkin, martin, straube.linux,
	linux-media, linux-rockchip, linux-staging, linux-kernel,
	xuqiang36

On Mon, Jul 04, 2022 at 03:27:01PM +0200, Greg KH wrote:
> On Mon, Jul 04, 2022 at 12:31:40PM +0000, Zhang Zekun wrote:
> > Use 'is_zero_ether_addr' to identify an empty ethernet address, intead
> > of using 'memcpy' directly.
> > 
> > Reported-by: Hulk Robot <hulkci@huawei.com>
> 
> I am now just going to ignore all patch submissions with this line in it
> based on a total lack of responses by the developers using it.  See
> https://lore.kernel.org/r/Yr7DQJTPrSWTOa0c@kroah.com for why.

And now just ignore patches from your domain, see:
	https://lore.kernel.org/r/YsLq5vXtJgLWCqqz@kroah.com
for why (patches sent totally ignoring previous requests.)

Please work with the open source developers in your company to fix this
broken process, as it is causing more problems for me than it is worth.

greg k-h

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH -next] staging: r8188eu: use 'is_zero_ether_addr' to identify an empty address
  2022-07-04 12:31 ` Zhang Zekun
@ 2022-07-04 13:27   ` Greg KH
  -1 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2022-07-04 13:27 UTC (permalink / raw)
  To: Zhang Zekun
  Cc: Larry.Finger, phil, paskripkin, martin, straube.linux,
	linux-media, linux-rockchip, linux-staging, linux-kernel,
	xuqiang36

On Mon, Jul 04, 2022 at 12:31:40PM +0000, Zhang Zekun wrote:
> Use 'is_zero_ether_addr' to identify an empty ethernet address, intead
> of using 'memcpy' directly.
> 
> Reported-by: Hulk Robot <hulkci@huawei.com>

I am now just going to ignore all patch submissions with this line in it
based on a total lack of responses by the developers using it.  See
https://lore.kernel.org/r/Yr7DQJTPrSWTOa0c@kroah.com for why.

greg k-h

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH -next] staging: r8188eu: use 'is_zero_ether_addr' to identify an empty address
@ 2022-07-04 13:27   ` Greg KH
  0 siblings, 0 replies; 8+ messages in thread
From: Greg KH @ 2022-07-04 13:27 UTC (permalink / raw)
  To: Zhang Zekun
  Cc: Larry.Finger, phil, paskripkin, martin, straube.linux,
	linux-media, linux-rockchip, linux-staging, linux-kernel,
	xuqiang36

On Mon, Jul 04, 2022 at 12:31:40PM +0000, Zhang Zekun wrote:
> Use 'is_zero_ether_addr' to identify an empty ethernet address, intead
> of using 'memcpy' directly.
> 
> Reported-by: Hulk Robot <hulkci@huawei.com>

I am now just going to ignore all patch submissions with this line in it
based on a total lack of responses by the developers using it.  See
https://lore.kernel.org/r/Yr7DQJTPrSWTOa0c@kroah.com for why.

greg k-h

_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [PATCH -next] staging: r8188eu: use 'is_zero_ether_addr' to identify an empty address
@ 2022-07-04 12:31 ` Zhang Zekun
  0 siblings, 0 replies; 8+ messages in thread
From: Zhang Zekun @ 2022-07-04 12:31 UTC (permalink / raw)
  To: Larry.Finger, phil, paskripkin, gregkh, martin, straube.linux
  Cc: linux-media, linux-rockchip, linux-staging, linux-kernel, xuqiang36

Use 'is_zero_ether_addr' to identify an empty ethernet address, intead
of using 'memcpy' directly.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Zhang Zekun <zhangzekun11@huawei.com>
---
 drivers/staging/r8188eu/core/rtw_mlme_ext.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/r8188eu/core/rtw_mlme_ext.c b/drivers/staging/r8188eu/core/rtw_mlme_ext.c
index cce0575e93b7..365f5b2786cf 100644
--- a/drivers/staging/r8188eu/core/rtw_mlme_ext.c
+++ b/drivers/staging/r8188eu/core/rtw_mlme_ext.c
@@ -3513,7 +3513,6 @@ static unsigned int on_action_public_p2p(struct recv_frame *precv_frame)
 	u32	p2p_ielen;
 	struct	wifidirect_info	*pwdinfo = &padapter->wdinfo;
 	u8	result = P2P_STATUS_SUCCESS;
-	u8	empty_addr[ETH_ALEN] = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 };
 
 	frame_body = (unsigned char *)(pframe + sizeof(struct ieee80211_hdr_3addr));
 
@@ -3551,7 +3550,7 @@ static unsigned int on_action_public_p2p(struct recv_frame *precv_frame)
 
 		/*	Commented by Kurt 20120113 */
 		/*	Get peer_dev_addr here if peer doesn't issue prov_disc frame. */
-		if (!memcmp(pwdinfo->rx_prov_disc_info.peerDevAddr, empty_addr, ETH_ALEN))
+		if (is_zero_ether_addr(pwdinfo->rx_prov_disc_info.peerDevAddr))
 			memcpy(pwdinfo->rx_prov_disc_info.peerDevAddr, GetAddr2Ptr(pframe), ETH_ALEN);
 
 		result = process_p2p_group_negotation_req(pwdinfo, frame_body, len);
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread
* [PATCH -next] staging: r8188eu: use 'is_zero_ether_addr' to identify an empty address
@ 2022-07-04 12:31 ` Zhang Zekun
  0 siblings, 0 replies; 8+ messages in thread
From: Zhang Zekun @ 2022-07-04 12:31 UTC (permalink / raw)
  To: Larry.Finger, phil, paskripkin, gregkh, martin, straube.linux
  Cc: linux-media, linux-rockchip, linux-staging, linux-kernel, xuqiang36

Use 'is_zero_ether_addr' to identify an empty ethernet address, intead
of using 'memcpy' directly.

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Zhang Zekun <zhangzekun11@huawei.com>
---
 drivers/staging/r8188eu/core/rtw_mlme_ext.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/r8188eu/core/rtw_mlme_ext.c b/drivers/staging/r8188eu/core/rtw_mlme_ext.c
index cce0575e93b7..365f5b2786cf 100644
--- a/drivers/staging/r8188eu/core/rtw_mlme_ext.c
+++ b/drivers/staging/r8188eu/core/rtw_mlme_ext.c
@@ -3513,7 +3513,6 @@ static unsigned int on_action_public_p2p(struct recv_frame *precv_frame)
 	u32	p2p_ielen;
 	struct	wifidirect_info	*pwdinfo = &padapter->wdinfo;
 	u8	result = P2P_STATUS_SUCCESS;
-	u8	empty_addr[ETH_ALEN] = { 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 };
 
 	frame_body = (unsigned char *)(pframe + sizeof(struct ieee80211_hdr_3addr));
 
@@ -3551,7 +3550,7 @@ static unsigned int on_action_public_p2p(struct recv_frame *precv_frame)
 
 		/*	Commented by Kurt 20120113 */
 		/*	Get peer_dev_addr here if peer doesn't issue prov_disc frame. */
-		if (!memcmp(pwdinfo->rx_prov_disc_info.peerDevAddr, empty_addr, ETH_ALEN))
+		if (is_zero_ether_addr(pwdinfo->rx_prov_disc_info.peerDevAddr))
 			memcpy(pwdinfo->rx_prov_disc_info.peerDevAddr, GetAddr2Ptr(pframe), ETH_ALEN);
 
 		result = process_p2p_group_negotation_req(pwdinfo, frame_body, len);
-- 
2.17.1


_______________________________________________
Linux-rockchip mailing list
Linux-rockchip@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-rockchip

^ permalink raw reply related	[flat|nested] 8+ messages in thread
end of thread, other threads:[~2022-07-04 13:39 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-04 12:45 [PATCH -next] staging: r8188eu: use 'is_zero_ether_addr' to identify an empty address Zhang Zekun
2022-07-04 12:45 ` Zhang Zekun
  -- strict thread matches above, loose matches on Subject: below --
2022-07-04 12:31 Zhang Zekun
2022-07-04 12:31 ` Zhang Zekun
2022-07-04 13:27 ` Greg KH
2022-07-04 13:27   ` Greg KH
2022-07-04 13:38   ` Greg KH
2022-07-04 13:38     ` Greg KH

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.