ripd, ripngd: fix null pointer dereference in YANG RPCs

These RPCs should do nothing when RIP/RIPng aren't configured. Signed-off-by: Renato Westphal <[email protected]>
qlyoung · Jan 18, 2019 · bb5d09c · bb5d09c
1 parent 70fc94c
commit bb5d09c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/ripd/rip_northbound.c b/ripd/rip_northbound.c
@@ -1228,6 +1228,9 @@ static int clear_rip_route_rpc(const char *xpath, const struct list *input,
 	struct list *list;
 	struct listnode *listnode;
 
+	if (!rip)
+		return NB_OK;
+
 	/* Clear received RIP routes */
 	for (rp = route_top(rip->table); rp; rp = route_next(rp)) {
 		list = rp->info;

diff --git a/ripngd/ripng_northbound.c b/ripngd/ripng_northbound.c
@@ -785,6 +785,9 @@ static int clear_ripng_route_rpc(const char *xpath, const struct list *input,
 	struct list *list;
 	struct listnode *listnode;
 
+	if (!ripng)
+		return NB_OK;
+
 	/* Clear received RIPng routes */
 	for (rp = agg_route_top(ripng->table); rp; rp = agg_route_next(rp)) {
 		list = rp->info;