[U-Boot] [PATCH 1/1] net/arp: remove superfluous assignments

Heinrich Schuchardt xypron.glpk at gmx.de
Mon May 8 19:24:28 UTC 2017


The value of variable pkt is never used.

The problem was indicated by clang scan-build.

Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
---
 net/arp.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/net/arp.c b/net/arp.c
index 824d2e9393..f3ceff9ed8 100644
--- a/net/arp.c
+++ b/net/arp.c
@@ -125,7 +125,6 @@ void arp_receive(struct ethernet_hdr *et, struct ip_udp_hdr *ip, int len)
 {
 	struct arp_hdr *arp;
 	struct in_addr reply_ip_addr;
-	uchar *pkt;
 	int eth_hdr_size;
 
 	/*
@@ -163,9 +162,7 @@ void arp_receive(struct ethernet_hdr *et, struct ip_udp_hdr *ip, int len)
 	case ARPOP_REQUEST:
 		/* reply with our IP address */
 		debug_cond(DEBUG_DEV_PKT, "Got ARP REQUEST, return our IP\n");
-		pkt = (uchar *)et;
 		eth_hdr_size = net_update_ether(et, et->et_src, PROT_ARP);
-		pkt += eth_hdr_size;
 		arp->ar_op = htons(ARPOP_REPLY);
 		memcpy(&arp->ar_tha, &arp->ar_sha, ARP_HLEN);
 		net_copy_ip(&arp->ar_tpa, &arp->ar_spa);
-- 
2.11.0



More information about the U-Boot mailing list