| Age | Commit message (Collapse) | Author | 
|---|
|  |  | 
|  |  | 
|  | Add an struct interface paramenter and adjust the affected
    code accordingly.
    The old code was a mess looping over all interfaces several times
    when one interface was added/changed.
    * ospfd/ospfd.h: Add struct interface parameter to ospf_if_update()
    * ospfd/ospf_zebra.c: Add ifp arg to ospf_if_update() calls.
      (ospf_interface_address_delete) delete ospf_if_update() call,
      redundant as function calls ospf_if_free() itself.
    * ospfd/ospfd.c: (ospf_network_unset) handle deconfiguration here,
      rather than ospf_if_update.
      (ospf_network_run_interface) ospf_network_run, for
      any given interface.
      (ospf_network_run) move guts to previous, and use it.
      (ospf_if_update) Adjust to take struct interface as argument, as
      all callers have a specific ifp in mind.
      Iterate over ifp's connected list and call ospf_network_run_interface
      instead of ospf_network_run, turning this path into O(nm) rather
      than O(n^2).
      Adjust all code dealing with opsf_if_update and ospf_network_run to
      pass the new struct interface * arg.
     (some minor modifications and bug-additions by Paul Jakma).
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 'remotes/jocke/snmp_crosscompile' | 
|  |  | 
|  |  | 
|  | accepted conns"
Revert commit d664ae1182c29b74b409bc8594b7bd0575e91ce9. An experimental
patch which violates RFC4271 quite badly, but managed to accidently sneak
its way in. | 
|  | Update ospf_db_desc_send(), ospf_ls_upd_queue_send() and ospf_ls_req_send()
to always use OSPF_ALLSPFROUTERS for PtP links.
See RFC 2328, chap 8.1 for details:
    "The IP destination address for the packet is selected as
     follows.  On physical point-to-point networks, the IP
     destination is always set to the address AllSPFRouters."
Without this, it won't be possible to establish adjacencies on
multiple unnumbered links to the same router.
ChangeLog:
2008-07-25 Joakim Tjernlund <Joakim.Tjernlund@transmode.se>
	* ospfd/ospf_packet.c: Set destination for PtP links to
	  OSPF_ALLSPFROUTERS. | 
