From 98954844ae56d142e96341d5dff959ec5518111e Mon Sep 17 00:00:00 2001 From: Paul Jakma Date: Sun, 15 Oct 2006 23:33:50 +0000 Subject: [lib] Handle upgrade from SUNWzebra to Quagga for 'interface fooX:Y' commands 2006-10-14 Paul Jakma * if.c: (general) Handle upgrades from SUNWzebra, which tried to track each logical interface as a seperate struct interface, to Quagga, which assigns only one struct interface per ifindex. (if_sunwzebra_get) Try decompose a logical interface name (fooX:Y) to the 'primary' name (fooX), for Solaris. (interface_cmd) Use if_sunwzebra_get on Solaris. --- lib/ChangeLog | 10 ++++++++++ lib/if.c | 48 ++++++++++++++++++++++++++++++++++++++++++++++-- 2 files changed, 56 insertions(+), 2 deletions(-) (limited to 'lib') diff --git a/lib/ChangeLog b/lib/ChangeLog index 00ba44e0..f076e2f0 100644 --- a/lib/ChangeLog +++ b/lib/ChangeLog @@ -1,3 +1,13 @@ +2006-10-14 Paul Jakma + + * if.c: (general) Handle upgrades from SUNWzebra, which tried + to track each logical interface as a seperate struct + interface, to Quagga, which assigns only one struct interface + per ifindex. + (if_sunwzebra_get) Try decompose a logical interface name + (fooX:Y) to the 'primary' name (fooX), for Solaris. + (interface_cmd) Use if_sunwzebra_get on Solaris. + 2006-09-26 Pierre-Yves Ritschard * smux.c: (smux_open,smux_trap,smux_register) Fix various diff --git a/lib/if.c b/lib/if.c index c43d9564..4493cc74 100644 --- a/lib/if.c +++ b/lib/if.c @@ -501,9 +501,49 @@ DEFUN (no_interface_desc, return CMD_SUCCESS; } + +#ifdef SUNOS_5 +/* Need to handle upgrade from SUNWzebra to Quagga. SUNWzebra created + * a seperate struct interface for each logical interface, so config + * file may be full of 'interface fooX:Y'. Solaris however does not + * expose logical interfaces via PF_ROUTE, so trying to track logical + * interfaces can be fruitless, for that reason Quagga only tracks + * the primary IP interface. + * + * We try accomodate SUNWzebra by: + * - looking up the interface name, to see whether it exists, if so + * its useable + * - for protocol daemons, this could only because zebra told us of + * the interface + * - for zebra, only because it learnt from kernel + * - if not: + * - search the name to see if it contains a sub-ipif / logical interface + * seperator, the ':' char. If it does: + * - text up to that char must be the primary name - get that name. + * if not: + * - no idea, just get the name in its entirety. + */ +static struct interface * +if_sunwzebra_get (const char *name, size_t nlen) +{ + struct interface *ifp; + size_t seppos = 0; - -/* See also wrapper function zebra_interface() in zebra/interface.c */ + if ( (ifp = if_lookup_by_name_len(name, nlen)) != NULL) + return ifp; + + /* hunt the primary interface name... */ + while (seppos < nlen && name[seppos] != ':') + seppos++; + + /* Wont catch seperator as last char, e.g. 'foo0:' but thats invalid */ + if (seppos < nlen) + return if_get_by_name_len (name, seppos); + else + return if_get_by_name_len (name, nlen); +} +#endif /* SUNOS_5 */ + DEFUN (interface, interface_cmd, "interface IFNAME", @@ -521,7 +561,11 @@ DEFUN (interface, return CMD_WARNING; } +#ifdef SUNOS_5 + ifp = if_sunwzebra_get (argv[0], sl); +#else ifp = if_get_by_name_len(argv[0], sl); +#endif /* SUNOS_5 */ vty->index = ifp; vty->node = INTERFACE_NODE; -- cgit v1.2.1