diff options
| author | Morgan Stewart <morgan@cumulusnetworks.com> | 2015-09-17 19:04:30 -0400 |
|---|---|---|
| committer | Paul Jakma <paul@quagga.net> | 2015-09-24 15:26:46 +0100 |
| commit | c8394ace7081ef0e71f3d162067c83c2629fc088 (patch) | |
| tree | 7736d5d1ca6d3ea79c3f43e32dccf4baf0fa1094 /zebra/interface.c | |
| parent | 75daab1784f515104ceef747d134981ecee7b0af (diff) | |
| download | quagga-c8394ace7081ef0e71f3d162067c83c2629fc088.tar.bz2 quagga-c8394ace7081ef0e71f3d162067c83c2629fc088.tar.xz | |
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>
Diffstat (limited to 'zebra/interface.c')
| -rw-r--r-- | zebra/interface.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/zebra/interface.c b/zebra/interface.c index 14c3e78a..162439fa 100644 --- a/zebra/interface.c +++ b/zebra/interface.c @@ -368,6 +368,8 @@ if_add_update (struct interface *ifp) 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) |