On Sun, Aug 11, 2013 at 02:33:21PM +0200, Antonio Quartulli wrote:
From: Antonio Quartulli antonio@open-mesh.com
Signed-off-by: Antonio Quartulli antonio@open-mesh.com
originator.c | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/originator.c b/originator.c index b3f6910..3ee8806 100644 --- a/originator.c +++ b/originator.c @@ -380,6 +380,8 @@ batadv_purge_orig_neighbors(struct batadv_priv *bat_priv, bool neigh_purged = false; unsigned long last_seen; struct batadv_hard_iface *if_incoming;
struct batadv_algo_ops *bao = bat_priv->bat_algo_ops;
uint32_t neigh_metric, best_metric;
*best_neigh_node = NULL;
@@ -414,10 +416,11 @@ batadv_purge_orig_neighbors(struct batadv_priv *bat_priv, batadv_bonding_candidate_del(orig_node, neigh_node); batadv_neigh_node_free_ref(neigh_node); } else {
if ((!*best_neigh_node) ||(neigh_node->bat_iv.tq_avg >(*best_neigh_node)->bat_iv.tq_avg))
neigh_metric = bao->bat_metric_get(neigh_node);if (!*best_neigh_node || (neigh_metric > best_metric)) {
Shouldn't you initialize best_metric at some point?
*best_neigh_node = neigh_node;
best_metric = bao->bat_metric_get(neigh_node);}-- 1.8.1.5