summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorpaul <paul>2003-08-01 00:24:13 +0000
committerpaul <paul>2003-08-01 00:24:13 +0000
commit106d2fd572c18bebe3dc44031c6d5e56e7b9030f (patch)
treedf226870be1178b347947aecbe7d468022b47a91 /lib
parent12ab19f1863e80134353244967a87805b12fe722 (diff)
2003-08-01 Cougar <cougar@random.ee>
* lib/if.c: (if_cmp_func) new function, compare interface names in alphabetical order. (if_create) Take name as argument and add interface in sorted order. (if_get_by_name),(interface_cmd) fixup calls to if_create - see above. (if_init) register list comparison function. * lib/if.h: Add comparison function, modify if_create prototype. * lib/zclient.c: Modify call to if_create. * ospfd/ospf_interface.c: (ospf_vl_new) modify call to if_create. change sprintf to snprintf. * zebra/kernel_socket.c: (ifm_read) modify call to if_create.
Diffstat (limited to 'lib')
-rw-r--r--lib/if.c69
-rw-r--r--lib/if.h3
-rw-r--r--lib/zclient.c5
3 files changed, 61 insertions, 16 deletions
diff --git a/lib/if.c b/lib/if.c
index f003754a..30de6805 100644
--- a/lib/if.c
+++ b/lib/if.c
@@ -1,3 +1,4 @@
+
/*
* Interface functions.
* Copyright (C) 1997, 98 Kunihiro Ishiguro
@@ -46,6 +47,52 @@ struct if_master
int (*if_delete_hook) (struct interface *);
} if_master;
+/* Compare interface names */
+int
+if_cmp_func (struct interface *ifp1, struct interface *ifp2)
+{
+ unsigned int l1, l2;
+ long int x1, x2;
+ char *p1, *p2;
+ int res;
+
+ p1 = ifp1->name;
+ p2 = ifp2->name;
+
+ while (1) {
+ /* look up to any number */
+ l1 = strcspn(p1, "0123456789");
+ l2 = strcspn(p2, "0123456789");
+
+ /* name lengths are different -> compare names */
+ if (l1 != l2)
+ return (strcmp(p1, p2));
+
+ res = strncmp(p1, p2, l1);
+
+ /* names are different -> compare them */
+ if (res)
+ return res;
+
+ /* with identical name part, go to numeric part */
+
+ p1 += l1;
+ p2 += l1;
+
+ x1 = strtol(p1, &p1, 10);
+ x2 = strtol(p2, &p2, 10);
+
+ /* let's compare numbers now */
+ if (x1 < x2)
+ return -1;
+ if (x1 > x2)
+ return 1;
+
+ /* numbers were equal, lets do it again..
+ (it happens with name like "eth123.456:789") */
+ }
+}
+
/* Create new interface structure. */
struct interface *
if_new ()
@@ -58,13 +105,17 @@ if_new ()
}
struct interface *
-if_create ()
+if_create (char *name, int namelen)
{
struct interface *ifp;
ifp = if_new ();
- listnode_add (iflist, ifp);
+ assert (name);
+ assert (namelen <= (INTERFACE_NAMSIZ + 1));
+ strncpy (ifp->name, name, namelen);
+ ifp->name[INTERFACE_NAMSIZ] = '\0';
+ listnode_add_sort (iflist, ifp);
ifp->connected = list_new ();
ifp->connected->del = (void (*) (void *)) connected_free;
@@ -248,10 +299,7 @@ if_get_by_name (char *name)
ifp = if_lookup_by_name (name);
if (ifp == NULL)
- {
- ifp = if_create ();
- strncpy (ifp->name, name, IFNAMSIZ);
- }
+ ifp = if_create (name, INTERFACE_NAMSIZ);
return ifp;
}
@@ -430,10 +478,7 @@ DEFUN (interface,
ifp = if_lookup_by_name (argv[0]);
if (ifp == NULL)
- {
- ifp = if_create ();
- strncpy (ifp->name, argv[0], INTERFACE_NAMSIZ);
- }
+ ifp = if_create (argv[0], INTERFACE_NAMSIZ);
vty->index = ifp;
vty->node = INTERFACE_NODE;
@@ -803,8 +848,10 @@ if_init ()
iflist = list_new ();
ifaddr_ipv4_table = route_table_init ();
- if (iflist)
+ if (iflist) {
+ iflist->cmp = (int (*)(void *, void *))if_cmp_func;
return;
+ }
memset (&if_master, 0, sizeof if_master);
}
diff --git a/lib/if.h b/lib/if.h
index 9ffe74cf..3fbeed9a 100644
--- a/lib/if.h
+++ b/lib/if.h
@@ -180,8 +180,9 @@ struct connected
#endif /* IFF_LINK2 */
/* Prototypes. */
+int if_cmp_func (struct interface *, struct interface *);
struct interface *if_new (void);
-struct interface *if_create (void);
+struct interface *if_create (char *name, int namelen);
struct interface *if_lookup_by_index (unsigned int);
struct interface *if_lookup_by_name (char *);
struct interface *if_lookup_exact_address (struct in_addr);
diff --git a/lib/zclient.c b/lib/zclient.c
index bb7747fa..bba06ee4 100644
--- a/lib/zclient.c
+++ b/lib/zclient.c
@@ -555,10 +555,7 @@ zebra_interface_add_read (struct stream *s)
/* If such interface does not exist, make new one. */
if (! ifp)
- {
- ifp = if_create ();
- strncpy (ifp->name, ifname_tmp, IFNAMSIZ);
- }
+ ifp = if_create (ifname_tmp, INTERFACE_NAMSIZ);
/* Read interface's index. */
ifp->ifindex = stream_getl (s);