diff options
author | Thomas Haller <thaller@redhat.com> | 2020-07-21 17:49:23 +0200 |
---|---|---|
committer | Thomas Haller <thaller@redhat.com> | 2020-07-23 15:29:24 +0200 |
commit | 3f771c55ac9ee6b3179e9bf8b0743a2e544a108c (patch) | |
tree | a94e7eeed913679efb906283463bc63aafd1ef5d | |
parent | 84d93315d81182deec59ca0365fc9ad0cd79f28c (diff) | |
download | NetworkManager-3f771c55ac9ee6b3179e9bf8b0743a2e544a108c.tar.gz |
core: use nmp_object_ref_set() instead of _nm_ip_config_best_default_route_set()
_nm_ip_config_best_default_route_set() doesn't really do anything
special. Use the generic helper function for the same job.
Also because NMIP4Config in the current form will be replaced by
something else, and this code needs to change.
-rw-r--r-- | src/nm-ip4-config.c | 25 | ||||
-rw-r--r-- | src/nm-ip4-config.h | 1 | ||||
-rw-r--r-- | src/nm-ip6-config.c | 14 |
3 files changed, 14 insertions, 26 deletions
diff --git a/src/nm-ip4-config.c b/src/nm-ip4-config.c index 9ad8116423..35e7fb8f4b 100644 --- a/src/nm-ip4-config.c +++ b/src/nm-ip4-config.c @@ -437,22 +437,11 @@ _nm_ip_config_best_default_route_find_better (const NMPObject *obj_cur, const NM } gboolean -_nm_ip_config_best_default_route_set (const NMPObject **best_default_route, const NMPObject *new_candidate) -{ - if (new_candidate == *best_default_route) - return FALSE; - nmp_object_ref (new_candidate); - nm_clear_nmp_object (best_default_route); - *best_default_route = new_candidate; - return TRUE; -} - -gboolean _nm_ip_config_best_default_route_merge (const NMPObject **best_default_route, const NMPObject *new_candidate) { new_candidate = _nm_ip_config_best_default_route_find_better (*best_default_route, new_candidate); - return _nm_ip_config_best_default_route_set (best_default_route, new_candidate); + return nmp_object_ref_set (best_default_route, new_candidate); } const NMPObject * @@ -1505,8 +1494,8 @@ nm_ip4_config_subtract (NMIP4Config *dst, } } if (changed_default_route) { - _nm_ip_config_best_default_route_set (&dst_priv->best_default_route, - _nm_ip4_config_best_default_route_find (dst)); + nmp_object_ref_set (&dst_priv->best_default_route, + _nm_ip4_config_best_default_route_find (dst)); _notify (dst, PROP_GATEWAY); } if (changed) @@ -1659,7 +1648,7 @@ _nm_ip4_config_intersect_helper (NMIP4Config *dst, nm_assert_not_reached (); changed = TRUE; } - if (_nm_ip_config_best_default_route_set (&dst_priv->best_default_route, new_best_default_route)) { + if (nmp_object_ref_set (&dst_priv->best_default_route, new_best_default_route)) { nm_assert (changed); _notify (dst, PROP_GATEWAY); } @@ -1896,7 +1885,7 @@ nm_ip4_config_replace (NMIP4Config *dst, const NMIP4Config *src, gboolean *relev new_best_default_route = _nm_ip_config_best_default_route_find_better (new_best_default_route, obj_new); } nm_dedup_multi_index_dirty_remove_idx (dst_priv->multi_idx, &dst_priv->idx_ip4_routes, FALSE); - if (_nm_ip_config_best_default_route_set (&dst_priv->best_default_route, new_best_default_route)) + if (nmp_object_ref_set (&dst_priv->best_default_route, new_best_default_route)) _notify (dst, PROP_GATEWAY); _notify_routes (dst); } @@ -2990,8 +2979,8 @@ nm_ip4_config_nmpobj_remove (NMIP4Config *self, break; case NMP_OBJECT_TYPE_IP4_ROUTE: if (priv->best_default_route == obj_old) { - if (_nm_ip_config_best_default_route_set (&priv->best_default_route, - _nm_ip4_config_best_default_route_find (self))) + if (nmp_object_ref_set (&priv->best_default_route, + _nm_ip4_config_best_default_route_find (self))) _notify (self, PROP_GATEWAY); } _notify_routes (self); diff --git a/src/nm-ip4-config.h b/src/nm-ip4-config.h index ea06174976..f09d99134c 100644 --- a/src/nm-ip4-config.h +++ b/src/nm-ip4-config.h @@ -88,7 +88,6 @@ nm_ip_config_best_default_route_is (const NMPObject *obj) } const NMPObject *_nm_ip_config_best_default_route_find_better (const NMPObject *obj_cur, const NMPObject *obj_cmp); -gboolean _nm_ip_config_best_default_route_set (const NMPObject **best_default_route, const NMPObject *new_candidate); gboolean _nm_ip_config_best_default_route_merge (const NMPObject **best_default_route, const NMPObject *new_candidate); /*****************************************************************************/ diff --git a/src/nm-ip6-config.c b/src/nm-ip6-config.c index deb30e77d7..9c596bf95d 100644 --- a/src/nm-ip6-config.c +++ b/src/nm-ip6-config.c @@ -1120,8 +1120,8 @@ nm_ip6_config_subtract (NMIP6Config *dst, } } if (changed_default_route) { - _nm_ip_config_best_default_route_set (&dst_priv->best_default_route, - _nm_ip6_config_best_default_route_find (dst)); + nmp_object_ref_set (&dst_priv->best_default_route, + _nm_ip6_config_best_default_route_find (dst)); _notify (dst, PROP_GATEWAY); } if (changed) @@ -1244,7 +1244,7 @@ _nm_ip6_config_intersect_helper (NMIP6Config *dst, nm_assert_not_reached (); changed = TRUE; } - if (_nm_ip_config_best_default_route_set (&dst_priv->best_default_route, new_best_default_route)) { + if (nmp_object_ref_set (&dst_priv->best_default_route, new_best_default_route)) { nm_assert (changed); _notify (dst, PROP_GATEWAY); } @@ -1481,7 +1481,7 @@ nm_ip6_config_replace (NMIP6Config *dst, const NMIP6Config *src, gboolean *relev new_best_default_route = _nm_ip_config_best_default_route_find_better (new_best_default_route, obj_new); } nm_dedup_multi_index_dirty_remove_idx (dst_priv->multi_idx, &dst_priv->idx_ip6_routes, FALSE); - if (_nm_ip_config_best_default_route_set (&dst_priv->best_default_route, new_best_default_route)) + if (nmp_object_ref_set (&dst_priv->best_default_route, new_best_default_route)) _notify (dst, PROP_GATEWAY); _notify_routes (dst); } @@ -1955,7 +1955,7 @@ nm_ip6_config_reset_routes_ndisc (NMIP6Config *self, if (nm_dedup_multi_index_dirty_remove_idx (priv->multi_idx, &priv->idx_ip6_routes, FALSE) > 0) changed = TRUE; - if (_nm_ip_config_best_default_route_set (&priv->best_default_route, new_best_default_route)) { + if (nmp_object_ref_set (&priv->best_default_route, new_best_default_route)) { changed = TRUE; _notify (self, PROP_GATEWAY); } @@ -2458,8 +2458,8 @@ nm_ip6_config_nmpobj_remove (NMIP6Config *self, break; case NMP_OBJECT_TYPE_IP6_ROUTE: if (priv->best_default_route == obj_old) { - if (_nm_ip_config_best_default_route_set (&priv->best_default_route, - _nm_ip6_config_best_default_route_find (self))) + if (nmp_object_ref_set (&priv->best_default_route, + _nm_ip6_config_best_default_route_find (self))) _notify (self, PROP_GATEWAY); } _notify_routes (self); |