proxmox-mirrors/mirror_frr
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_frr synced 2025-08-13 21:10:28 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

lib: if_lookup_by_name should be more robust to null argument

Browse Source 
* if.c: (if_lookup_by_name) shouldn't crash just cause we got a NULL name
This commit is contained in:
Paul Jakma 2009-08-06 12:08:50 +01:00
parent 27f5dc8120
commit 3e4ee9591a
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
lib/if.c
View File

@ -205,7 +205,8 @@ ifname2ifindex (const char *name)
{ {
struct interface *ifp; struct interface *ifp;
return ((ifp = if_lookup_by_name(name)) != NULL) ? ifp->ifindex : 0; return ((ifp = if_lookup_by_name(name)) != NULL) ? ifp->ifindex
: IFINDEX_INTERNAL;
} }
/* Interface existance check by interface name. */ /* Interface existance check by interface name. */
@ -214,12 +215,13 @@ if_lookup_by_name (const char *name)
{ {
struct listnode *node; struct listnode *node;
struct interface *ifp; struct interface *ifp;
for (ALL_LIST_ELEMENTS_RO (iflist, node, ifp)) if (name)
{ for (ALL_LIST_ELEMENTS_RO (iflist, node, ifp))
if (strcmp(name, ifp->name) == 0) {
return ifp; if (strcmp(name, ifp->name) == 0)
} return ifp;
}
return NULL; return NULL;
} }