|  | PtP links with /32 masks adds host routes to the remote host,
see RFC 2328, 12.4.1.1, Option 1.
Make sure that such routes are ignored | 
|  | --without-crypto
Autoconfig work by me, the rest was done by
"Kirill K. Smirnov" <lich@math.spbu.ru> | 
|  | Dont use vty->index in show_ip_ospf_neighbor_all_cmd(),
It is invalid, use ospf_lookup() instead. | 
|  | * lib/command.h: Add a RESTRICTED_NODE, intended for use with
	  anonymous, 'no login' vtys, to provide a subset of 'view' mode
	  commands.
	* lib/command.c: Add RESTRICTED_NODE bits, nothing special, just
	  following VIEW_NODE.
	* lib/vty.c: (vty_auth) enable authentication should fall back to
	  restricted/view node as appropriate.
	  (vty_create) init vty's to restricted/view node as appropriate,
	  for the 'no login' case.
	  (vty_{no_,}restricted_mode_cmd) config commands to enable
	  'anonymous restricted' in vty configuration.
	  (vty_config_write) 'anonymous restricted' config.
	  (vty_init) Install some commands to restricted mode, and the
	  'anonymous restricted' config commands into VTY_NODE.
	* bgpd/*.c: Install some of the safe(r) BGP commands into
	  'restricted mode', i.e. lookup commands of non-sensitive data.
	  Useful with looking-glass route-servers. | 
|  | * ChangeLog: Add back a dummy, top-level ChangeLog
	* m4/Makefile.am: remove ChangeLog from EXTRA_DIST, why it was
	  listed there, who knows. | 
|  | - Add more stuff, archive libraries, etc.. | 
|  |  | 
|  |  | 
|  |  | 
|  | 2008-08-17 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* bgp_damp.?: (bgp_config_write_damp) remove useless check
	  of statically allocated config storage, and useless return
	  value
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-17 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* zebra_rib.c: (process_subq) mark static
	  (rib_meta_queue_add) ditto
	  (meta_queue_new) ditt
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-06-17 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* zebra_rib.c: static qualifier on local data
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-17 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* zebra_rib.c: When doing a RIB update, routes in process of removal
	  should be ignored. This fixes bugs where a route is removed but a
	  recursive route is not changed.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-06-17 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* rt_netlink.c: (netlink_parse_info) Linux doesn't check privledges
	  on receive, so avoid the overhead of lowering and raising the
	  priviledges on each received message
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-06-17 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* rt_netlink.c: (netlink_parse_info) There can be harmless races
	  such as adding an existing route, or deleting an already deleted
	  route. Don't bother logging these unless debugging is turned on.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-16 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* rt_netlink.c: (netlink_recvbuf) consolidate recvbuf setting
	  (netlink_socket) no need to set receive buffer on netlink sockets
	  generally
	  (kernel_init) set recvbuffer on the listen socket
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-16 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* configure.ac: If netlink is available, then that is how the
	initial routes and interfaces should be read, rather than through
	/proc, so don't set IF_PROC.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-16 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* zebra/rt_netlink.c: This change removes the overhead of twiddling
          the socket twice on every route change. The netlink support has
          two sockets, one is used for request/response and should always
          block. The other is used for asynchronous notification and should
          always be non-blocking.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-16 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* zebra/rt_netlink.c: Update the netlink socket BPF to cancel out
	  unwanted ACKs on the listen socket by matching on the PID.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-15 Paul Jakma <paul.jakma@sun.com>
	* */*: Fix various problems flagged by Sun Studio compiler.
	  - '<qualifier> <storage>' obsolescent in declarations
	  - empty statements (';' after ALIAS definitions)
	  - implicit declarations (e.g printstack in lib/log.c)
	  - "\%" in printf string instead of "%%"
	  - loops that return on the first iteration (legitimately, but
 	    compiler can't really know), e.g. bgp_routemap.c
 	  - internal declarations which mask prototypes. | 
|  | 2008-08-15 Paul Jakma <paul.jakma@sun.com>
	* lib/sockopt.c: (sockopt_tcp_signature) ifdef'ing hid variable
	  declaration for HAVE_DECL_TCP_MD5SIG && !GNU_LINUX, breaking
	  compile - fix. | 
|  | 2008-08-15 Paul Jakma <paul.jakma@sun.com>
	* {ospf6d,ripngd}/*: Finish job of marking functions as static, or
	  exporting declarations for them, to quell warning noise with
	  Quagga's GCC default high-level of warning flags. Thus allowing
	  remaining, more useful warnings to be more easily seen. | 
|  | 2008-08-14 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* lib/log.{c,h}: struct message argument should point to const
	* */*.c: adjust to suit,
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  |  | 
|  | 2008-08-14 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* zebra/rt_netlink.c: local functions should be static.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-14 Stephen Hemminger <stephen.hemminger@vyatta.com>
	* lib/hash.h: (struct hash) Hash comparator callback really
	  ought to treat storage behind arguments as constant - a compare
	  function with side-effects would be evil.
	* */*.c: Adjust comparator functions similarly, thus fixing at least
	  a few compiler warnings about const qualifier being dropped.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-13 Daniel Ng <daniel_ng11@lycos.com>
	* ospf_nsm.c: (nsm_change_state) Refresh default route on
	  state change - a hack, but it works.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-13 Jingjing Duan <Jingjing.Duan@sun.com>
	* isisd/isis_dlpi.c: (open_dlpi_dev) Clearview-UV device nodes are
	  under /dev/net, try opening there before attempting style 1 or 2
	  names.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-13 Jingjing Duan <Jingjing.Duan@sun.com>
	* ospfd/: Remove the old checksum implementation and
	  use the consolidated version.
	* isisd/: ditto, thus fixing isisd checksuming on big-endian.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-13 Jingjing Duan <Jingjing.Duan@sun.com>
	* isisd/isis_dlpi.c: Change ioctl from PFIOCSETF (transparent mode)
	  to I_STR (non-transparent mode). The old code resulted in no
	  filtering at all.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-13 Jingjing Duan <Jingjing.Duan@sun.com>
	* lib/checksum.?: (fletcher_checksum) implementation of
	  Fletcher checksum, as per RFC1008.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-13 Timo Teräs <timo.teras@iki.fi>
	* zebra/rt_netlink.c: (netlink_route{,_multipath) Set protocol and
	  scope appropriately for both NEW and DELROUTE messages, to avoid
	  wiping out non-zebra routes on delete.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-13 roy <quagga@de.vio.us>
	* lib/vty.c: (vty_delete_char) move check for authentication
	  down a bit, so we do the delete, but still not re-write of line.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-13 Paul P Komkoff Jr <i@stingr.net>
	* configure.ac: fix AC_ARG_ENABLE argument to match help string,
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2008-08-13 Paul P Komkoff Jr <i@stingr.net>
	* configure.ac: add a configure flag and autoconf macro, which will
	  determine if your toolchain supports PIE.
	* */Makefile.am: add corresponding CFLAGS and LDFLAGS into
	  appropriate places.
Signed-off-by: Paul Jakma <paul@quagga.net> | 
|  | 2007-08-31 Paul Jakma <paul.jakma@sun.com>
	* (general) Add 'bgp open-accept' option, to allow bgpd to send OPEN
	  on accepted connections, i.e. to not wait till after
	  collision-detect to send OPEN, which appears to be allowed in
	  RFC4271. This may help speed up establishing sessions, or help
	  avoid FSM problems with sessions to certain peers. Not enabled by
	  default though. | 
|  | (cherry picked from c5b85efd767b82fcb46b20ac6202c828d91aacb3 commit) | 
|  |  | 
|  |  | 
|  | 2008-07-22 Paul Jakma <paul.jakma@sun.com>
	* HACKING: Document preference for compiler conditional code, over
	  cpp conditional.
	* configure.ac: DISABLE_BGP_ANNOUNCE always should be defined.
	* bgp_{packet,route,advertise}.c: change to compiler testing of
	  DISABLE_BGP_ANNOUNCE, rather than cpp.
2008-07-22 MIYAJIMA Mitsuharu <miyajima.mitsuharu@anchor.jp>
	* bgp_packet.c: (bgp_update_packet_eor) Fix crash triggerable
	  if a bgpd was compiled with --disable-bgp-announce and if GR is
	  advertised by peer. | 
|  | 2008-07-22 Paul Jakma <paul.jakma@sun.com>
	* bgp_community.c: (community_str2com) assigns defaults to local
	  vars
	* bgp_attr.c: (bgp_attr_parse) match format specifier to arg
	* bgp_table.{c,h}: (bgp_table_top) can take a * to a const, quelling
	  warning in bgp_route.c | 
|  | 2008-07-21 Paul Jakma <paul.jakma@sun.com>
	* bgp_packet.c: (bgp_open_receive) fix warning in a zlog call
	* bgp_vty.c: (bgp_vty_return) add return code
	* bgpd.c: (bgp_master_init) setup the socket list.
	* bgp_network.c: Remove the dual IPv4/6 socket thing for now, which
	  was implemented by Michael, until such time as its clear its
	  required for Linux (see sockopt comments). IPv6 support, including
	  IPv4 sessions on AF_INET6 sockets, therefore is broken, and the
	  '-l 0.0.0.0' arguments would need to be given to bgpd to make
	  things work here.
2008-07-21 Michael H. Warfield <mhw@wittsend.com>
           YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
	   Tomohiko Kusuda <kusuda@inetcore.com>
           Leigh Brown <leigh@solinno.co.uk>
	* bgp_network.c: (bgp_md5_set_one) shim between libzebra tcp-md5
	  sockopt and bgpd.
	  (bgp_md5_set_socket) Helper for bgp_connect
	  (bgp_md5_set) setup TCP-MD5SIG for the given peer.
	  (bgp_connect) call out to bgp_md5_set_socket for the outgoing
	  connect socket.
	  (bgp_socket) save references to the listen sockets, needed if
	  TCP-MD5SIG is applied later or changed.
	* bgp_vty.c: (*neighbor_password_cmd) New 'neighbor ... password'
	  commands.
	* bgpd.c: (peer_{new,delete) manage TCP-MD5 password
	  (peer_group2peer_config_copy) inherit TCP-MD5 password
	  (peer_password_{un,}set) orchestrate the whole add/remove of TCP-MD5
	  passwords: applying checks, stopping peers, and trying to return
	  errors to UI, etc.
	  (bgp_config_write_peer) save password.
	  Fix missing newline in writeout of neighbor ... port.
2008-07-21 Paul Jakma <paul.jakma@sun.com>
	* sockunion.c: ifdef out various places that converted
	  v4mapped sockets to pure v4. Doesn't seem necessary at all,
	  presumably a workaround for now historical inet_ntop bugs (?)
2008-07-21 Michael H. Warfield <mhw@wittsend.com>
           YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
	* sockopt.{c,h}: (sockopt_tcp_signature) Add TCP-MD5SIG support. |