lq plugins: safer checking against LINK_COST_BROKEN and ROUTE_COST_BROKEN
authorFerry Huberts <ferry.huberts@pelagic.nl>
Sat, 9 Apr 2016 21:36:57 +0000 (23:36 +0200)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Mon, 11 Apr 2016 10:38:29 +0000 (12:38 +0200)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
src/lq_plugin.c
src/lq_plugin_default_ff.c
src/lq_plugin_default_ffeth.c
src/lq_plugin_default_float.c
src/lq_plugin_default_fpm.c

index 492431f..96eb80e 100644 (file)
@@ -320,7 +320,7 @@ get_linkcost_text(olsr_linkcost cost, bool route, struct lqtextbuffer *buffer)
   static const char *infinite = "INFINITE";
 
   if (route) {
-    if (cost == ROUTE_COST_BROKEN) {
+    if (cost >= ROUTE_COST_BROKEN) {
       return infinite;
     }
   } else {
index 5b17624..7fddfbb 100644 (file)
@@ -288,7 +288,7 @@ default_lq_calc_cost_ff(const void *ptr)
 
   cost = fpmidiv(itofpm(255 * 255), (int)lq->valueLq * (int)lq->valueNlq);
 
-  if (cost > LINK_COST_BROKEN)
+  if (cost >= LINK_COST_BROKEN)
     return LINK_COST_BROKEN;
   if (cost == 0)
     return 1;
index 63328f2..d2cdb25 100644 (file)
@@ -327,7 +327,7 @@ default_lq_calc_cost_ffeth(const void *ptr)
     cost /= 10;
   }
 
-  if (cost > LINK_COST_BROKEN)
+  if (cost >= LINK_COST_BROKEN)
     return LINK_COST_BROKEN;
   if (cost == 0)
     return 1;
index 09baec3..4d383a1 100644 (file)
@@ -108,7 +108,7 @@ default_lq_calc_cost_float(const void *ptr)
 
   cost = (olsr_linkcost) (1.0f / (lq->lq * lq->nlq) * (float)LQ_PLUGIN_LC_MULTIPLIER);
 
-  if (cost > LINK_COST_BROKEN)
+  if (cost >= LINK_COST_BROKEN)
     return LINK_COST_BROKEN;
   if (cost == 0) {
     return 1;
index e1fe841..bb969ea 100644 (file)
@@ -116,7 +116,7 @@ default_lq_calc_cost_fpm(const void *ptr)
 
   cost = LQ_FPM_LINKCOST_MULTIPLIER * 255 / (int)lq->valueLq * 255 / (int)lq->valueNlq;
 
-  if (cost > LINK_COST_BROKEN)
+  if (cost >= LINK_COST_BROKEN)
     return LINK_COST_BROKEN;
   if (cost == 0)
     return 1;