net: update check condition from IFF_UP to IFF_RUNNING in the network sending process#17589
Conversation
20698b4 to
2d6df93
Compare
acassis
left a comment
There was a problem hiding this comment.
@wangchen61698 please provide more details why this modification was necessary. You just said you modified it, but you didn't say why? What was the issue you were solving? Please also provide valid testing showing the issue and showing it was fixed
…process Modify check condition from IFF_UP to IFF_RUNNING in the sending process Signed-off-by: wangchen <wangchen41@xiaomi.com>
The issue we encountered is that the network card is in a non-running state but is up. Messages should not be sent out, but the Address Conflict Detection (ACD) function still sends ARP messages. This behavior is incorrect, so we modified the judgment condition. ARP messages can only be sent if the network card is in a running state; otherwise, they cannot be sent. We also simultaneously modified the relevant judgment logic in other business codes to ensure consistency in the judgment of network card status across other businesses @acassis please review it again,Thanks. |
Note: Please adhere to Contributing Guidelines.
Summary
Modify check condition from IFF_UP to IFF_RUNNING in the network sending process
Impact
network sending process
Testing
log info

before patch

after patch
wlan0 is running status
wlan0 is up status