Merge commit 'remotes/jocke/delete_ghost_route'

Merge commit '53d0deceeb855493aa03c59477f8af7f6bb75140'

[ospfd] Restructure opsf_if_update() and ospf_network_run()

    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>

Merge commit 'remotes/jocke/ospf_neighbour'; commit 'remotes/jocke/snmp_crosscompile'

[doc] Update HACKING to reflect SCM changes and latest practice

test

Revert "[bgpd] Add 'bgp open-accept' option, to send OPEN immediately on accepted conns"

Revert commit d664ae1182c29b74b409bc8594b7bd0575e91ce9. An experimental
patch which violates RFC4271 quite badly, but managed to accidently sneak
its way in.

Set destination for PtP links to OSPF_ALLSPFROUTERS.

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.

Ignore host routes to self.

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

Make --enable-snmp cross compile and make libcrypto optional with --without-crypto

Autoconfig work by me, the rest was done by
"Kirill K. Smirnov" <lich@math.spbu.ru>

Fix SEGV in ip ospf neighbor all

Dont use vty->index in show_ip_ospf_neighbor_all_cmd(),
It is invalid, use ospf_lookup() instead.

[trivia] Make 'make dist' happy about ChangeLog expunge

* ChangeLog: Add back a dummy, top-level ChangeLog
* m4/Makefile.am: remove ChangeLog from EXTRA_DIST, why it was
  listed there, who knows.

[administrivia] Update gitignore files

- Add more stuff, archive libraries, etc..

[administrivia] Git should ignore backup files and .loT files

[trivia] Remove ChangeLog files, as this data is now maintained in SCM

[zebra] trivial: fix spelling mistake

[bgpd] remove useless check and return in bgp_config_write_damp

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>

[zebra] trivia: static qualifiers for metaq

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>

[zebra] make some data local

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>

[zebra] ignore dead routes in RIB update

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>

[zebra:linux] netlink: no need to change privs for receive

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>

[zebra:linux] netlink: quiet harmless errors

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>

[zebra:linux] netlink buffer size tweaking only needed on receive socket

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>

[build] Linux netlink doesn't need IF_PROC

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>

[zebra:linux] clean up blocking setting of netlink sockets

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>

[zebra:linux] Filter out unwanted netlink messages by PID

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>

[warnings] Fix various SOS warnings

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.

[lib] Fix mistake in tcp-md5sig commit which broke compile on BSD

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.

[trivia] finish off static'ification of ospf6d and ripngd

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.

[lib] mes_lookup string lookup table argument should be marked const

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>

[trivia] ignore automake generated 'compile' file

[zebra] mark some rt_netlink functions static

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>

[lib] hash compare function arguments ought to be const qualified

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>

[ospfd] Default route needs to be refreshed after neighbour state change

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>

[isisd:DLPI] Try open vanity-named DLPI dev before style 1,2

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>

[ospfd/isisd] Switch to lib/ Fletcher checksum, fixing bug in isisd

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>

[isisd] Fix packet filtering with DLPI

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>

[lib] Add fletcher checksum implementation

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>

[zebra:netlink] Set proto/scope on all route update messages

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>

[vty] Allow delete during password entry

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>

[build] --enable...user/group didn't match help text

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>

[build] Test for GNU-style PIE support in toolchain and enable

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>

[bgpd] Add 'bgp open-accept' option, to send OPEN immediately on accepted conns

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.

[administrivia] Ignore rebase files and .msg
(cherry picked from c5b85efd767b82fcb46b20ac6202c828d91aacb3 commit)

[git administrivia] remove auto-built quagga.info, add to gitignore.

[administrivia] Add .gitignore files, based on .cvsignores.

[bgpd] Fix triggerable crash when compiled with --disable-bgp-announce

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.

[bgpd] fix a couple of trivial compiler warnings

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

[bgpd] TCP-MD5: password vty configuration and initial Linux support

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.

[tools/multiple-bgpd.sh] make it easier to use IPv6 peer addresses

[solaris] Sync SMF bits with OpenSolaris SFW

2008-07-03 Paul Jakma <paul.jakma@sun.com>

