Fixed eapol target selection sort algorithm

Sorting did not check correctly the RSSI threshold values

Pan cost did not check correctly and assumed better as equal

Author: Mika Tervonen

source/6LoWPAN/ws/ws_bootstrap.c

@@ -892,7 +892,7 @@ static void ws_bootstrap_pan_advertisement_analyse_active(struct protocol_interf
 static parent_info_t *ws_bootstrap_candidate_parent_get_best(protocol_interface_info_entry_t *cur)
 {
     ns_list_foreach_safe(parent_info_t, entry, &cur->ws_info->parent_list_reserved) {
-        tr_info("candidate list a:%s panid:%x rssi:%d age:%"PRIu32, trace_array(entry->addr, 8), entry->pan_id, entry->signal_dbm, protocol_core_monotonic_time - entry->age);
+        tr_info("candidate list a:%s panid:%x cost:%d size:%d rssi:%d age:%"PRIu32, trace_array(entry->addr, 8), entry->pan_id, entry->pan_information.routing_cost, entry->pan_information.pan_size, entry->signal_dbm, protocol_core_monotonic_time - entry->age);
     }
 
     return ns_list_get_first(&cur->ws_info->parent_list_reserved);
@@ -988,12 +988,19 @@ static bool ws_bootstrap_candidate_parent_compare(parent_info_t *p1, parent_info
         // above threshold is always better than not.
         return true;
     }
+    if (ws_neighbor_class_rsl_from_dbm_calculate(p2->signal_dbm) < (DEVICE_MIN_SENS + CAND_PARENT_THRESHOLD + CAND_PARENT_HYSTERISIS) &&
+            ws_neighbor_class_rsl_from_dbm_calculate(p1->signal_dbm) > (DEVICE_MIN_SENS + CAND_PARENT_THRESHOLD + CAND_PARENT_HYSTERISIS)) {
+        // P2 is less than threshold and P1 is larger so P1 is always better.
+        return false;
+    }
 
     // Select the lowest PAN cost
     uint16_t p1_pan_cost = (p1->pan_information.routing_cost / PRC_WEIGHT_FACTOR) + (p1->pan_information.pan_size / PS_WEIGHT_FACTOR);
     uint16_t p2_pan_cost = (p2->pan_information.routing_cost / PRC_WEIGHT_FACTOR) + (p2->pan_information.pan_size / PS_WEIGHT_FACTOR);
     if (p1_pan_cost > p2_pan_cost) {
         return true;
+    } else if (p1_pan_cost < p2_pan_cost) {
+        return false;
     }
 
     // If pan cost is the same then we select the one we hear highest