matthieu/frr
1
0
Fork 0
forked from Mirror/frr
Code Pull requests Activity

lib: guard close() in netns code

Browse source 
clang-SA complains that ns->fd could be invalid.  Add a guard.

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
This commit is contained in:
David Lamparter 2024-10-16 12:42:02 +02:00
parent 40d4a47c0e
commit a67df2a17f
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
lib/netns_linux.c
View file

@ -258,7 +258,7 @@ static void ns_disable_internal(struct ns *ns)
if (ns_master.ns_disable_hook) if (ns_master.ns_disable_hook)
(*ns_master.ns_disable_hook)(ns); (*ns_master.ns_disable_hook)(ns);
if (have_netns()) if (have_netns() && ns->fd >= 0)
close(ns->fd); close(ns->fd);
ns->fd = -1; ns->fd = -1;