* quagga.{xml,init}.in: Sync with OpenSolaris SFW. This changes the
  SMF schema used, from the SMF schema supplied initially with
  Quagga, to the version which was approved by the Sun PSARC for
  inclusion into Solaris. This change is incompatible. Settings,
  such as vty_port, etc. will not transfer over. The Sun schema is
  however supported by Sun, and supports running Quagga in a zone
  with IP instances, and configuration via the routeadm utility.

[vtysh] small fix to previous CMD_AS_RANGE patch, and add missing command

2008-07-03 MIYAJIMA Mitsuharu <miyajima.mitsuharu@anchor.jp>

* extract.pl.in: Fix as-range to match CPP output.
* vtysh.c: Add missing router_bgp_view_cmd alias.

[vty] CMD_AS_RANGE accidently quoted and so not expanded in vty

2008-07-02 MIYAJIMA Mitsuharu <miyajima.mitsuharu@anchor.jp>

* *.c: CMD_AS_RANGE was being used inside command strings, and thus
  not being expanded, fix by dequoting.

[bgpd] Fix double-free crash in bgp_table_finish, seen with rs-client

2008-07-02 Stephen Hemminger <stephen.hemminger@vyatta.com>

* bgp_table.{c,h}: (bgp_table_finish) Take a double pointer and
  scrub pointer in caller, so fixing double-free crashes seen
  with route-server-client configuration.
* *.c: Adjust all callers of bgp_table_finish

2008-07-02 Paul Jakma <paul.jakma@sun.com>

* bgp_table.{c,h}: (bgp_node_delete, bgp_table_free) shouldn't be
  exported.

[zebra] Make BSD link-state deal more gracefully with GIFMEDIA ioctl error

2008-07-01 Paul Jakma <paul.jakma@sun.com>

* ioctl.c: (if_get_flags) Deal more gracefully with failure
  of the BSD link-state SIOCGIFMEDIA ioctl, as some interfaces
  apparently don't implement it (e.g. tun).
  Also, make BSD link-state checking be conditional on the
  'link-detect' interface configuration flag, as it should be.
  Fixes bug #465.

[tools/multiple-bgpd] setup IPv6 advertisments

[administrivia] commit missing ChangeLog update...

[release] Bump version to 0.99.10

2008-06-10 Paul Jakma <paul@jakma.org>

* configure.ac: Bump version to 0.99.10

[tests] BGP MP_(UN)REACH_NLRI unit tests (actual file..)

2008-06-07 Paul Jakma <paul@jakma.org

* bgp_mp_attr_test.c: MP_(UN)REACH_NLRI unit tests

[lib] trivial: add const qualifier to stream_put/write

2008-06-07 Paul Jakma <paul@jakma.org>

* stream.{c,h}: (stream_{put,write}) add const qualifier to source
  argument. Change u_char to void *.

[bgpd] minor changes to bgp_mp_reach_parse

2008-06-07 Paul Jakma <paul@jakma.org>

* bgp_attr.{c,h}: (bgp_mp_{un,}reach_parse) export, for unit tests.
* bgp_attr.c: (bgp_mp_reach_parse) Add logging. Tighten length test
  to bounds check against the attribute length rather than the
  stream length..

[tests] BGP MP_(UN)REACH_NLRI unit tests

2008-06-07 Paul Jakma <paul@jakma.org

* bgp_mp_attr_test.c: MP_(UN)REACH_NLRI unit tests

+ note the meta-queue in NEWS

