diff options
| author | anlan_cs <vic.lan@pica8.com> | 2022-08-17 03:26:24 -0400 |
|---|---|---|
| committer | anlan_cs <vic.lan@pica8.com> | 2023-07-12 17:00:27 +0800 |
| commit | f8d94e8a628d123f299a4f388c58c8f8b222c6c2 (patch) | |
| tree | 5bc78db1aa7239b0baaf906c7389cee8bc2bd208 /zebra/zebra_mpls.c | |
| parent | 30f511e63ac853465be7114322282fb5d833fa0c (diff) | |
zebra: remove unnecessary check for default vrf
The default vrf is generally non-NULL, except when shutdown. So, most
of the time it is not necessary to check if it is NULL, we should
remove the useless checks for it.
Searched them with exact match:
```
grep -rI "zebra_vrf_lookup_by_id(VRF_DEFAULT)" | wc -l
31
```
Signed-off-by: anlan_cs <vic.lan@pica8.com>
Diffstat (limited to 'zebra/zebra_mpls.c')
| -rw-r--r-- | zebra/zebra_mpls.c | 8 |
1 files changed, 0 insertions, 8 deletions
diff --git a/zebra/zebra_mpls.c b/zebra/zebra_mpls.c index ef109774d9..eac4fcc8e0 100644 --- a/zebra/zebra_mpls.c +++ b/zebra/zebra_mpls.c @@ -1029,8 +1029,6 @@ static void lsp_processq_del(struct work_queue *wq, void *data) return; zvrf = zebra_vrf_lookup_by_id(VRF_DEFAULT); - assert(zvrf); - lsp_table = zvrf->lsp_table; if (!lsp_table) // unexpected return; @@ -1776,9 +1774,6 @@ void zebra_mpls_lsp_dplane_result(struct zebra_dplane_ctx *ctx) case DPLANE_OP_LSP_UPDATE: /* Look for zebra LSP object */ zvrf = zebra_vrf_lookup_by_id(VRF_DEFAULT); - if (zvrf == NULL) - break; - lsp_table = zvrf->lsp_table; tmp_ile.in_label = label; @@ -2093,9 +2088,6 @@ void zebra_mpls_process_dplane_notify(struct zebra_dplane_ctx *ctx) /* Look for zebra LSP object */ zvrf = zebra_vrf_lookup_by_id(VRF_DEFAULT); - if (zvrf == NULL) - return; - lsp_table = zvrf->lsp_table; tmp_ile.in_label = dplane_ctx_get_in_label(ctx); |