commit c8394ace7081ef0e71f3d162067c83c2629fc088
author Morgan Stewart <morgan@cumulusnetworks.com> Thu Sep 17 19:04:30 2015 -0400
committer Paul Jakma <paul@quagga.net> Thu Sep 24 15:26:46 2015 +0100
tree 7736d5d1ca6d3ea79c3f43e32dccf4baf0fa1094
parent 75daab1784f515104ceef747d134981ecee7b0af

Fixed if_add_update possible null dereference

Coverity Scan #1221454
In zebra/interface.c if_data could be null dereferenced without early
check.

Signed-off-by: Morgan Stewart <morgan@cumulusnetworks.com>
Reviewed-by: Donald Sharp <sharpd@cumulusnetworks.com>

zebra/interface.c

diff --git a/zebra/interface.c b/zebra/interface.c
index 14c3e78..162439f 100644
--- a/zebra/interface.c
+++ b/zebra/interface.c
@@ -368,6 +368,8 @@
   struct zebra_if *if_data;
 
   if_data = ifp->info;
+  assert(if_data);
+
   if (if_data->multicast == IF_ZEBRA_MULTICAST_ON)
     if_set_flags (ifp, IFF_MULTICAST);
   else if (if_data->multicast == IF_ZEBRA_MULTICAST_OFF)