diff options
| author |   Alberto Compagno <acompagn+fdio@cisco.com> | 2020-01-21 15:38:45 +0000 |
|---|---|---|
| committer |   Gerrit Code Review <gerrit@fd.io> | 2020-01-21 15:38:45 +0000 |
| commit | 50bedb707155de675121a556a8b129280440cdf5 (patch) | |
| tree | a21b90b2a3178429330ff461a4ca2ae2d8bcfe9b /hicn-plugin/src/hicn_api.c | |
| parent | 43980f3096655df2b2ecec50e700dd6989b0e0d6 (diff) | |
| parent | f2de2bfea184034f675ab4a521e7deaae58c5ff8 (diff) | |
Merge "[HICN-477] Fixed strategy get ctx that could lead to a segfault"
Diffstat (limited to 'hicn-plugin/src/hicn_api.c')
| -rw-r--r-- | hicn-plugin/src/hicn_api.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/hicn-plugin/src/hicn_api.c b/hicn-plugin/src/hicn_api.c index df3365999..806de2b4e 100644 --- a/hicn-plugin/src/hicn_api.c +++ b/hicn-plugin/src/hicn_api.c @@ -702,7 +702,7 @@ static void vl_api_hicn_api_route_get_t_handler { hicn_dpo_vft = hicn_dpo_get_vft(hicn_dpo_id->dpoi_type); hicn_dpo_ctx = hicn_dpo_vft->hicn_dpo_get_ctx(hicn_dpo_id->dpoi_index); - for (int i = 0; i < hicn_dpo_ctx->entry_count; i++) + for (int i = 0; hicn_dpo_ctx != NULL && i < hicn_dpo_ctx->entry_count; i++) { if (dpo_id_is_valid(&hicn_dpo_ctx->next_hops[i])) { @@ -739,7 +739,7 @@ send_route_details (vl_api_registration_t * reg, { hicn_dpo_vft = hicn_dpo_get_vft (hicn_dpo_id->dpoi_type); hicn_dpo_ctx = hicn_dpo_vft->hicn_dpo_get_ctx (hicn_dpo_id->dpoi_index); - for (int i = 0; i < hicn_dpo_ctx->entry_count; i++) + for (int i = 0; hicn_dpo_ctx != NULL && i < hicn_dpo_ctx->entry_count; i++) { if (dpo_id_is_valid (&hicn_dpo_ctx->next_hops[i])) { |