Pan coordinator blacklist update

Added support to clean coordinator blacklist

Integrated free call to stack after pana if NO_MLE is true and after
link request state

Added blacklist coordinator also after MLE link request fail state.
(cherry picked from commit 7911211e72305ec7cd104bfcbeed89cea0d44b24)

Author: Juha Heiskanen

source/6LoWPAN/Bootstraps/Generic/protocol_6lowpan_bootstrap.c

@@ -100,6 +100,7 @@ static void protocol_6lowpan_neighbor_information_remove(int8_t interface_id, ml
 static int8_t protocol_6lowpan_host_challenge(int8_t interface_id, const uint8_t *mac64);
 static int8_t protocol_6lowpan_router_challenge(int8_t interface_id, const uint8_t *mac64);
 static void protocol_6lowpan_address_reg_ready(protocol_interface_info_entry_t *cur_interface);
+static void coordinator_black_list(protocol_interface_info_entry_t *cur);
 
 static mle_6lowpan_data_t *mle_6lowpan_data;
 
@@ -814,6 +815,7 @@ static bool mle_parent_link_req_cb(int8_t interface_id, uint16_t msgId, bool use
         if (cur->nwk_bootstrap_state == ER_MLE_LINK_REQ) {
             //Enter ND scan
             bootsrap_next_state_kick(ER_SCAN, cur);
+            pan_coordinator_blacklist_free(&cur->pan_cordinator_black_list);
         }
 #ifdef HAVE_RPL
         else if (cur->nwk_bootstrap_state == ER_ROUTER_SYNCH) {
@@ -842,6 +844,9 @@ static bool mle_parent_link_req_cb(int8_t interface_id, uint16_t msgId, bool use
             case ER_MLE_LINK_REQ:
             case ER_MLE_LINK_ADDRESS_SYNCH:
             case ER_MLE_LINK_SHORT_SYNCH:
+                if (cur->nwk_bootstrap_state == ER_MLE_LINK_REQ) {
+                    coordinator_black_list(cur);
+                }
                 tr_debug("Link synch fail %u", cur->nwk_bootstrap_state);
                 bootsrap_next_state_kick(ER_BOOTSTRAP_CONNECTION_DOWN, cur);
                 break;
@@ -2283,15 +2288,8 @@ static void nwk_6lowpan_bootsrap_pana_authentication_start(protocol_interface_in
 
 #endif
 
-static void nwk_6lowpan_network_authentication_fail(protocol_interface_info_entry_t *cur)
+static void coordinator_black_list(protocol_interface_info_entry_t *cur)
 {
-    nwk_scan_params_t *scan_params =
-            &cur->mac_parameters->nwk_scan_params;
-
-    tr_warn("Pana Auhth er");
-
-    scan_params->nwk_cur_active = mac_helper_free_pan_descriptions(scan_params->nwk_cur_active);
-    //Black List coordinator
     uint8_t coord_pan_address[10];
     addrtype_t cord_adr_type = mac_helper_coordinator_address_get(cur, coord_pan_address +2);
 
@@ -2304,6 +2302,18 @@ static void nwk_6lowpan_network_authentication_fail(protocol_interface_info_entr
 
         pan_cordinator_blacklist_pan_set(&cur->pan_cordinator_black_list, coord_pan_address, 300);
     }
+}
+
+static void nwk_6lowpan_network_authentication_fail(protocol_interface_info_entry_t *cur)
+{
+    nwk_scan_params_t *scan_params =
+            &cur->mac_parameters->nwk_scan_params;
+
+    tr_warn("Pana Auhth er");
+
+    scan_params->nwk_cur_active = mac_helper_free_pan_descriptions(scan_params->nwk_cur_active);
+    //Black List coordinator
+    coordinator_black_list(cur);
 
     nwk_bootsrap_state_update(ARM_NWK_AUHTENTICATION_FAIL, cur);
 }
@@ -2379,6 +2389,7 @@ static void nwk_6lowpan_network_authentication_done(protocol_interface_info_entr
             tr_debug("Link request start fail");
         }
 #else
+        pan_coordinator_blacklist_free(&cur->pan_cordinator_black_list);
         cur->nwk_bootstrap_state = ER_SCAN;
         nwk_6lowpan_router_scan_state(cur);
 #endif

source/Service_Libs/pan_blacklist/pan_blacklist.c

@@ -91,7 +91,13 @@ void pan_coordinator_blacklist_time_update(pan_coordinator_blaclist_cache_s *lis
     }
 }
 
-
+void pan_coordinator_blacklist_free(pan_coordinator_blaclist_cache_s *list_ptr)
+{
+    ns_list_foreach_safe(pan_coordinator_blacklist_entry_t, cur_ptr, &list_ptr->head) {
+        ns_list_remove(&list_ptr->head, cur_ptr);
+        ns_dyn_mem_free(cur_ptr);
+    }
+}
 
 bool pan_blacklist_filter(pan_blaclist_cache_s *list_ptr, uint16_t panid)
 {

source/Service_Libs/pan_blacklist/pan_blacklist_api.h

@@ -57,6 +57,8 @@ void pan_blacklist_cache_init(pan_blaclist_cache_s *blacklist_cache);
 
 void pan_coordinator_blacklist_cache_init(pan_coordinator_blaclist_cache_s *blacklist_cache);
 
+void pan_coordinator_blacklist_free(pan_coordinator_blaclist_cache_s *list_ptr);
+
 void pan_blacklist_pan_set(pan_blaclist_cache_s *list_ptr, uint16_t panid, uint16_t timeout);
 
 void pan_cordinator_blacklist_pan_set(pan_coordinator_blaclist_cache_s *list_ptr, uint8_t *cordinator_data, uint16_t timeout);

test/nanostack/unittest/stub/pan_blacklist_stub.c

@@ -63,3 +63,8 @@ void pan_cordinator_blacklist_pan_set(pan_coordinator_blaclist_cache_s *list_ptr
 {
 
 }
+
+void pan_coordinator_blacklist_free(pan_coordinator_blaclist_cache_s *list_ptr)
+{
+
+}