+ initial edition of meta-queue for RIB updates processing (bug #431)

[bgpd] bug #419: partial aspath-limit incorrectly causes session reset

2008-06-01 jfletche@gmail.com

* bgp_attr.c: (bgp_attr_aspathlimit) fix silly bug in flags check
  that was causing BGP to drop sessions if it received a
  aspath-limit with partial set. Fixes bug #419.

[tools] multiple-bgpd.sh: make paths more configurable

[ripd] Fix mistaken empty string test

2008-05-29 Stephen Hemminger <stephen.hemminger@vyatta.com>

* ripd.c: (rip_auth_md5) fix bogus empty string test

[daemons] Sanity check port number arguments before use

2008-05-29 Martin Nagy <mnagy@redhat.com>

* */*main.c: Sanity check port numbers before using.

[zebra/linux] Use BPF to filter out responses, to try avoid netlink overruns

2008-05-29 Stephen Hemminger <stephen.hemminger@vyatta.com>

* rt_netlink.c: (netlink_install_filter) BPF filter to catch and
  drop responses to zebra's own route messages.
  (kernel_init) add BPF filter on the netlink socket.

+ fix the set statement name

+ [bgpd] Added new route-map set statement: "as-path ignore"

[bgpd] remove unnecessary 0 entries from struct message's

2008-03-13 Paul Jakma <paul.jakma@sun.com>

* (various) Remove 0 entries from struct message's, unneeded due to
  recent improvements in mes_lookup/LOOKUP.

[ripd] remove unnecessary 0 entries from struct message's

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.

[lib] Fix the struct message LOOKUP function to be more robust

2008-02-28 Paul Jakma <paul.jakma@sun.com>

* log.c: (mes_lookup) Sowmini Varadhan diagnosed a problem where
  this function can cause a NULL dereference, on lookups for unknown
  indices, or messages with NULL strings. Can occur, e.g., debug
  logging code when processing received messages. Fixed to accept a
  pointer to a default string to be used if there is no match.
* log.h: LOOKUP adjusted to match

[lib/linklist] Enforce "nodes must have data" invariant more rigorously

2008-02-28 Paul Jakma <paul.jakma@sun.com>

* linklist.c: This implementation expects that the data pointer not
  be null, e.g. listgetdata() asserts this. The list add methods
  don't apply the same sanity check.

  Noted by Jim Carlson in bug #437.

+ fix bug#326 by rib_lookup_and_pushup()

[c++] remove/rename some names in headers that clash with C++ reserved words

2008-01-30 Peter Szilagyi <sp615@hszk.bme.hu>

* lib/stream.h: Remove named 'new' parameter in prototype
  for c++ header compatibility.
* ospfd/ospf_opaque.h: ditto
* ospfd/ospfd.h: Renamed struct export to _export for c++
  header compatibility.
* ospf6d/ospf6_area.h: ditto

[tests:bgp] Further tests for 0 sized as_path segments

2008-02-23 Paul Jakma <paul.jakma@sun.com>

* aspath_test.c: Test for 0-ASN sequences that still have data.

[isisd] Bug #437: fix ssert caused by bad list management

2008-01-29 James Carlson <james.d.carlson@sun.com>

* Fix bug #437, assert due to bogus index management
* isis_flags.c: (flags_initialize) new
* (flags_get_index) fix off by one, leading to list assert
  on null node data.
  (flags_free_index) ditto.
* isisd.c: (isis_area_create) use flags_initialize
  (isis_area_destroy) deconfigure circuits when
  taking down area.

[bgpd] fix crash on startup if compiled IPv4-only

2008-01-29 Jorge Boncompte <jorge@dti2.net>

* bgp_network.c: (bgp_socket) IPv4-only version crashes if -l is not
  used as address will be null.

[solaris] add missing makefile.am dependency

2008-01-13 Paul Jakma <paul.jakma@sun.com>

* Makefile.am: pkg target should depend on the 'depend.%' files.
  Crops up now that solaris/ doesn't get descended into by
  general Quagga build.

[link-detect] Improve BSD support.
2008-01-11 Ingo Flaschberger <if@xip.at>

* configure.ac: Improve HAVE_BSD_LINK_DETECT test.

[link-detect] Improve BSD support.
2008-01-11 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* lib/zebra.h: Revert previous change, no need to include
  <net/if_media.h> here.
* zebra/ioctl.c: If HAVE_BSD_LINK_DETECT is defined, include
  <net/if_media.h>
  (if_get_flags) Remove debug messages about BSD link state.
* zebra/kernel_socket.c: (bsd_linkdetect_translate) If link state
  is unknown, we should set the IFF_RUNNING flag.

[link-detect] Try to get BSD link-detect to work properly.

2008-01-10 Ingo Flaschberger <if@xip.at>

* configure.ac: Define HAVE_BSD_LINK_DETECT if <net/if_media.h> is
  present.
* lib/zebra.h: If HAVE_BSD_LINK_DETECT is defined,
  include <net/if_media.h>.
* zebra/ioctl.c: (if_get_flags) If HAVE_BSD_LINK_DETECT, use the
  SIOCGIFMEDIA ioctl to ascertain link state.
* zebra/kernel_socket.c: (bsd_linkdetect_translate) New function to
  map the ifm_data.ifi_link_state value into the IFF_RUNNING flag.
  (ifm_read) Call bsd_linkdetect_translate to fix the IFF_RUNNING
  flag before calling if_flags_update.

[link-detect] Static interface routes should behave properly with link-detect.

2008-01-08 Michael Larson <mike@vyatta.com>

* zebra_rib.c: (nexthop_active_check) Replace if_is_up with
  if_is_operative to solve problems with static interface
  routes not behaving properly with link-detect.

[lib] add mising UL qualifier to numerical constant

2008-01-08 Pavol Rusnak <prusnak@suse.cz>

* memory.c: (mtype_memstr) Fix accidental shift past width of type,
  constant should have been forced to UL, rather than being left to
  default to int.

[bgpd] Fix number of DoS security issues, restricted to configured peers.

2007-12-22 Paul Jakma <paul.jakma@sun.com>

* Fix series of vulnerabilities reported by "Mu Security
  Research Team", where bgpd can be made to crash by sending
  malformed packets - requires that bgpd be configured with a
  session to the peer.
* bgp_attr.c: (bgp_attr_as4_path) aspath_parse may fail, only
  set the attribute flag indicating AS4_PATH if we actually managed
  to parse one.
  (bgp_attr_munge_as4_attrs) Assert was too general, it is possible
  to receive AS4_AGGREGATOR before AGGREGATOR.
  (bgp_attr_parse) Check that we have actually received the extra
  byte of header for Extended-Length attributes.
* bgp_attr.h: Fix BGP_ATTR_MIN_LEN to account for the length byte.
* bgp_open.c: (cap_minsizes) Fix size of CAPABILITY_CODE_RESTART,
  incorrect -2 left in place from a development version of as4-path
  patch.
* bgp_packet.c: (bgp_route_refresh_receive) ORF length parameter
  needs to be properly sanity checked.
* tests/bgp_capability_test.c: Test for empty capabilities.

+ fixed no_set_aspath_prepend() to correctly match existing "set" statement

[bgpd] Fix typo in previous commit to bgp_main.c

[bgpd] small fix for crash if 'listenon' argument is not given

2007-11-13 Paul Jakma <paul.jakma@sun.com>

* bgp_main.c: Small fix, format string parameters should never
  be NULL - it's not strictly valid and some libc's don't let
  you away with.

+ fixed bug #418 (changing address on an existing interface doesn't cause existing static routes to be revalidated)

2007-10-30 Nick Hilliard <nick@inex.ie>

* bgp_main.c: Add 'listenon' argument, to pass address to bind
  to.
* bgp_network.c: (bgp_socket) Extend to take bind address.
* bgpd.c: (bgp_init) Pass stored address.
* bgpd.h: (struct bgp_master) storage for bind address

2007-11-01 Paul Jakma <paul.jakma@sun.com>

* tools/multiple-bgpd.sh: New, quick script to launch a bunch of
  bgpds.

+ fix the bug reported by Milan Kocian (IPv6 route handling was broken by the RIB debug changeset).
after

[ospf6d] Fix removal of defunct ASBR routes

2007-10-22 Phil Spagnolo <phillip.a.spagnolo@boeing.com>

* ospf6_asbr.c: (ospf6_asbr_lsentry_remove) Remove shortcut
  of LSDB search - it's based on assumption non-BEST routes
  can't have ASBR routes, which appears to be wrong. Safest to
  search.

[snmp-smux] Fix problems if 'smux peer ...' is issued multiple times

2007-10-22 Lorenzo Colitti <lorenzo@colitti.com>

* smux.c: (smux_stop) Avoid cancelling a defunct thread pointer
  (smux_start) Stop smux before trying to start it, possibly again.

[news] Fix top-line version

[tests] Forgot to commit ecommunity_tests.c

[bgpd] Merge AS4 support

2007-10-14 Paul Jakma <paul.jakma@sun.com>

* NEWS: Note that MRT dumps are now version 2
* (general) Merge in Juergen Kammer's AS4 patch.

2007-09-27 Paul Jakma <paul.jakma@sun.com>

* bgp_aspath.c: (assegment_normalise) remove duplicates from
  from sets.
  (aspath_reconcile_as4) disregard a broken part of the RFC around
  error handling in path reconciliation.
* aspath_test.c: Test dupe-weeding from sets.
  Test that reconciliation merges AS_PATH and AS4_PATH where
  former is shorter than latter.

2007-09-26 Paul Jakma <paul.jakma@sun.com>

* aspath_test.c: Test AS4_PATH reconcilation where length
  of AS_PATH and AS4_PATH is same.

2007-09-25 Paul Jakma <paul.jakma@sun.com>

* bgp_open.c: (peek_for_as4_capability) Fix to work.
* bgp_packet.c: (bgp_open_receive) Fix sanity check of as4.
* tests/bgp_capability_test.c: (general) Extend tests to validate
  peek_for_as4_capability.
  Add test of full OPEN Option block, with multiple capabilities,
  both as a series of Option, and a single option.
  Add some crap to beginning of stream, to prevent code depending
  on getp == 0.

2007-09-18 Paul Jakma <paul.jakma@sun.com>

* bgp_open.c: (bgp_capability_as4) debug printf inline with others.
  (peek_for_as4_capability) There's no need to signal failure, as
  failure is better dealt with through full capability parser -
  just return the AS4, simpler.
* bgp_packet.c: (bgp_open_receive) Update to match
  peek_for_as4_capability change.
  Allow use of BGP_AS_TRANS by 2b speakers.
  Use NOTIFY_OPEN_ERR rather than CEASE for OPEN parsing errors.
  (bgp_capability_msg_parse) missing argument to debug print
  (bgp_capability_receive) missing return values.
* tests/bgp_capability_test.c: (parse_test) update for changes to
  peek_for_as4_capability

2007-07-25 Paul Jakma <paul.jakma@sun.com>

* Remove 2-byte size macros, just make existing macros take
  argument to indicate which size to use.
  Adjust all users - typically they want '1'.
* bgp_aspath.c: (aspath_has_as4) New, return 1 if there are any
  as4's in a path.
  (aspath_put) Return the number of bytes actually written, to
  fix the bug Juergen noted: Splitting of segments will change
  the number of bytes written from that already written to the
  AS_PATH header.
  (aspath_snmp_pathseg) Pass 2-byte flag to aspath_put. SNMP
  is still defined as 2b.
  (aspath_aggregate) fix latent bug.
  (aspath_reconcile_as4) AS_PATH+NEW_AS_PATH reconciliation
  function.
  (aspath_key_make) Hash the AS_PATH string, rather than
  just taking the addition of assegment ASes as the hash value,
  hopefully sligthly more collision resistant.
  (bgp_attr_munge_as4_attrs) Collide the NEW_ attributes
  together with the OLD 2-byte forms, code Juergen
  had in bgp_attr_parse but re-organised a bit.
  (bgp_attr_parse) Bunch of code from Juergen moves
  to previous function.
  (bgp_packet_attribute) Compact significantly by
  just /always/ using extended-length attr header.
  Fix bug Juergen noted, by using aspath_put's
  (new) returned size value for the attr header rather
  than the (guesstimate) of aspath_size() - the two could
  differ when aspath_put had to split large segments, unlikely
  this bug was ever hit in the 'wild'.
  (bgp_dump_routes_attr) Always use extended-len and
  use aspath_put return for header length. Output 4b ASN
  for AS_PATH and AGGREGATOR.
* bgp_ecommunity.c: (ecommunity_{hash_make,cmp}) fix
  hash callback declarations to match prototypes.
  (ecommunity_gettoken) Updated for ECOMMUNITY_ENCODE_AS4,
  complete rewrite of Juergen's changes (no asdot support)
* bgp_open.c: (bgp_capability_as4) New, does what it says
  on the tin.
  (peek_for_as4_capability) Rewritten to use streams and
  bgp_capability_as4.
* bgp_packet.c: (bgp_open_send) minor edit
  checked (in the abstract at least) with Juergen.
  Changes are to be more accepting, e.g, allow AS_TRANS on
  a 2-byte session.
* (general) Update all commands to use CMD_AS_RANGE.
* bgp_vty.c: (bgp_clear) Fix return vals to use CMD_..
  Remove stuff replicated by VTY_GET_LONG
  (bgp_clear_vty) Return bgp_clear directly to vty.
* tests/aspath_test.c: Exercise 32bit parsing. Test reconcile
  function.
* tests/ecommunity_test.c: New, test AS4 ecommunity changes,
  positive test only at this time, error cases not tested yet.

2007-07-25 Juergen Kammer <j.kammer@eurodata.de>

* (general) AS4 support.
* bgpd.h: as_t changes to 4-bytes.
* bgp_aspath.h: Add BGP_AS4_MAX and BGP_AS_TRANS defines.
* bgp_aspath.c: AS_VALUE_SIZE becomes 4-byte, AS16_VALUE_SIZE
  added for 2-byte.
  Add AS16 versions of length calc macros.
  (aspath_count_numas) New, count number of ASes.
  (aspath_has_as4) New, return 1 if there are any as4's in a
  path.
  (assegments_parse) Interpret assegment as 4 or 2 byte,
  according to how the caller instructs us, with a new
  argument.
  (aspath_parse) Add use32bit argument to pass to
  assegments_parse. Adjust all its callers to pass 1, unless
  otherwise noted.
  (assegment_data_put) Adjust to be able to write 2 or 4 byte
  AS, according to new use32bit argument.
  (aspath_put) Adjust to write 2 or 4.
  (aspath_gettoken) Use a long for passed in asno.
* bgp_attr.c: (attr_str) Add BGP_ATTR_AS4_PATH and
  BGP_ATTR_AS4_AGGREGATOR.
  (bgp_attr_aspath) Call aspath_parse with right 2/4 arg, as
  determined by received-capability flag.
  (bgp_attr_aspath_check) New, code previously in attr_aspath
  but moved to new func so it can be run after NEW_AS_PATH
  reconciliation.
  (bgp_attr_as4_path) New, handle NEW_AS_PATH.
  (bgp_attr_aggregator) Adjust to cope with 2/4 byte ASes.
  (bgp_attr_as4_aggregator) New, read NEW_AGGREGATOR.
  (bgp_attr_parse) Add handoffs to previous parsers for the two
  new AS4 NEW_ attributes.
  Various checks added for NEW/OLD reconciliation.
  (bgp_packet_attribute) Support 2/4 for AS_PATH and
  AGGREGATOR, detect when NEW_ attrs need to be sent.
* bgp_debug.{c,h}: Add 'debug bgp as4'.
* bgp_dump.c: MRTv2 support, unconditionally enabled, which
  supports AS4. Based on patches from Erik (RIPE?).
* bgp_ecommunity.c: (ecommunity_ecom2str) ECOMMUNITY_ENCODE_AS4
  support.
* bgp_open.c: (peek_for_as4_capability) New, peek for AS4
  capability prior to full capability parsing, so we know which
  ASN to use for struct peer lookup.
  (bgp_open_capability) Always send AS4 capability.
* bgp_packet.c: (bgp_open_send) AS4 handling for AS field
  (bgp_open_receive) Peek for AS4 capability first, and figure
  out which AS to believe.
* bgp_vty.c: (bgp_show_peer) Print AS4 cap
* tests/aspath_test.c: Support asn32 changes, call aspath_parse
  with 16 bit.
* vtysh/extract.pl: AS4 compatibility for router bgp ASNUMBER
* vtysh/extract.pl.in: AS4 compatibility for router bgp ASNUMBER
* vtysh/vtysh.c: AS4 compatibility for router bgp ASNUMBER

+ pidfiles are now always created with 0644 perms instead if LOGFILE_MASK (0600)

+ rib_process() speedup for multi-nexthop route nodes