summaryrefslogtreecommitdiff
path: root/lib
diff options
context:
space:
mode:
authorhasso <hasso>2003-06-05 17:50:01 +0000
committerhasso <hasso>2003-06-05 17:50:01 +0000
commita939514c59fe1a56a820e82bc59d82c721076c03 (patch)
tree85a2d3d1d3e0a4f35d0865778c7522b8569b40f8 /lib
parent41908818b8f446befc3654fa5bbc98c6c0af92a3 (diff)
Patch from Cougar - sort iflist by name.
Diffstat (limited to 'lib')
-rw-r--r--lib/if.c69
-rw-r--r--lib/if.h2
-rw-r--r--lib/zclient.c5
3 files changed, 60 insertions, 16 deletions
diff --git a/lib/if.c b/lib/if.c
index f003754a..cb2d086e 100644
--- a/lib/if.c
+++ b/lib/if.c
@@ -46,6 +46,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 +104,18 @@ if_new ()
}
struct interface *
-if_create ()
+if_create (char *name, int namelen)
{
struct interface *ifp;
ifp = if_new ();
- listnode_add (iflist, ifp);
+ if (name) {
+ strncpy (ifp->name, name, namelen);
+ listnode_add_sort (iflist, ifp);
+ } else {
+ listnode_add (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, IFNAMSIZ);
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..c6469ea6 100644
--- a/lib/if.h
+++ b/lib/if.h
@@ -181,7 +181,7 @@ struct connected
/* Prototypes. */
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..f6be5138 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, IFNAMSIZ);
/* Read interface's index. */
ifp->ifindex = stream_getl (s);