diff options
| author | Anuradha Karuppiah <anuradhak@cumulusnetworks.com> | 2019-04-06 17:37:13 +0200 |
|---|---|---|
| committer | Anuradha Karuppiah <anuradhak@cumulusnetworks.com> | 2019-04-20 17:33:23 +0200 |
| commit | 7b227dafe60f7bc1a226b11d01d1e294acbb7587 (patch) | |
| tree | ea1f692c7b02edf657c11bd257cfcfeece48a2db /pimd/pim_vxlan.c | |
| parent | pimd: create pimreg implicity if ipmr-lo is the first pim device (diff) | |
| download | frr-7b227dafe60f7bc1a226b11d01d1e294acbb7587.tar.xz frr-7b227dafe60f7bc1a226b11d01d1e294acbb7587.zip | |
pimd: cleanup unncessary null pointer check
This was resulting in static analyzer warnings for subsequent usage
of the same pointer -
pimd/pim_vxlan.c:962:36: warning: Access to field 'info' results in a
dereference of a null pointer (loaded from variable 'ifp')
pim_ifp = (struct pim_interface *)ifp->info;
^~~~~~~~~
1 warning generated.
Signed-off-by: Anuradha Karuppiah <anuradhak@cumulusnetworks.com>
Diffstat (limited to 'pimd/pim_vxlan.c')
| -rw-r--r-- | pimd/pim_vxlan.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pimd/pim_vxlan.c b/pimd/pim_vxlan.c index a4a6ce692..e973c8c04 100644 --- a/pimd/pim_vxlan.c +++ b/pimd/pim_vxlan.c @@ -956,7 +956,7 @@ void pim_vxlan_add_term_dev(struct pim_instance *pim, if (PIM_DEBUG_VXLAN) zlog_debug("vxlan term oif changed from %s to %s", pim->vxlan.term_if ? pim->vxlan.term_if->name : "-", - ifp ? ifp->name : "-"); + ifp->name); /* enable pim on the term ifp */ pim_ifp = (struct pim_interface *)ifp->info; |