From ab80e474f26f2e982bc15dd2506ff2eeef129dcc Mon Sep 17 00:00:00 2001 From: Rafael Zalamena Date: Thu, 19 Jan 2023 10:32:18 -0300 Subject: [PATCH] zebra: fix possible null dereference Don't attempt to dereference `ifp` directly if it might be null: there is a check right before this usage: `ifp ? ifp->info : NULL`. In this context it should be safe to assume `ifp` is not NULL because the only caller of this function checks that for this `ifindex`. For consistency we'll check for null anyway in case this ever changes (and with this the coverity scan warning gets silenced). Found by Coverity Scan (CID 1519776) Signed-off-by: Rafael Zalamena --- zebra/interface.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/zebra/interface.c b/zebra/interface.c index 87bb49042a..59563834ef 100644 --- a/zebra/interface.c +++ b/zebra/interface.c @@ -1436,7 +1436,8 @@ static void zebra_if_netconf_update_ctx(struct zebra_dplane_ctx *ctx, if (IS_ZEBRA_DEBUG_KERNEL) zlog_debug( "%s: if %s(%u) zebra info pointer is NULL", - __func__, ifp->name, ifp->ifindex); + __func__, ifp ? ifp->name : "(null)", + ifp ? ifp->ifindex : ifindex); return; } if (afi == AFI_IP) { -- 2.39.5