Reset the stored ping IP address before entering a netloop with different
protocol to ensure that it won't be interrupted by the received
correct ICMP_ECHO_REPLY packet.

Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>
---
v2:
- Moved the net_ping_ip reset to the beginning of the net_loop() if the
  protocol is not equal to PING. This is required, because the netretry
  feature of the network framework relies on the fact that the internal
  state of the commands is preserved between net_loop() calls.

v1: 
https://patchwork.ozlabs.org/project/uboot/patch/20200325134200.18959-1-m.szyprow...@samsung.com/
- Initial version
---
 net/net.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/net/net.c b/net/net.c
index 37932919d04..72b445774f9 100644
--- a/net/net.c
+++ b/net/net.c
@@ -409,6 +409,10 @@ int net_loop(enum proto_t protocol)
        int ret = -EINVAL;
        enum net_loop_state prev_net_state = net_state;
 
+#if defined(CONFIG_CMD_PING)
+       if (protocol != PING)
+               net_ping_ip.s_addr = 0;
+#endif
        net_restarted = 0;
        net_dev_exists = 0;
        net_try_count = 1;
-- 
2.17.1

Reply via email to