Skip to content

Commit

Permalink
networkd: do not drop config for unmanaged interfaces
Browse files Browse the repository at this point in the history
Flushing foreign configuration for unmanaged interfaces is outright
evil, especially when it's a regular occurence with Wi-Fi.

Fixes: 3104883 "networkd: remove route if carrier is lost"
Ref: systemd#3831
  • Loading branch information
grawity committed Aug 25, 2016
1 parent 3dea75d commit c436d55
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions src/network/networkd-link.c
Original file line number Diff line number Diff line change
Expand Up @@ -2957,9 +2957,11 @@ static int link_carrier_lost(Link *link) {
if (r < 0)
return r;

r = link_drop_foreign_config(link);
if (r < 0)
return r;
if (link->state != LINK_STATE_UNMANAGED) {
r = link_drop_foreign_config(link);
if (r < 0)
return r;
}

r = link_handle_bound_by_list(link);
if (r < 0)
Expand Down

0 comments on commit c436d55

Please sign in to comment.