)]}' { "log": [ { "commit": "471ea39ce54537194ff2f22420589ddec5ef5ada", "tree": "65b04ef42d4ece7e4a44d230fcc0a15f388755c4", "parents": [ "5a5702fac5458d63d7a099c4db7ac387afefd2de" ], "author": { "name": "Feng Lu", "email": "lu.feng@6wind.com", "time": "Fri May 22 11:40:00 2015 +0200" }, "committer": { "name": "David Lamparter", "email": "equinox@opensourcerouting.org", "time": "Tue Jun 02 06:59:36 2015 +0200" }, "message": "lib, zebra, vtysh: configure an interface in non-default VRF\n\nIntroduce a new command \"interface IFNAME vrf N\" to configure an\ninterface in the non-default VRF.\n\nTill now, only zebra uses this command. Other daemons will install\nthe command when they support multiple VRFs.\n\nSigned-off-by: Feng Lu \u003clu.feng@6wind.com\u003e\nReviewed-by: Alain Ritoux \u003calain.ritoux@6wind.com\u003e\nSigned-off-by: Nicolas Dichtel \u003cnicolas.dichtel@6wind.com\u003e\nAcked-by: Vincent JARDIN \u003cvincent.jardin@6wind.com\u003e\nSigned-off-by: David Lamparter \u003cequinox@opensourcerouting.org\u003e\n" }, { "commit": "5a5702fac5458d63d7a099c4db7ac387afefd2de", "tree": "0547fea145cb10fb9ed6d9493963d374556eb65f", "parents": [ "126215c1238eb42cc92d23aefbe1fac3b204438f" ], "author": { "name": "Feng Lu", "email": "lu.feng@6wind.com", "time": "Fri May 22 11:39:59 2015 +0200" }, "committer": { "name": "David Lamparter", "email": "equinox@opensourcerouting.org", "time": "Tue Jun 02 06:59:28 2015 +0200" }, "message": "lib: move the interface list into \"struct vrf\"\n\nAn interface belongs to a specific VRF. So move the interface list\ninto the \"struct vrf\".\n\n* vrf.c/vrf.h:\n - add a new member \"struct list *iflist\" to the the \"struct vrf\";\n - call if_init() in vrf_new();\n - call if_terminate() in vrf_delete();\n - add utilities to access the interface list and VRF ID in the\n specified VRF.\n\n* if.c/if.h:\n - the global \"iflist\" now only exists for the default VRF;\n - the global \"if_master\" is initialized on the definition;\n - in if_create(), the interface is added into the list in the\n specified VRF; if the VRF does not exist, create one;\n - add parameters to if_init()/if_terminate() so that the\n interface list in the VRF can be initialized/destroyed;\n - in if_dump_all() scan the interfaces in all the VRFs;\n - add a command \"show address vrf N\" to show addresses in a\n specified VRF;\n - add a command \"show address vrf all\" to show addresses in all\n VRFs;\n - new APIs ifxxx_vrf() are added to access an interface in a\n specified VRF.\n\nThe old interface APIs (the global variable \"iflist\" and the API\nfunctions) are not changed to keep the backward compatibility.\nThe new APIs are used in the daemons which support multiple VRFs\n(till now only zebra).\n\nSigned-off-by: Feng Lu \u003clu.feng@6wind.com\u003e\nReviewed-by: Alain Ritoux \u003calain.ritoux@6wind.com\u003e\nSigned-off-by: Nicolas Dichtel \u003cnicolas.dichtel@6wind.com\u003e\nAcked-by: Vincent JARDIN \u003cvincent.jardin@6wind.com\u003e\nSigned-off-by: David Lamparter \u003cequinox@opensourcerouting.org\u003e\n" }, { "commit": "2fc97f6335dd5d7df2c285b363c6ef56bb98dcf8", "tree": "22361093e3faf9587d055474c1163796aae94194", "parents": [ "41f44a23e86a65a5cad7e5e8cafd7e935f153232" ], "author": { "name": "Feng Lu", "email": "lu.feng@6wind.com", "time": "Fri May 22 11:39:57 2015 +0200" }, "committer": { "name": "David Lamparter", "email": "equinox@opensourcerouting.org", "time": "Tue Jun 02 06:58:12 2015 +0200" }, "message": "lib, zebra: add \"vrf_id\" into the \"struct interface\"\n\nLater, an interface will belong to a specific VRF. Now we add a\nproperty \"vrf_id\" to the \"struct interface\", and keep it as the\ndefault value 0.\n\nThis property is shown when displaying interfaces information.\nIt is also added in some logs.\n\nThis is just the preparation to move the interace list into the\n\"struct vrf\". The main logic is not changed.\n\nSigned-off-by: Feng Lu \u003clu.feng@6wind.com\u003e\nReviewed-by: Alain Ritoux \u003calain.ritoux@6wind.com\u003e\nSigned-off-by: Nicolas Dichtel \u003cnicolas.dichtel@6wind.com\u003e\nAcked-by: Vincent JARDIN \u003cvincent.jardin@6wind.com\u003e\nSigned-off-by: David Lamparter \u003cequinox@opensourcerouting.org\u003e\n" }, { "commit": "b81e97a8a7f85ecc7489dc8a7c7b9d403d9c4bc6", "tree": "48e2c8492f01cc177a6691878cf4e5a329fe4d2d", "parents": [ "17d003da963d6294e5ab95c690de285eccf0bac0" ], "author": { "name": "Dinesh Dutt", "email": "ddutt@cumulusnetworks.com", "time": "Sat Aug 24 07:55:50 2013 +0000" }, "committer": { "name": "David Lamparter", "email": "equinox@opensourcerouting.org", "time": "Thu Nov 07 18:15:43 2013 -0800" }, "message": "ospf6d: correct nexthop through directly connected networks\n\nThis is implementing this part of RFC 2328:\n\nThis is the \"first case\", see below,\n\n16.1.1. The next hop calculation\n\n...\nIf there is at least one intervening router in the current\nshortest path between the destination and the root, the\ndestination simply inherits the set of next hops from the\nparent. Otherwise, there are two cases. In the first case,\nthe parent vertex is the root (the calculating router\nitself). This means that the destination is either a\ndirectly connected network or directly connected router.\nThe outgoing interface in this case is simply the OSPF\ninterface connecting to the destination network/router.\n...\n\nThe current Quagga code always tries to inherit the nexthop from\na parent vertex, but does not cover the case that the destination\nis directly connected to the root vertex. This patch adds support\nfor that case.\n\nSigned-off-by: James Li \u003cjli at cumulusnetworks.com\u003e\nReviewed-by: Dinesh G Dutt \u003cddutt at cumulusnetworks.com\u003e\nSigned-off-by: David Lamparter \u003cequinox@opensourcerouting.org\u003e\n" }, { "commit": "f7f740fe58fb838fc87e82dc7e1e2d4e5ccf085c", "tree": "5f3d9c866aaf7f8f23aaa5956eb7323d49ef472f", "parents": [ "c7df92de2dc91d0a934cf892e543728cb1a10849" ], "author": { "name": "Christian Franke", "email": "chris@opensourcerouting.org", "time": "Thu Jan 24 14:04:48 2013 +0000" }, "committer": { "name": "David Lamparter", "email": "equinox@opensourcerouting.org", "time": "Thu Sep 19 17:51:16 2013 +0200" }, "message": "zebra: add ZEBRA_IFC_QUEUED to keep track of kernel state\n\nAs there are timeframes when we don\u0027t get a notification from the kernel\nabout new addresses. (e.g. while Linux performs IPv6 DAD), we need to\nhave some information whether an address has been sent to the kernel or\nnot.\n\nOne case where this is relevant would be a user adding an IPv6 address,\nbut deleting it before DAD has been complete. With the next patch which\nremoves some (ill assuming) synchronous parts in address setup,\nipv6_address_uninstall would not know whether or not it has to actually\ndelete the prefix from the kernel. Resolving these windows where we lack\ninformation is what the flag ZEBRA_IFC_QUEUED is intended for.\n\nSigned-off-by: Christian Franke \u003cchris@opensourcerouting.org\u003e\nSigned-off-by: David Lamparter \u003cequinox@opensourcerouting.org\u003e\n" }, { "commit": "ca3ccd8748434719e4670ce812d1310013fad518", "tree": "3fae89070206bc47704a456219350c34b5f8ea54", "parents": [ "8d083b9ec5bb0375ebb6d8b2b05c848febd92cb5" ], "author": { "name": "David Lamparter", "email": "equinox@opensourcerouting.org", "time": "Wed Sep 26 14:52:39 2012 +0200" }, "committer": { "name": "David Lamparter", "email": "equinox@opensourcerouting.org", "time": "Wed Jan 16 01:45:57 2013 +0100" }, "message": "zebra: fix sockaddr_dl length assumptions (BZ#737)\n\nQuagga makes bad assumptions about sockaddr_dl (on NetBSD, but possibly\non other systems as well). Particularly, sizeof(struct sockaddr_dl)\nreturns a size that does not include the full sdl_data field, leading to\nnot enough data being copied. This breaks IPv6 RAs in particular, as\na broken mac address from sockaddr_dl will be included in the packets.\n\nFrom: Matthias-Christian Ott \u003cott@mirix.org\u003e\nTested-by: Uwe Toenjes \u003c6bone@6bone.informatik.uni-leipzig.de\u003e\n[further simplified + more comments]\nSigned-off-by: David Lamparter \u003cequinox@opensourcerouting.org\u003e\n" }, { "commit": "4bd045d5dd28653af899c01d6f0c413d74677114", "tree": "f1e633f50e4dc19b4b0cda9dc6e6feb46e3bab46", "parents": [ "c8e7b8959051bf525183bdbff605e54b4b532e2c" ], "author": { "name": "Tom Goff", "email": "thomas.goff@boeing.com", "time": "Wed Nov 10 13:00:54 2010 -0800" }, "committer": { "name": "Paul Jakma", "email": "paul@quagga.net", "time": "Mon Mar 21 13:30:54 2011 +0000" }, "message": "lib: Add a function to delete all interfaces\n\nif.c: (if_terminate) This adds a cleanup function that can be called\n when a daemon exits, similar to vty_terminate().\n" }, { "commit": "6f0e3f6e17687eb25b7b77c4fdc8324837d4700f", "tree": "8a94d9212060f0f2ee7cbd0079c2687eed7348dc", "parents": [ "08d3d5b398ae81de7659509f159e814d1bbd4375" ], "author": { "name": "Paul Jakma", "email": "paul.jakma@sun.com", "time": "Thu May 10 02:38:51 2007 +0000" }, "committer": { "name": "Paul Jakma", "email": "paul.jakma@sun.com", "time": "Thu May 10 02:38:51 2007 +0000" }, "message": "[autoconf] bugs 162,303,178: Fix \u0027present but can not be compiled\u0027 warnings\n\n2007-05-09 Paul Jakma \u003cpaul.jakma@sun.com\u003e\n\n\t* configure.ac: sys/conf.h depends on sys/param.h, at least on\n\t FBSD 6.2.\n\t (bug #363) Should check for in_pktinfo for IRDP\n\n2006-05-27 Paul Jakma \u003cpaul.jakma@sun.com\u003e\n\n\t* configure.ac: General cleanup of header and type checks, introducing\n\t an internal define, QUAGGA_INCLUDES, to build up a list of\n\t stuff to include so as to avoid \u0027present but cant be compiled\u0027\n\t warnings.\n\t Misc additional checks of things missing according to autoscan.\n\t Add LIBM, for bgpd\u0027s use of libm, so as to avoid burdening\n\t LIBS, and all the binaries, with libm linkage.\n\t Remove the bad practice of using m4 changequote(), just\n\t quote the []\u0027s in the case statements properly.\n\t This should fix bugs 162, 303 and 178.\n\t* */*.{c,h}: Update all HAVE_* to the standard autoconf namespaced\n\t HAVE_* defines. I.e. HAVE_SA_LEN -\u003e HAVE_STRUCT_SOCKADDR_SA_LEN,\n\t* bgpd/Makefile.am: Add LIBM to bgpd\u0027s LDADD, for pow().\n" }, { "commit": "e4529636b77124285cca96a62799d0ff6a7addeb", "tree": "a8a50de7413833d5277ed7309248e03719e9efd3", "parents": [ "43cd33a44e010f818633b7f144b5a0be352b41e7" ], "author": { "name": "Andrew J. Schorr", "email": "ajschorr@alumni.princeton.edu", "time": "Tue Dec 12 19:18:21 2006 +0000" }, "committer": { "name": "Andrew J. Schorr", "email": "ajschorr@alumni.princeton.edu", "time": "Tue Dec 12 19:18:21 2006 +0000" }, "message": "[PtP over ethernet] New peer flag allows much more addressing flexibility\n\n2006-12-12 Andrew J. Schorr \u003cajschorr@alumni.princeton.edu\u003e\n\n\t* if.h: (struct connected) Add new ZEBRA_IFA_PEER flag indicating\n\t whether a peer address has been configured. Comment now shows\n\t the new interpretation of the destination addr: if ZEBRA_IFA_PEER\n\t is set, then it must contain the destination address, otherwise\n\t it may contain the broadcast address or be NULL.\n\t (CONNECTED_DEST_HOST,CONNECTED_POINTOPOINT_HOST) Remove obsolete\n\t macros that were specific to IPv4 and not fully general.\n\t (CONNECTED_PEER) New macro to check ZEBRA_IFA_PEER flag.\n\t (CONNECTED_PREFIX) New macro giving the prefix to insert into\n\t the RIB: if CONNECTED_PEER, then use the destination (peer) address,\n\t else use the address field.\n\t (CONNECTED_ID) New macro to come up with an identifying address\n\t for the struct connected.\n\t* if.c: (if_lookup_address, connected_lookup_address) Streamline\n\t logic with new CONNECTED_PREFIX macro.\n\t* prefix.h: (PREFIX_COPY_IPV4, PREFIX_COPY_IPV6) New macros\n\t for better performance than the general prefix_copy function.\n\t* zclient.c: (zebra_interface_address_read) For non-null destination\n\t addresses, set prefixlen to equal the address prefixlen. This\n\t is needed to get the new CONNECTED_PREFIX macro to work properly.\n\t* connected.c: (connected_up_ipv4, connected_down_ipv4,\n\t connected_up_ipv6, connected_down_ipv6) Simplify logic using the\n\t new CONNECTED_PREFIX macro.\n\t (connected_add_ipv4) Set prefixlen in destination addresses (required\n\t by the CONNECTED_PREFIX macro). Use CONNECTED_PEER macro instead\n\t of testing for IFF_POINTOPOINT. Delete invalid warning message.\n\t Warn about cases where the ZEBRA_IFA_PEER is set but no\n\t destination address has been supplied (and turn off the flag).\n\t (connected_add_ipv6) Add new flags argument so callers may set\n\t the ZEBRA_IFA_PEER flag. If peer/broadcast address satisfies\n\t IN6_IS_ADDR_UNSPECIFIED, then reject it with a warning.\n\t Set prefixlen in destination address so CONNECTED_PREFIX will work.\n\t* connected.h: (connected_add_ipv6) Add new flags argument so\n\t callers may set the ZEBRA_IFA_PEER flag.\n\t* interface.c: (connected_dump_vty) Use CONNECTED_PEER macro\n\t to decide whether the destination address is a peer or broadcast\n\t address (instead of checking IFF_BROADCAST and IFF_POINTOPOINT).\n\t* if_ioctl.c: (if_getaddrs) Instead of setting a peer address\n\t only when the IFF_POINTOPOINT is set, we now accept a peer\n\t address whenever it is available and not the same as the local\n\t address. Otherwise (no peer address assigned), we check\n\t for a broadcast address (regardless of the IFF_BROADCAST flag).\n\t And must now pass a flags value of ZEBRA_IFA_PEER to\n\t connected_add_ipv4 when a peer address is assigned.\n\t The same new logic is used with the IPv6 code as well (and we\n\t pass the new flags argument to connected_add_ipv6).\n\t (if_get_addr) Do not bother to check IFF_POINTOPOINT: just\n\t issue the SIOCGIFDSTADDR ioctl and see if we get back\n\t a peer address not matching the local address (and set\n\t the ZEBRA_IFA_PEER in that case). If there\u0027s no peer address,\n\t try to grab SIOCGIFBRDADDR regardless of whether IFF_BROADCAST is set.\n\t* if_ioctl_solaris.c: (if_get_addr) Just try the SIOCGLIFDSTADDR ioctl\n\t without bothering to check the IFF_POINTOPOINT flag. And if\n\t no peer address was found, just try the SIOCGLIFBRDADDR ioctl\n\t without checking the IFF_BROADCAST flag. Call connected_add_ipv4\n\t and connected_add_ipv6 with appropriate flags.\n\t* if_proc.c: (ifaddr_proc_ipv6) Must pass new flags argument to\n\t connected_add_ipv6.\n\t* kernel_socket.c: (ifam_read) Must pass new flags argument to\n\t connected_add_ipv6.\n\t* rt_netlink.c: (netlink_interface_addr) Copy logic from iproute2\n\t to determine local and possible peer address (so there\u0027s no longer\n\t a test for IFF_POINTOPOINT). Set ZEBRA_IFA_PEER flag appropriately.\n\t Pass new flags argument to connected_add_ipv6.\n\t (netlink_address) Test !CONNECTED_PEER instead of if_is_broadcast\n\t to determine whether the connected destination address is a\n\t broadcast address.\n\t* bgp_nexthop.c: (bgp_connected_add, bgp_connected_delete)\n\t Simplify logic by using new CONNECTED_PREFIX macro.\n\t* ospf_interface.c: (ospf_if_is_configured, ospf_if_lookup_by_prefix,\n\t ospf_if_lookup_recv_if) Simplify logic using new CONNECTED_PREFIX\n\t macro.\n\t* ospf_lsa.c: (lsa_link_ptop_set) Using the new CONNECTED_PREFIX\n\t macro, both options collapse into the same code.\n\t* ospf_snmp.c: (ospf_snmp_if_update) Simplify logic using new\n\t CONNECTED_ID macro.\n\t (ospf_snmp_is_if_have_addr) Simplify logic using new CONNECTED_PREFIX\n\t macro.\n\t* ospf_vty.c: (show_ip_ospf_interface_sub) Use new CONNECTED_PEER macro\n\t instead of testing the IFF_POINTOPOINT flag.\n\t* ospfd.c: (ospf_network_match_iface) Use new CONNECTED_PEER macro\n\t instead of testing with if_is_pointopoint. And add commented-out\n\t code to implement alternative (in my opinion) more elegant behavior\n\t that has no special-case treatment for PtP addresses.\n\t (ospf_network_run) Use new CONNECTED_ID macro to simplify logic.\n\t* rip_interface.c: (rip_interface_multicast_set) Use new CONNECTED_ID\n\t macro to simplify logic.\n\t (rip_request_interface_send) Fix minor bug: ipv4_broadcast_addr does\n\t not give a useful result if prefixlen is 32 (we require a peer\n\t address in such cases).\n\t* ripd.c: (rip_update_interface) Fix same bug as above.\n" }, { "commit": "9c37851eac4e75cda515761cae7c63309add30ea", "tree": "3fcf98573c83f117f1e0219fcab8e160af70e56a", "parents": [ "0f38dc4d4729b562551f8f53e7343f5a53b2dbf5" ], "author": { "name": "Andrew J. Schorr", "email": "ajschorr@alumni.princeton.edu", "time": "Sun May 21 04:04:49 2006 +0000" }, "committer": { "name": "Andrew J. Schorr", "email": "ajschorr@alumni.princeton.edu", "time": "Sun May 21 04:04:49 2006 +0000" }, "message": "[interface configuration] Try to avoid losing address info after shutdown.\n2006-05-21 Andrew J. Schorr \u003cajschorr@alumni.princeton.edu\u003e\n\n\t* if.h: (struct connected) Document the meaning of the\n\t ZEBRA_IFC_REAL and ZEBRA_IFC_CONFIGURED flags.\n\t* connected.c: (connected_withdraw) Do not delete the connected\n\t address if the ZEBRA_IFC_CONFIGURED flag is set.\n\t (connected_add_ipv4,connected_add_ipv6) Before calling\n\t connected_withdraw, unset the ZEBRA_IFC_CONFIGURED flag\n\t on the superseded connected structure.\n" }, { "commit": "478ccfd61bd61fd6925c84811c2c0e893a38a38b", "tree": "0babd52f1f13b4de28d29836f5e4f3f0cdc4802b", "parents": [ "3e0c78ef8a8fb80f52a36dab99d76a5913281e4c" ], "author": { "name": "Paul Jakma", "email": "paul.jakma@sun.com", "time": "Mon Mar 06 18:18:37 2006 +0000" }, "committer": { "name": "Paul Jakma", "email": "paul.jakma@sun.com", "time": "Mon Mar 06 18:18:37 2006 +0000" }, "message": "[lib] export show_address_cmd debug command in if.h\n\n2006-03-06 Paul Jakma \u003cpaul.jakma@sun.com\u003e\n\n\t* if.h: export show_address_cmd, for anyone who wishes to use\n\t it.\n" }, { "commit": "c77d4546619f9d3e0ccfac5acbc0ebb42658a61f", "tree": "007f6c4e2c85e4df115c39011aa7ad3fb379f434", "parents": [ "44316fef4d93b3d94cf45663ce35797d32acce71" ], "author": { "name": "paul", "email": "paul", "time": "Wed Jan 11 01:59:04 2006 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Wed Jan 11 01:59:04 2006 +0000" }, "message": "[zserv] Update interface flags to 8 bytes wide.\n\n2006-01-11 Paul Jakma \u003cpaul.jakma@sun.com\u003e\n\n\t* if.h: (struct interface) expand flags to 8 bytes.\n\t* zclient.c: (zebra_interface_{add,state}_read) stream read of\n\t interface flags now need to use stream_getq.\n\t (zebra_interface_if_set_value) ditto\n\n2006-01-11 Paul Jakma \u003cpaul.jakma@sun.com\u003e\n\n\t* zserv.c: (zsend_interface_{add,delete,update}) if flags are\n\t 8 bytes now, update to write out with stream_putq.\n" }, { "commit": "8cc4198f9fabe5f10f5a773de1503d82f33a01fb", "tree": "77045da709ff66629bd12029b9ee17700360909b", "parents": [ "e7fe8c88c3d552400e1ae3ae9243319ab95d6f2d" ], "author": { "name": "paul", "email": "paul", "time": "Fri May 06 21:25:49 2005 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Fri May 06 21:25:49 2005 +0000" }, "message": "2005-05-06 Paul Jakma \u003cpaul@dishone.st\u003e\n\n\t* (general) extern and static\u0027ification of functions in code and\n\t header.\n\t Cleanup any definitions with unspecified arguments.\n\t Add casts for callback assignments where the callback is defined,\n\t typically, as passing void *, but the function being assigned has\n\t some other pointer type defined as its argument, as gcc complains\n\t about casts from void * to X* via function arguments.\n\t Fix some old K\u0026R style function argument definitions.\n\t Add noreturn gcc attribute to some functions, as appropriate.\n\t Add unused gcc attribute to some functions (eg ones meant to help\n\t while debugging)\n\t Add guard defines to headers which were missing them.\n\t* command.c: (install_node) add const qualifier, still doesnt shut\n\t up the warning though, because of the double pointer.\n\t (cmp_node) ditto\n\t* keychain.c: (key_str2time) Add GET_LONG_RANGE() macro, derived\n\t fromn vty.h ones to fix some of the (long) \u003c 0 warnings.\n\t* thread.c: (various) use thread_empty\n\t (cpu_record_hash_key) should cast to uintptr_t, a stdint.h type\n\t* vty.h: Add VTY_GET_IPV4_ADDRESS and VTY_GET_IPV4_PREFIX so they\n\t removed from ospfd/ospf_vty.h\n\t* zebra.h: Move definition of ZEBRA_PORT to here, to remove\n\t dependence of lib on zebra/zserv.h\n" }, { "commit": "08dbfb691d8e03c7200138d55447fc29916e0362", "tree": "54d8718c7bdb035197cebcedef7d3de4f0038d55", "parents": [ "e4319de388b748ba66551f33fecdf7446e754b80" ], "author": { "name": "ajs", "email": "ajs", "time": "Sun Apr 03 03:40:52 2005 +0000" }, "committer": { "name": "ajs", "email": "ajs", "time": "Sun Apr 03 03:40:52 2005 +0000" }, "message": "2005-04-02 Andrew J. Schorr \u003cajschorr@alumni.princeton.edu\u003e\n\n\t* if_ioctl.c: (interface_list_ioctl) Use if_get_by_name_len.\n\t* if_proc.c: (ifaddr_proc_ipv6) Increase size of ifname buffer to\n\t avoid overflow.\n\t* kernel_socket.c: (ifan_read) Use if_get_by_name_len.\n\t* if.h: Fix comments to reflect that if_lookup_by_name and\n\t if_get_by_name now require the argument strings to be NUL-terminated.\n\t* if.c: (if_lookup_by_name) Compare using strcmp.\n\t (if_get_by_name) Pass strlen(ifname) as 2nd arg to if_create.\n" }, { "commit": "a349198fd3e4e5692cdc91223f8153cb53c086ce", "tree": "e5b366a8ab7ab9ed1c9670faf1f18eef26877ade", "parents": [ "f695b01ff8e9aebc46bdf41f104ad4abbb0ef59e" ], "author": { "name": "ajs", "email": "ajs", "time": "Sat Apr 02 22:50:38 2005 +0000" }, "committer": { "name": "ajs", "email": "ajs", "time": "Sat Apr 02 22:50:38 2005 +0000" }, "message": "2005-04-02 Andrew J. Schorr \u003cajschorr@alumni.princeton.edu\u003e\n\n\t* if.h: (if_lookup_by_name_len, if_get_by_name_len) New functions.\n\t* if.c: (if_lookup_by_name_len, if_get_by_name_len) New functions.\n\t (if_get_by_name) Tighten up code.\n\t (interface) Use new function if_get_by_name_len.\n\t* zclient.c: (zebra_interface_add_read) Use new if_get_by_name_len\n\t function.\n\t (zebra_interface_state_read) Use new if_lookup_by_name_len function.\n\t* kernel_socket.c: (ifm_read) Use new if_lookup_by_name_len function\n\t to save a memcpy.\n\t* if_ioctl_solaris.c: (interface_list_ioctl) Fix subtle bug with new\n\t if_get_by_name_len function.\n\t* ospf_interface.c: (ospf_vl_new) Use strnlen to fix call to if_create.\n" }, { "commit": "d2fc88962a9a494ecb34167871bb9e7273a25d33", "tree": "304a1c1a744972bc10b1f677a134907c404b361d", "parents": [ "386e61502d391adb985ac1487825349ec85e6a8c" ], "author": { "name": "ajs", "email": "ajs", "time": "Sat Apr 02 18:38:43 2005 +0000" }, "committer": { "name": "ajs", "email": "ajs", "time": "Sat Apr 02 18:38:43 2005 +0000" }, "message": "2005-04-02 Andrew J. Schorr \u003cajschorr@alumni.princeton.edu\u003e\n\n\tFix problems when netlink interfaces are renamed (same ifindex used\n\tfor a new interface). Start cleaning up some problems with the way\n\tinterface names are handled.\n\t* interface.c: (if_new_intern_ifindex) Remove obsolete function.\n\t (if_delete_update) After distributing the interface deletion message,\n\t set ifp-\u003eifindex to IFINDEX_INTERNAL.\n\t (if_dump_vty) Detect pseudo interface by checking if ifp-\u003eifindex is\n\t IFINDEX_INTERNAL.\n\t (zebra_interface) Check return code from interface_cmd.func.\n\t Do not set internal ifindex values to if_new_intern_ifindex(),\n\t since we now use IFINDEX_INTERNAL for all pseudo interfaces.\n\t* kernel_socket.c: (ifm_read) Fix code and comments to reflect that\n\t all internal interfaces now have ifp-\u003eifindex set to IFINDEX_INTERNAL.\n * rt_netlink.c: (set_ifindex) New function used to update ifp-\u003eifindex.\n\t Detects interface rename events by checking if that ifindex is already\n\t being used. If it is, delete the old interface before assigning\n\t the ifindex to the new interface.\n\t (netlink_interface, netlink_link_change) Call set_ifindex to update\n\t the ifindex.\n\t* if.h: Remove define for IFINDEX_INTERNBASE and add define\n\t IFINDEX_INTERNAL 0, since all internal (i.e. non-kernel) pseudo-\n\t interfaces should have ifindex set to 0.\n\t (if_new) Remove function.\n\t (if_delete_retain) New function to delete an interface without\n\t removing from iflist and freeing the structure.\n\t (ifname2ifindex) New function.\n\t* if.c: (if_new) Remove function (absorb into if_create).\n\t (if_create) Replace function if_new with call to calloc.\n\t Set ifp-\u003eifindex to IFINDEX_INTERNAL. Fix off-by-one error\n\t in assert to check length of interface name. Add error message\n\t if interface with this name already exists.\n\t (if_delete_retain) New function to delete an interface without\n\t removing from iflist and freeing the structure.\n\t (if_delete) Implement with help of if_delete_retain.\n\t (ifindex2ifname) Reimplement using if_lookup_by_index.\n\t (ifname2ifindex) New function to complement ifindex2ifname.\n\t (interface) The interface command should check the name length\n\t and fail with a warning message if it is too long.\n\t (no_interface) Fix spelling in warning message.\n\t (if_nametoindex) Reimplement using if_lookup_by_name.\n\t (if_indextoname, ifaddr_ipv4_lookup) Reimplement using\n\t if_lookup_by_index.\n\t* bgp_zebra.c: (bgp_interface_delete) After deleting, set ifp-\u003eifindex\n\t to IFINDEX_INTERNAL.\n\t* isis_zebra.c: (isis_zebra_if_del) Call if_delete_retain instead\n\t of if_delete, since it is generally not safe to remove interface\n\t structures. After deleting, set ifp-\u003eifindex to IFINDEX_INTERNAL.\n\t (zebra_interface_if_lookup) Tighten up code.\n\t* ospf6_zebra.c: (ospf6_zebra_if_del) Previously, this whole function\n\t was commented out. But this is not safe: we should at least update\n\t the ifindex when the interface is deleted. So the new version\n\t updates the interface status and sets ifp-\u003eifindex to\n\t IFINDEX_INTERNAL.\n\t (ospf6_zebra_route_update) Use if_indextoname properly.\n\t* ospf_vty.c: (show_ip_ospf_interface_sub) Show ifindex and interface\n\t flags to help with debugging.\n\t* ospf_zebra.c: (ospf_interface_delete) After deleting, set ifp-\u003eifindex\n\t to IFINDEX_INTERNAL.\n\t (zebra_interface_if_lookup) Make function static. Tighten up code.\n\t* rip_interface.c: (rip_interface_delete) After deleting, set\n\t ifp-\u003eifindex to IFINDEX_INTERNAL.\n\t* ripng_interface.c: (ripng_interface_delete) After deleting, set\n\t ifp-\u003eifindex to IFINDEX_INTERNAL.\n" }, { "commit": "847947f20b152f0b64057a7115e118b44d238b5c", "tree": "68f726878177a594e258ce5312b7937951d3b421", "parents": [ "dd4c593fb88b7c6cf5f27aa8a8286644743ed58a" ], "author": { "name": "ajs", "email": "ajs", "time": "Wed Feb 02 18:38:48 2005 +0000" }, "committer": { "name": "ajs", "email": "ajs", "time": "Wed Feb 02 18:38:48 2005 +0000" }, "message": "2005-02-02 Andrew J. Schorr \u003cajschorr@alumni.princeton.edu\u003e\n\n\t* ospf_packet.c: (ospf_read) Fix bug: must check for state ISM_Down,\n\t not for event ISM_InterfaceDown. And improve the message by\n\t adding the interface flags.\n\t* if.h: Declare if_flag_dump.\n\n\t[backport candidate]\n" }, { "commit": "4ba9b9247222a6031a8060ddd39bfb19d90624e8", "tree": "4a738bb9239bd171135e61e81d17df833c07fd66", "parents": [ "b8adec1f98b155c6bf5e54067c8e31ae1de3c133" ], "author": { "name": "paul", "email": "paul", "time": "Tue Dec 21 22:34:58 2004 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Tue Dec 21 22:34:58 2004 +0000" }, "message": "2004-12-21 Paul Jakma \u003cpaul.jakma@sun.com\u003e\n\n\t* if.h: Add more \u0027non-generic\u0027 IFF_ flags.\n\t* if.c: IFF_NOXMIT/IFF_VIRTUAL interfaces are \u0027loopback like\u0027\n" }, { "commit": "3fb9cd6ef456959b6eff939d5c316f6785c2dda4", "tree": "e350cb3ef7b20b8bbccfb1aa9309152311e845bd", "parents": [ "5ae35f45f1292dce3a01f241accafeb2e59b10cc" ], "author": { "name": "hasso", "email": "hasso", "time": "Tue Oct 19 19:44:43 2004 +0000" }, "committer": { "name": "hasso", "email": "hasso", "time": "Tue Oct 19 19:44:43 2004 +0000" }, "message": "OK. Here it is - PtP patch from Andrew J. Schorr. No problems with ospfd,\nripd might need some more testing though.\n" }, { "commit": "c9eca01b298b83a849a608477f5f5b978a58167e", "tree": "afd25d986e49e57c73a4d555d4c083ab66d454fa", "parents": [ "6c83567192ada1a66822c3f35580ce6a85f51ac9" ], "author": { "name": "paul", "email": "paul", "time": "Mon Oct 11 11:28:44 2004 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Mon Oct 11 11:28:44 2004 +0000" }, "message": "2004-10-11 Paul Jakma \u003cpaul@dishone.st\u003e\n\n\t* if.h: mtu\u0027s should be unsigned.\n\t* routemap.{c,h}: const char updates\n\t* smux.{c,h}: ditto\n" }, { "commit": "9035efaa924c69f4f4fcb1049c7dc4f43b9da980", "tree": "f81d25b0e069ad8fb5f2843172a4e60fa0d6cbdf", "parents": [ "ddd119fd3d94e95dd44aa9fb3bc3fca4f26078a0" ], "author": { "name": "paul", "email": "paul", "time": "Sun Oct 10 11:56:56 2004 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Sun Oct 10 11:56:56 2004 +0000" }, "message": "2004-10-10 Paul Jakma \u003cpaul@dishone.st\u003e\n\n\t* version.h.in: (pid_output*) add const qualifier.\n\t* command.h: Change DEFUN func to take const char *[] rather\n than char **, to begin process of fixing compile warnings in lib/.\n Nearly all other changes in this commit follow from this change.\n * buffer.{c,h}: (buffer_write) pointer-arithmetic is gccism, take\n const void * and cast an automatic const char *p to it.\n (buffer_putstr) add const\n * command.c: (zencrypt) const qualifier\n (cmd_execute_command_real) ditto\n (cmd_execute_command_strict) ditto\n (config_log_file) ditto.\n Fix leak of getcwd() returned string.\n * memory.{c,h}: Add MTYPE_DISTRIBUTE_IFNAME for struct dist ifname.\n * distribute.{c,h}: Update with const qualifier.\n (distribute_free) use MTYPE_DISTRIBUTE_IFNAME\n (distribute_lookup) Cast to char *, note that it\u0027s ok.\n (distribute_hash_alloc) use MTYPE_DISTRIBUTE_IFNAME.\n (distribute_get) Cast to char *, note that it\u0027s ok.\n * filter.c: Update with const qualifier.\n * if.{c,h}: ditto.\n * if_rmap.{c,h}: ditto.\n (if_rmap_lookup) Cast to char *, note that it\u0027s ok.\n (if_rmap_get) ditto.\n * log.{c,h}: Update with const qualifier.\n * plist.{c,h}: ditto.\n * routemap.{c,h}: ditto.\n * smux.{c,h}: ditto. Fix some signed/unsigned comparisons.\n * sockopt.c: (getsockopt_cmsg_data) add return for error case.\n * vty.c: Update with const qualifier.\n" }, { "commit": "52dc7ee65f8d887b0730abc0a5d44d27fc6ecafd", "tree": "a557339540c56dd3953c29a50ca0e48c1911efc8", "parents": [ "44983cf8a9c587dfbcad294b9dfe4dccbb68ba98" ], "author": { "name": "hasso", "email": "hasso", "time": "Thu Sep 23 19:18:23 2004 +0000" }, "committer": { "name": "hasso", "email": "hasso", "time": "Thu Sep 23 19:18:23 2004 +0000" }, "message": "Remove usage of evil list and listnode typedefs.\n" }, { "commit": "4a7aac1bc609ee147492b228af9222f526e2ca2f", "tree": "4f764b16562766422bca65ff2f6668a51757b9ec", "parents": [ "5bd4189c17bad7a9e115ef573c0324c4b969a641" ], "author": { "name": "paul", "email": "paul", "time": "Sat May 08 05:00:31 2004 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Sat May 08 05:00:31 2004 +0000" }, "message": "2004-05-08 Sowmini Varadhan \u003csowmini.varadhan@sun.com\u003e\n\n * if.h: Add mtu6 field to struct interface, IPv6 MTU may differ\n from IPv4, and Solaris treats the MTU\u0027s differently.\n Add connected_add_by_prefix, for use by later patch.\n * if.c: (connected_add_by_prefix) Add prefix to connected list.\n (if_flag_dump) Solaris: Dump IFF_IPv4/6 flag\n (if_dump) Dump mtu6 flag, for HAVE_IPV6.\n" }, { "commit": "948cf55c4447257960fda0d89fb45e93644393b1", "tree": "87360f9330b45da6aa9b4d2d168cb787a26d391d", "parents": [ "3738d8cab744b1e11148d23ce16db5891b5e747e" ], "author": { "name": "paul", "email": "paul", "time": "Mon Oct 27 22:12:02 2003 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Mon Oct 27 22:12:02 2003 +0000" }, "message": "2003-10-27 Paul Jakma \u003cpaul@dishone.st\u003e\n\n\t* lib/if.{ch}: remove ifc_pointtopoint() - left over from the\n\t reverted RFC3021 patch.\n" }, { "commit": "106d2fd572c18bebe3dc44031c6d5e56e7b9030f", "tree": "df226870be1178b347947aecbe7d468022b47a91", "parents": [ "12ab19f1863e80134353244967a87805b12fe722" ], "author": { "name": "paul", "email": "paul", "time": "Fri Aug 01 00:24:13 2003 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Fri Aug 01 00:24:13 2003 +0000" }, "message": "2003-08-01 Cougar \u003ccougar@random.ee\u003e\n\n\t* lib/if.c: (if_cmp_func) new function, compare interface names in\n\talphabetical order.\n\t(if_create) Take name as argument and add interface in sorted order.\n\t(if_get_by_name),(interface_cmd) fixup calls to if_create - see\n\tabove.\n\t(if_init) register list comparison function.\n\t* lib/if.h: Add comparison function, modify if_create prototype.\n\t* lib/zclient.c: Modify call to if_create.\n\t* ospfd/ospf_interface.c: (ospf_vl_new) modify call to if_create.\n\tchange sprintf to snprintf.\n\t* zebra/kernel_socket.c: (ifm_read) modify call to if_create.\n" }, { "commit": "592c8148981d196a7bda2f1b85a63943fd0cfdf2", "tree": "273fd4151921ffbb91cc42535083ff7e1bf04bdb", "parents": [ "bf9392c6e932080156e70c0436977fd5dbcdb0cd" ], "author": { "name": "paul", "email": "paul", "time": "Fri Jun 06 23:24:55 2003 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Fri Jun 06 23:24:55 2003 +0000" }, "message": "2003-06-07 Paul Jakma \u003cpaul@dishone.st\u003e\n\n\t* Revert Cougar\u0027s sort interface names patch, causes problems with\n\tenabling of interfaces for OSPF in ospfd.\n" }, { "commit": "a939514c59fe1a56a820e82bc59d82c721076c03", "tree": "85a2d3d1d3e0a4f35d0865778c7522b8569b40f8", "parents": [ "41908818b8f446befc3654fa5bbc98c6c0af92a3" ], "author": { "name": "hasso", "email": "hasso", "time": "Thu Jun 05 17:50:01 2003 +0000" }, "committer": { "name": "hasso", "email": "hasso", "time": "Thu Jun 05 17:50:01 2003 +0000" }, "message": "Patch from Cougar - sort iflist by name.\n" }, { "commit": "32d2463ca26529ad4700af6d5748f927e1305694", "tree": "9f8dc5d94e1fb11a86585864e45401f552ecf2dd", "parents": [ "7515cffe5e11fdfbc437c02b7af8b0f7b4d36fce" ], "author": { "name": "paul", "email": "paul", "time": "Fri May 23 09:25:20 2003 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Fri May 23 09:25:20 2003 +0000" }, "message": "From: Hasso Tepper \u003chasso@estpak.ee\u003e\n\nAdd the \u0027no interface\u0027 command to all the daemons and vtysh. now it\u0027s\npossible to delete interface from routeing daemons as well only if it\ndoesn\u0027t exist in os.\n\nhttp://hasso.linux.ee/zebra/ht-no_interface_fix.patch\n" }, { "commit": "7515cffe5e11fdfbc437c02b7af8b0f7b4d36fce", "tree": "9acbaf0a01080920f25cf1107465a7f79ee7e2f8", "parents": [ "1921e6f8965701040fe1b4f27619b2d7f3fec254" ], "author": { "name": "paul", "email": "paul", "time": "Fri May 23 09:21:46 2003 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Fri May 23 09:21:46 2003 +0000" }, "message": "Undo Hasso last patch, about to apply a slightly updated patch.\n" }, { "commit": "68f43848fa0f8ac16c768bafb8891495dcf232ae", "tree": "8121fe0ee7c19001dd3c37c798e8368681a59493", "parents": [ "8d6b00e4f34dfb6f627b49713bf80b0745b50abc" ], "author": { "name": "paul", "email": "paul", "time": "Fri May 23 06:58:44 2003 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Fri May 23 06:58:44 2003 +0000" }, "message": "From: Hasso Tepper \u003chasso@estpak.ee\u003e\n\nhttp://hasso.linux.ee/zebra/ht-no_interface-23052003.patch\n\nAdd the \u0027no interface\u0027 command to all the daemons and vtysh.\n" }, { "commit": "2e3b2e474ed5ba04744b167132a84f9954485af4", "tree": "cc07fb34a1bf21ead5cc3ad69e4a9dfde2b43b0e", "parents": [ "7afa08da2d3690baee1739da400775f8543f5863" ], "author": { "name": "paul", "email": "paul", "time": "Fri Dec 13 21:03:13 2002 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Fri Dec 13 21:03:13 2002 +0000" }, "message": "zebra link state detection support\n" }, { "commit": "727d104b15d6293d7223e7830f2e37ae98022cac", "tree": "9ab7e7fecd699f30abc63a5f4612fa9f592db098", "parents": [ "4fc01e67edf1763c9812438effe1d4b1c8b9897b" ], "author": { "name": "paul", "email": "paul", "time": "Fri Dec 13 20:50:29 2002 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Fri Dec 13 20:50:29 2002 +0000" }, "message": "Added RIPv1 patch - bug fixes and improved/more interoperable classful\nsubnet handling\n" }, { "commit": "718e3744195351130f4ce7dbe0613f4b3e23df93", "tree": "bac2ad39971cd43f31241ef123bd4e470f695ac9", "parents": [], "author": { "name": "paul", "email": "paul", "time": "Fri Dec 13 20:15:29 2002 +0000" }, "committer": { "name": "paul", "email": "paul", "time": "Fri Dec 13 20:15:29 2002 +0000" }, "message": "Initial revision\n" } ] }