summaryrefslogtreecommitdiffstats
path: root/ripd
Commit message (Collapse)AuthorAgeFilesLines
...
| * 2003-06-20 Paul Jakma <paul@dishone.st>paul2003-06-201-4/+6
| | | | | | | | * rip_interface.c: add 'default' to passive-interface command DEFUN
| * 2003-06-19 Paul Jakma <paul@dishone.st>paul2003-06-191-3/+5
| | | | | | | | | | * Fix lib/smux.c's reliance on daemons exporting struct thread_master *master.
| * 2003-06-15 Paul Jakma <paul@dishone.st>paul2003-06-151-1/+1
| | | | | | | | | | | | | | | | * lib/vty.{c,h}: Remove vty layer depending on a 'master' global, pass the thread master in explicitly to vty_init. Sort out some header dependency problems with lib/command.h * zebra/: Move globals to struct zebrad. Update vty_init(). * (.*)/\1_main.c: update call to vty_init().
| * 2003-06-08 Paul Jakma <paul@dishone.st>paul2003-06-082-6/+6
| | | | | | | | Compile and warning fixes for the ripd 'passive-interface default' patch.
| * From: Andrew J. Schorr <aschorr@telemetry-investments.com>paul2003-06-074-58/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Subject: [zebra 12403] patch for ripd to accept any version of RIP by default The default Cisco IOS behavior is to send RIP version 1 packets and receive version 1 and version 2 packets. But zebra version 0.92a sends and receives only version 2 packets by default. I have patched the code to change zebra's default behavior to sending version 2 packets (same as before) but receiving both versions. While this is still not identical to Cisco's behavior, it does now accept packets of both versions and retains backwards compatibility with zebra configurations.
| * From: Andrew J. Schorr <aschorr@telemetry-investments.com>paul2003-06-073-30/+73
| | | | | | | | | | | | | | | | | | | | | | | | | | Subject: [zebra 12406] patch for ripd to support "passive-interface default" behavior In Cisco IOS, the router rip passive-interface subcommand can accept an interface name of "default". In that cases, all interfaces are set to be passive by default, and you must explicitly make it non-passive by using the "no passive-interface <ifname>" command if you want updates to go to that interface. <command implemented for zebra>
| * 2003-06-04 Paul Jakma <paul@dishone.st>paul2003-06-044-3/+46
| | | | | | | | * Merge of zebra privileges
| * Unbroke "set metric" command in vtysh again.hasso2003-05-251-3/+10
| |
| * Merge RIP part of 6Wind patch.hasso2003-05-257-111/+742
| |
| * Ripd has "no interface ..." command as well.hasso2003-05-241-0/+1
| |
| * Update Changelogspaul2003-05-161-0/+5
| |
| * Hasso Tepperpaul2003-04-191-10/+19
| | | | | | | | | | | | http://hasso.linux.ee/zebra/ht-routemap-18042003_5.patch Trivial fixes to sync daemon's route-map commands to have same syntax. Fixes a lot of errors with "match ..." and "set ..." commands when using vtysh.
| * Add Makefile.in to the remaining .cvsignore'spaul2003-04-091-0/+1
| | | | | | | | Add ospfclient to ospfclient/.cvsignore
| * [zebra 18626] move zebra socket paths to configure.acpaul2003-04-071-1/+0
| | | | | | | | | | | | | | | | | | | | | | moved definition of the various socket paths from the per daemon header files into configure.ac. it will set the paths to be in the directory specified by --localstatedir=<prefix> or otherwise will try to guess as best it can ( a la pid file path detection - which probably should try reference ${prefix} too). the present hardcoded socket path, /tmp, isnt really correct. should be in /var somewhere really.
| * Remove auto* files from the repository.paul2003-04-071-403/+0
| | | | | | | | | | | | | | | | | | Developers working with the repository should have the appropriate tools. Out-of-sync files cause far too many problems with users as well as auto* scripts not being half as portable across systems as they ought to be. make-dist exists for a reason. Todo: make the CVS snapshot script do make-dist, and use the resulting tarball as the snapshot.
| * 2003-03-25 Paul Jakma <paul@dishone.st>paul2003-03-251-203/+203
| | | | | | | | | | | | | | | | * sync to latest zebra CVS * spec file: updated and added define for ospf-api/client NB: OSPF-API has been broken by the zebra.org changes, which has added struct ospf * as a new arg to many functions
| * Fix build for net-snmppaul2003-03-181-0/+3
| |
| * Update auto files.paul2003-03-171-225/+210
| | | | | | | | No doubt builds will now break for everyone. (works here - autoconf 2.13)
| * Sync to Zebra.org 20030301paul2003-03-011-124/+53
| |
| * Revert to zebra.org automake filespaul2003-02-171-52/+123
| |
| * Update auto* build filespaul2003-02-131-123/+52
| |
| * Matthew Grant <grantma@anathoth.gen.nz>patch_z17290_portfixpaul2003-01-221-1/+9
| | | | | | | | | | | | [zebra 17290] [PATCHES] - Fixes for problems in 0.93b portfix patch
| * [zebra 14631] Generic PtP and RFC3021 interface addressing supportpatch_z14631_ptp_rfc3021paul2002-12-133-104/+2
| |
| * zebra link state detection supportpatch_z12269_linkstatepaul2002-12-132-5/+5
| |
| * Added RIPv1 patch - bug fixes and improved/more interoperable classfulpatch_z15769_ripv1paul2002-12-131-31/+101
| | | | | | | | subnet handling
| * Initial revisionpaul2002-12-1317-0/+11186
|
* flag field printf formatting in debug stringsStephen Hemminger2008-05-281-8/+14
| | | | | | | Potential format incompatiablities because flags field in zebra is uint64_t but the printf format was only 32 bit. For safety, convert the flags field to unsigned long long. Since it is really a bit field print in hex.
* comment out unused codeStephen Hemminger2008-05-281-0/+2
| | | | remove unused code, compiler flags this as unused
* Handle empty auth string properlyStephen Hemminger2008-05-281-1/+1
| | | | | | | | | | | Compiler warned about dubious code where test was: char auth_str[]; ... if (!auth_str) return 0; Looks like what was meant was testing for empty auth string.
* [ripd] remove unnecessary 0 entries from struct message'spaul2008-03-133-2/+6
| | | | | | | | 2008-03-13 Paul Jakma <paul.jakma@sun.com> * ripd.c/rip_interface.c: Remove 0 entries from rip_msg ri_version_msg struct message's, not needed with recent fixes to mes_lookup.
* [autoconf] bugs 162,303,178: Fix 'present but can not be compiled' warningspaul2007-05-101-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | 2007-05-09 Paul Jakma <paul.jakma@sun.com> * configure.ac: sys/conf.h depends on sys/param.h, at least on FBSD 6.2. (bug #363) Should check for in_pktinfo for IRDP 2006-05-27 Paul Jakma <paul.jakma@sun.com> * configure.ac: General cleanup of header and type checks, introducing an internal define, QUAGGA_INCLUDES, to build up a list of stuff to include so as to avoid 'present but cant be compiled' warnings. Misc additional checks of things missing according to autoscan. Add LIBM, for bgpd's use of libm, so as to avoid burdening LIBS, and all the binaries, with libm linkage. Remove the bad practice of using m4 changequote(), just quote the []'s in the case statements properly. This should fix bugs 162, 303 and 178. * */*.{c,h}: Update all HAVE_* to the standard autoconf namespaced HAVE_* defines. I.e. HAVE_SA_LEN -> HAVE_STRUCT_SOCKADDR_SA_LEN, * bgpd/Makefile.am: Add LIBM to bgpd's LDADD, for pow().
* [ripd, ripngd] Remove useless code in rip_vty_out_uptime functionajs2007-04-272-3/+4
| | | | | | | | 2007-04-27 Andrew J. Schorr <ajschorr@alumni.princeton.edu> * ripd/ripd.c: (rip_vty_out_uptime) Remove unused variable timer_now. * ripngd/ripngd.c: (ripng_vty_out_uptime) Remove unused variable timer_now.
* Fix the display of route timeout in "show ip rip". (Use ↵vincent2007-04-112-2/+7
| | | | thread_timer_remain_second)
* [ripd] Fix "show ip rip status" display of time until next updateajs2007-03-212-13/+9
| | | | | | | | | 2007-03-21 Andrew J. Schorr <ajschorr@alumni.princeton.edu> * ripd.c: (show_ip_rip_status) Use new thread_timer_remain_second function instead of rip_next_thread_timer to display the time until next update properly. (rip_next_thread_timer) Remove obsolete function.
* [PtP over ethernet] New peer flag allows much more addressing flexibilityajs2006-12-123-14/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2006-12-12 Andrew J. Schorr <ajschorr@alumni.princeton.edu> * if.h: (struct connected) Add new ZEBRA_IFA_PEER flag indicating whether a peer address has been configured. Comment now shows the new interpretation of the destination addr: if ZEBRA_IFA_PEER is set, then it must contain the destination address, otherwise it may contain the broadcast address or be NULL. (CONNECTED_DEST_HOST,CONNECTED_POINTOPOINT_HOST) Remove obsolete macros that were specific to IPv4 and not fully general. (CONNECTED_PEER) New macro to check ZEBRA_IFA_PEER flag. (CONNECTED_PREFIX) New macro giving the prefix to insert into the RIB: if CONNECTED_PEER, then use the destination (peer) address, else use the address field. (CONNECTED_ID) New macro to come up with an identifying address for the struct connected. * if.c: (if_lookup_address, connected_lookup_address) Streamline logic with new CONNECTED_PREFIX macro. * prefix.h: (PREFIX_COPY_IPV4, PREFIX_COPY_IPV6) New macros for better performance than the general prefix_copy function. * zclient.c: (zebra_interface_address_read) For non-null destination addresses, set prefixlen to equal the address prefixlen. This is needed to get the new CONNECTED_PREFIX macro to work properly. * connected.c: (connected_up_ipv4, connected_down_ipv4, connected_up_ipv6, connected_down_ipv6) Simplify logic using the new CONNECTED_PREFIX macro. (connected_add_ipv4) Set prefixlen in destination addresses (required by the CONNECTED_PREFIX macro). Use CONNECTED_PEER macro instead of testing for IFF_POINTOPOINT. Delete invalid warning message. Warn about cases where the ZEBRA_IFA_PEER is set but no destination address has been supplied (and turn off the flag). (connected_add_ipv6) Add new flags argument so callers may set the ZEBRA_IFA_PEER flag. If peer/broadcast address satisfies IN6_IS_ADDR_UNSPECIFIED, then reject it with a warning. Set prefixlen in destination address so CONNECTED_PREFIX will work. * connected.h: (connected_add_ipv6) Add new flags argument so callers may set the ZEBRA_IFA_PEER flag. * interface.c: (connected_dump_vty) Use CONNECTED_PEER macro to decide whether the destination address is a peer or broadcast address (instead of checking IFF_BROADCAST and IFF_POINTOPOINT). * if_ioctl.c: (if_getaddrs) Instead of setting a peer address only when the IFF_POINTOPOINT is set, we now accept a peer address whenever it is available and not the same as the local address. Otherwise (no peer address assigned), we check for a broadcast address (regardless of the IFF_BROADCAST flag). And must now pass a flags value of ZEBRA_IFA_PEER to connected_add_ipv4 when a peer address is assigned. The same new logic is used with the IPv6 code as well (and we pass the new flags argument to connected_add_ipv6). (if_get_addr) Do not bother to check IFF_POINTOPOINT: just issue the SIOCGIFDSTADDR ioctl and see if we get back a peer address not matching the local address (and set the ZEBRA_IFA_PEER in that case). If there's no peer address, try to grab SIOCGIFBRDADDR regardless of whether IFF_BROADCAST is set. * if_ioctl_solaris.c: (if_get_addr) Just try the SIOCGLIFDSTADDR ioctl without bothering to check the IFF_POINTOPOINT flag. And if no peer address was found, just try the SIOCGLIFBRDADDR ioctl without checking the IFF_BROADCAST flag. Call connected_add_ipv4 and connected_add_ipv6 with appropriate flags. * if_proc.c: (ifaddr_proc_ipv6) Must pass new flags argument to connected_add_ipv6. * kernel_socket.c: (ifam_read) Must pass new flags argument to connected_add_ipv6. * rt_netlink.c: (netlink_interface_addr) Copy logic from iproute2 to determine local and possible peer address (so there's no longer a test for IFF_POINTOPOINT). Set ZEBRA_IFA_PEER flag appropriately. Pass new flags argument to connected_add_ipv6. (netlink_address) Test !CONNECTED_PEER instead of if_is_broadcast to determine whether the connected destination address is a broadcast address. * bgp_nexthop.c: (bgp_connected_add, bgp_connected_delete) Simplify logic by using new CONNECTED_PREFIX macro. * ospf_interface.c: (ospf_if_is_configured, ospf_if_lookup_by_prefix, ospf_if_lookup_recv_if) Simplify logic using new CONNECTED_PREFIX macro. * ospf_lsa.c: (lsa_link_ptop_set) Using the new CONNECTED_PREFIX macro, both options collapse into the same code. * ospf_snmp.c: (ospf_snmp_if_update) Simplify logic using new CONNECTED_ID macro. (ospf_snmp_is_if_have_addr) Simplify logic using new CONNECTED_PREFIX macro. * ospf_vty.c: (show_ip_ospf_interface_sub) Use new CONNECTED_PEER macro instead of testing the IFF_POINTOPOINT flag. * ospfd.c: (ospf_network_match_iface) Use new CONNECTED_PEER macro instead of testing with if_is_pointopoint. And add commented-out code to implement alternative (in my opinion) more elegant behavior that has no special-case treatment for PtP addresses. (ospf_network_run) Use new CONNECTED_ID macro to simplify logic. * rip_interface.c: (rip_interface_multicast_set) Use new CONNECTED_ID macro to simplify logic. (rip_request_interface_send) Fix minor bug: ipv4_broadcast_addr does not give a useful result if prefixlen is 32 (we require a peer address in such cases). * ripd.c: (rip_update_interface) Fix same bug as above.
* [daemon startup] Add --dry-run/-C argument to daemons, to check config file ↵paul2006-10-151-1/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | syntax 2006-10-04 Oliver Hookins <ohookins@gmail.com> * bgpd/bgp_main.c: Add configuration check option, with '-C' rather than '-c' for consistency between daemons. * isisd/isis_main.c: ditto * ospf6d/ospf6_main.c: ditto * ospfd/ospf_main.c: ditto * ripngd/ripng_main.c: ditto * vtysh/vtysh_main.c: ditto * ripd/rip_main.c: Change the config check option to '-C' and tidy up the code. * zebra/main.c: ditto 2006-10-04 Stergiakis Alexandros <astergiakis@antcor.com> * ripd/rip_main.c: This trivial patch introduces a new command-line option '-c', which instructs zebra/ripd to check its configuration file for validity, print any error message, and then exit. This is useful when the configuration file is edited by hand or otherwise, and you simply want to validate it without any other effect. * zebra/main.c: ditto
* [ripd] bug #293: routemap set metric doesn't check for underflow correctlypaul2006-09-112-1/+4
| | | | | | | | 2006-09-11 Paul Jakma <paul.jakma@sun.com> * rip_routemap.c: (route_set_metric) underflow check needs to use signed, problem identified and diagnosed by Pavel Nikiforov in bug #293.
* [ripd] bug #278: remove gratuitous use of mid-function declarationpaul2006-09-112-2/+8
| | | | | | | 2006-09-11 Paul Jakma <paul.jakma@sun.com> * ripd.c: (rip_read) remove gratuitous use of mid-function declaration of vrecv, bug #278.
* [ripd] update to use auto-generated redistribute route-type defspaul2006-06-302-56/+30
| | | | | | | 2006-06-29 Paul Jakma <paul.jakma@sun.com> * rip_zebra: (general) convert redistribute commands to use the auto-generated defines.
* [ripd] bugs #261, #262: Fix RIPv1 info-leak and unauthenticated route updatespaul2006-05-043-108/+131
| | | | | | | | | | | | | | | | | | 2006-05-04 Paul Jakma <paul.jakma@sun.com> * (general) Fixes for bugs #261 and 262. Thanks to Konstantin V. Gavrilenko <kos@arhont.com> for the problem reports, testing of a series of proposed patches and comment on the proposed changes in behaviour. * rip_interface.c: (ip_rip_authentication_mode_cmd) Parse all of the command before making any changes to configured state. * ripd.c: (rip_read) RIP version control should be absolute and always apply, fixes bug #261 by allowing RIPv1 to be disabled. Fix bug #262: If authentication is enabled, then unauthenticated packets should not be accepted. We do however make an exception for RIPv1 REQUEST packets, to which we will reply as RIPv1 can now be disabled fully, to allow ripd to still provide routing /information/ to simple devices.
* [ripd] Fix logic to send updates on all connected addresses.ajs2006-04-282-31/+28
| | | | | | | | | | | | | 2006-04-28 Andrew J. Schorr <ajschorr@alumni.princeton.edu> * ripd.c: (rip_update_process) Try to fix the logic for sending an updated on each connected network. The new code will attempt to send the update on each connected network, whereas the previous code seemed to be attempting to avoid sending more than one RIPv1 update on a given interface, but was coded incorrectly. The actual effect of the old code was to send an update only on the first connected address in the cases where the interface is not multicast, or RIPv2 is not being used.
* ripd.c: correct bug that allowed route learnt through RIP to take precedence ↵vincent2006-01-302-1/+8
| | | | over connected routes
* [compiler] miscellaneous trivial compiler warning fixespaul2006-01-192-1/+5
| | | | | | | | | | | | | 2006-01-19 Paul Jakma <paul.jakma@sun.com> * (general) various miscellaneous compiler warning fixes. Remove redundant break statements from switch clauses which return. return from main, not exit, cause it annoys SOS. Remove stray semi-colons which cause empty-statement warnings. * zebra/main.c: (sighup) remove private declaration of external function.
* [ripd] Fix verification of received MD5 authenticated packetspaul2006-01-172-23/+33
| | | | | | | | | | | | | | | | | | 2006-01-17 Paul Jakma <paul.jakma@sun.com> * ripd.c: (rip_auth_md5) remove pdigest, not needed. Use a local buffer for the auth_str, where it can be properly nul padded. Do so, hence fixing MD5 authentication. Key looked up via key ring should be used in preference to the RIPv1 simple password, not other way around. No need to copy around digests, we can reference them directly. The auth_len received can't be trusted, some implementations lie (e.g. older ripd). (rip_auth_md5_ah_write) rename len local variable to doff to be consistent with other functions. (rip_auth_header_write) add the missing return. (rip_auth_md5_set) use the proper constructs to access stream.
* 2005-10-30 Andrew J. Schorr <ajschorr@alumni.princeton.edu>ajs2005-10-304-56/+11
| | | | | | | | | * ripd.c: (rip_response_process) Instead of calling rip_interface.c:if_valid_neighbor(), call the equivalent library function if_lookup_address(). * rip_interface.c: (if_valid_neighbor) Remove function, since it is essentially equivalent to the if_lookup_address() library function. * ripd.h: (if_valid_neighbor) Remove function declaration.
* 2005-10-28 Paul Jakma <paul.jakma@sun.com>paul2005-10-282-1/+6
| | | | | * Makefile.am: Add rip_interface.h, or else it doesn't get included in dists.
* 2005-10-26 Paul Jakma <paul.jakma@sun.com>paul2005-10-252-6/+8
| | | | | * ripd.c: Update couple more functions to specify void explicitely.
* 2005-10-26 Paul Jakma <paul.jakma@sun.com>paul2005-10-2512-233/+267
| | | | | | * (general) static/extern functions and definitions. * rip_interface.h: new file, export the public functions from rip_interface.c
* 2005-10-17 Vincent Jardin <vincent.jardin@6wind.com>jardin2005-10-192-0/+8
| | | | | | * ripd.c: rip_create_socket() for each packet, it does not bind to the proper interfaces because we forget to use the from address when it is specified.
* 2005-10-06 Alain Ritoux <alain.ritoux@6wind.com>vincent2005-10-062-1/+24
| | | | | | * rip_interface.c: Now the command "no ip rip split-horizon poisoned-reverse" just inhibates the poisoned-reverse effects but keep spli-horizon activ.