2004-08-18 Paul Jakma <paul@dishone.st>

        * vty.c: (vty_serv_un) set unix vty socket to nonblocking
          to prevent inadvertent blocking of daemons by use of
          vtysh. TODO: disentangle manual paging from the buffer_write
          path so that unix vty can use this path too and be reliable.

2004-08-17  Greg Troxel  <gdt@fnord.ir.bbn.com>

        * update-autotools: print tools versions to aid people in sending
        bug reports.

SVN revisions 916-920 from Zebra. ABR support is almost done.

Include padding in control message length.
Inspired by netbsd:src/usr.sbin/rtadvd/rtadvd.c (from KAME).

2004-08-11  Greg Troxel  <gdt@fnord.ir.bbn.com>

* rtadv.c (rtadv_send_packet): Allocate space for control messages
more carefully; it was wrong on NetBSD/sparc where CMSG alignment
is to 8 bytes instead of 4, and overwriting the address.  Use the
provided macros for determining lengths.

Add extra padding in cmsgdata buffer; there wasn't enough on
NetBSD/sparc, resulting in overwriting the address.

Add code to check if overwriting is happening, at least for now.

Set msg_flags to MSG_DONTROUTE; they had been stack garbage.

I am bored. But I hate questions why there are man pages from version 0.88.
Vtysh man page looks already how it should look IMHO, the rest need more
work.

Remove pointless privilege lower call. Thanks to Cougar for noticing.

2004-08-04 Paul Jakma <paul@dishone.st>

        * ospf_spf.c: (ospf_spf_consider_nexthop) Add comment about issue.
          Compare only against list head - all nexthops must be same cost
          anyway, fixes a reference-listnode-after-delete bug noted by
          Kir Kostuchenko.
          (ospf_nexthop_calculation) Use ospf_spf_consider_nexthop for all
          candidates attached to root.

SVN revision 907 from Zebra cvs repository.

oops, forgot to update changelog.

2004-07-31 Christian Hammers <ch@lathspell.de>

* snmp.texi: updated (rewritten almost)

enable irdp

2004-07-27 Paul Jakma <paul@dishone.st>

        * ospf_packet.c: (ospf_ls_upd_send_queue_event) fix thinko from
          last fix for ospfd wedging due to oversize LSAs: dont list loop on
          ospf_ls_upd_queue_send() - guaranteed segfault.

Paul Jakma: forgot an obvious TODO

Paul Jakma: update TODO.. there's plenty.

2004-07-27 Paul Jakma <paul@dishone.st>

* Makefile.am: tests/ must be in dist or building from dist breaks

2004-07-27 Paul Jakma <paul@dishone.st>

        * ospf_opaque.c: (ospf_opaque_lsa_flush_schedule) do not NULL out
          the LSA as then free_opaque_info_per_id() can never unlock (and
          free) the LSA. Reported by Gunnar Stigen.

2004-07-26 Paul Jakma <paul@dishone.st>

* configure.ac: reenable tests/Makefile
* tests/Makefile.am: automake file for tests dir
* tests/.cvsignore: update

Use ifp->mtu6 in ospf6d and ripngd.

add .libs

(somewhat unrelated cleanups, but all are very minor)

2004-07-23  Greg Troxel  <gdt@poblano.ir.bbn.com>

* */Makefile.am: Use ../dir/libfoo.la, rather than "-L../dir
-lfoo", to avoid linking against installed libraries from a
previous version.

* {lib,ospfd,ospfclient}/Makefile.am: explicitly define the shared
library version number to be 0.0

* configure.ac: remove spurious , so extract.pl is chmod'd +x.

* HACKING: explain shared library versioning rules

2004-07-23  Greg Troxel  <gdt@poblano.ir.bbn.com>

* sockopt.c (getsockopt_ipv4_pktinfo_ifindex): Make this compile
on NetBSD, and add comments to make it less confusing.  Change the
sense of the SUNOS_5 test to make parallel structure between the
variables and the code.

2004-07-23 Sowmini Varadhan <Sowmini.Varadhan@Sun.COM>

        * if_ioctl_solaris.c: HAVE_IPV6 ifdef fixups
        * zserv.c: ditto
        * ioctl_solaris.c: ditto.
        * interface.c: cast for LLADDR
        * interface.h: Add guards, include redistribute.h and remove
          extraneous definitions of zebra_interface_{up,down}_update
        * ioctl.h: Add AF_IOCTL define for non SOLARIS_IPV6
        * redistribute.h: include dependent header, zserv.h
        * zserv.h: include dependent header, rib.h

2004-07-23 Paul Jakma <paul@dishone.st>

        * irdp_main.c: use setsockopt_pktinfo_ipv4
        * irdp_packet.c: use SOPT_SIZE_CMSG_PKTINFO_IPV4 and
          getsockopt_pktinfo_ifindex()

Update cvsignore, add .libs

2004-07-23 Paul Jakma <paul@dishone.st>

        * ospf6_main.c: Delete LOG_PERROR, not portable.
        * ospf6_network.c: (ospf6_set_pktinfo) use setsockopt_ipv6_pktinfo

2004-07-23 Paul Jakma <paul@dishone.st>

        * ospf_network.c: Replace PKTINFO/RECVIF with call to
          setsockopt_pktinfo
        * ospf_packet.c: Use getsockopt_pktinfo_ifindex and
          SOPT_SIZE_CMSG_PKTINFO_IPV4.

2004-07-23 Paul Jakma <paul@dishone.st>

        * sockopt.h: Add SOPT_SIZE_CMSG_PKTINFO{_IPV{4,6}} define, for
          sizeof pktinfo as appropriate, to be used when allocating msg
          buffers. export setsockopt_pktinfo() and
          getsockopt_pktinfo_ifindex()
        * sockopt.c: (setsockopt_pktinfo_ifindex) new function to portably
          set received ifindex sock option.
          (getsockopt_pktinfo_ifindex) portably retrieve ifindex.
          (getsockopt_cmsg_data) retrieve indicated control info from
          message header.
          (getsockopt_ipv6_pktinfo_ifindex) ipv6 version of above.
          (setsockopt_ipv4_pktinfo) v4 version
          (setsockopt_pktinfo) the exported version
          (getsockopt_ipv4_pktinfo_ifindex) v4 specific version
          (getsockopt_pktinfo_ifindex) the exported version

2004-07-14 Paul Jakma <paul@dishone.st>

        * sigevent.c: (quagga_signal_handler) add a global caught flag, set
          the flags to a constant rather increment to be kinder.
          (quagga_sigevent_process) new function, to do core of what
          quagga_signal_timer did. dont block signals at all as sig->caught
          is volatile sig_atomic_t and should be safe to access from signal
          and normal contexts. The signal blocking is unneeded paranoia, but
          is left intact under an ifdef, should some platform require it.
          Check global caught flag before iterating through array.
          (quagga_signal_timer) nearly everything moved to
          quagga_sigevent_process. Left in under ifdef, in case some
          platform could use a regular timer check for signals.
        * sigevent.h: quagga_sigevent_process declaration.
        * thread.c: (thread_fetch) check for signals at beginning of
          scheduler loop, check for signals if select returns EINTR.

2004-07-22 Paul Jakma <paul@dishone.st>

        * configure.ac: modify default CFLAGS to be compiler agnostic
          build Makefile for tests/ subdir.

2004-07-20 Paul Jakma <paul@dishone.st>

* redhat/quagga.spec.in: update for shared lib install, *.la
  should be in -devel

2004-07-17 Paul Jakma <paul@dishone.st>

* if.c: (if_cmp_func) Add comment suggested by Greg Troxel.

2004-07-14 Paul Jakma <paul@dishone.st>

        * ospf_packet.c: (ospf_ls_upd_send_queue_event) Partial fix for
          problem reported by Peter Frost amongst others, where function
          will spin indefinitely if update list contains LSAs greater than
          MTU-headers or other condition leading to update list never being
          cleared. Problem of what to do with these LSAs remains.
          (ospf_make_ls_upd) add comment about large LSA problem,
          indentation cleanup.

2004-07-14  Greg Troxel  <gdt@poblano.ir.bbn.com>

        * Makefile.am (EXTRA_DIST): Add missing \, so tools stuff is
        really in distfile.

004-07-13  David Wiggins <dwiggins@bbn.com

* kernel_socket.c (rtm_flag_dump): terminate buffer with '\0', not '0'.

This is arguably not a security problem, since strlcat is used to copy
to the buffer -gdt.

2004-07-13  Greg Troxel  <gdt@poblano.ir.bbn.com>

* sigevent.c: Don't block SIGTRAP and SIGKILL.  Blocking SIGTRAP
confuses gdb, at least on NetBSD 2.0_BETA, where the block
succeeds.

Creating irdp socket requires more privileges.

1) extract.pl is braindead, but I will not touch it willingly.
2) Following common style is always better anyway.

Merge svn revision 855 from Zebra repository.

cvsignore *.libs in build dirs

2004-07-09 Paul Jakma <paul@dishone.st>

        Merge of GNU Zebra cvs2svn changesets r799, r800 and r807.

        * bgp_dump.c: (bgp_dump_attr) cleanup. return status code. check
          attributes present before printing.
        * bgp_dump.c: update bgp_dump_attr prototype.
        * bgp_packet.c: (bgp_update_receive) init attrstr. check status
          of bgp_dump_attr. Log end-of-rib UPDATEs.

2004-07-09 Paul Jakma <paul@dishone.st>

        * jhash.{c,h}: New files. Bob Jenkins' public domain hashing
          function, as implemented in linux kernel by David Miller.

2004-07-09 Paul Jakma <paul@dishone.st>

        * Merge Kunihiro's 'show route-map' change and add
          compatibility aliases for route-map continue

more cvsignores

Update cvsignore's for *.lo and *.la - libtool files

2004-07-09 Juris Kalnins <juris@mt.lv>

        * if.c: (if_cmp_func) fix for interface names where name is same,
          but one has no number, eg "devtyp" and "devtyp0".

2004-07-09 Sowmini Varadhan <sowmini.varadhan@sun.com>

        * bgp_packet.c: (bgp_collision_detect) Send NOTIFY on new socket
          if that is connection we're closing.
          (bgp_read) invalid marker check applies to KEEPALIVE too.
        * bgp_route.c: Ignore multicast NRLI, dont send NOTIFY.

2004-07-01  Greg Troxel  <gdt@fnord.ir.bbn.com>

        * ripng_interface.c (ripng_multicast_join): Use privs to do join,
        to work around bug on gif(4) on NetBSD 1.6.2.

add appearance of shlibs

There is no need to introduce new file of course, document big changes in
NEWS file.

Compile libospf shared as well.

Add libtool support.

libzebra and libospfapiclient are now built shared, and linked shared
with the daemons.  This reduces the memory needed when running
multiple daemons; each daemon is at least 150k smaller.  Static
libraries are still built, and libtool should use them on platforms
which don't have shared libaries.  As with autoconf, the user of a
distribution does not need libtool; one just needs that to build from
CVS.

libospf.a is still a non-shared library, and still installed, not
because that makese sense, but because I don't understand why it is
the way it is now.

Note that the tree was tagged 'libtool-before' just before this commit.

Use EXTRA_DIST to distribute files, and don't list directories.
Remove disthook that removed the crud that was copied in due to this.

Make vtysh work on NetBSD, and minor cleanups:

  if --enable-vtysh, look for perl, and substitute into vtysh/extract.pl,
  rather than assuming perl is in /usr/bin

  Look for tputs in libtermcap and libcurses, in addition to
  previously-searched locations.  Follow GNU readlines search order.

  Clean up --enable-vtysh definition in configure.ac.

  Add vtysh/vtysh_cmds.c to CLEANFILES.  This is important if
  extract.pl fails, so that 'make clean && make' will regenerate
  vtysh_cmds.c rhather than using the old zero-length file.

Use -rf with autom4te.cache.

2004-06-21 Paul Jakma <paul@dishone.st>

        * ChangeLog: fix my last update config.h -> zebra.h ;)
        * zebra.h: Fix gcc check.

Remove ifdef's HAVE_NSSA. NSSA support is stable enough.

Removing code which looked at current dir for config file before attempting
to read system one.

Zebra daemon has access lists.

We are going to change some stuff in incompatible way, so it's good idea to
document this stuff. Nobody reads loooong changelog ;).

OK, here it is - irdp support. But don't expect me to fix any bugs in it.

2004-06-11 Sowmini Varadhan <sowmini.varadhan@sun.com>

        * filter.c: (access_list_remark_cmd) buffer_putstr doesnt need cast
          to u_char. (ipv6_access_list_remark_cmd) ditto.
          if.c: ditto
        * network.c: (readn/writen) pointer arg should be type u_char.
        * plist.c: needs to include stream.h, not declare stream functions
          internally.
          (various) Add static qualifier to internal functions.
          (prefix_list_type_str) extraneous breaks in switch statement.
          (ip_prefix_list_description_cmd) buffer_putstr doesnt need cast
        * stream.h: depends on plist.h and export stream_put_prefix
        * vty.c: (vty_<telnet option build functions>) should use
          unsigned char, telnet options are 0 -> 255.
        * zclient.c: various u_char<->char type cleanups.
        * zebra.h: Having to define CMSG_* can apply to more than just
          BSDI_NRL.
        * ripd.c: (rip_distribute_update_all) distribute list hook
          function pointer prototype requires struct prefix_list * arg.
          (rip_distribute_update_all_wrapper) update to pass required arg,
          NULL.

2004-06-09 Paul Jakma <paul@dishone.st>

* config.h: __attribute__ is a gcc'ism

2004-06-06 Paul Jakma <paul.jakma@sun.com>

* ripd.h: Add define for the RIPv2 Authentication Data family
  Move the auth type defines up to where other defines live.
  Add RIP_AUTH_MD5_COMPAT_SIZE, for backwards compatible
  md5->auth_len size. Add md5_auth_len field to struct
          rip_interface: (rip_interface_new) Init md5_auth_len to compatible
          size.
          (ip_rip_authentication_mode_cmd) Extended to handle setting
          md5 auth-length. Appropriate aliases added.
          (no_ip_rip_authentication_mode_cmd) Reset md5_auth_len to
          compatible size.
          (rip_interface_config_write) Teach it about md5_auth_len.
          _always_ write out the auth-length, so that everyone will get
          the setting in their config file, and hence allow for a future
          change of default for md5_auth_len to be less painful - every md5
          user will have this setting in their config file.
          ripd.c: (rip_packet_dump) Change nasty hard coded constants to
          symbolic defines. Change various tests of 'ntoh.(variable) ==
          constant' to test 'variable == ntoh.(constant)'. Clean up
          indentation on some long lines.
          (rip_auth_simple_password) ditto.
          (rip_auth_md5) ditto, also add length argument and sanity check
          md5 data offset field. Sanity check md5 auth length, accept RFC
          or old-ripd/cisco lengths.
          (rip_auth_md5_set) as per (rip_packet_dump), also write out
          the configured md5 auth length for the interface (old-ripd or rfc)
          (rip_read) as per (rip_packet_dump)
          (rip_write_rte) ditto
          (rip_response_process) ditto
          (rip_write_rte) ditto

Fix stray dollar sign that had crept into OSPF6D_OPTS and prevented ospf6d
from starting.

CVS ignore Linux NFS silly-delete .nfs* files

Fix module path to use $ISA (needed to make vtysh work on, eg, FC2 x86_64)

2004-06-06 Paul Jakma <paul@dishone.st>

        * ospf_dump.c,ospf_lsa.c: Fix typos of merge of previous.
          ospf_flood.c: (ospf_process_self_originated_lsa) fix zlog format.

Remove autom4te.cache to avoid problems with newer auto* tools.

Add .cvsignore for isisd/topology

2004-06-04 Paul Jakma <paul@dishone.st>

        * type mismatch fixes

2004-06-04 JJ Ludman <jacques.ludman@sun.com>

        * ripd.c: Interoperability fix. Correct value for MD5 auth length
          is 16. Accept packets with this set to >= 16, and set to 16
          ourselves.

2004-05-31 Sagun Shakya <sagun.shakya@sun.com>

        * ospf_dump.c: (ospf_lsa_header_dump) LOOKUP can return null if
          index is out of range.
          ospf_flood.c: endianness fix
          ospf_lsa.c: Missing ntohl's on (struct lsa *)->data->ls_seqnum
          in various places.

Forgot to commit ChangeLog with last doc/ commit.

2004-05-31 Sowmini Varadhan <sowmini.varadhan@sun.com>

        * ripd.c: Fixup compile warnings
        * rip_routemap.c: Ditto

2004-05-27 Paul Jakma <paul@dishone.st>

        * Correct RIP port, s/521/520/, reported by xantius@xantius.com

Merge graceful restart capability display and some small fixes from Zebra
repository by Rivo Nurges.

Merge bgpd changeset 1185 from Zebra repository by Rivo Nurges.

Merge bgpd changeset 1184 from Zebra repository by Rivo Nurges.

Merge bgpd changeset 1176 from Zebra repository by Rivo Nurges.

Fixing crash fix attempt. It's still mess and logic is wrong and ... but at
least periodic spf works again.

Commit 100% working part of patch from Laurent Rabret ([quagga-dev 830]).
Also tiny temporary fix from me to spf thread crash. Will look at it later.

Make it compile in NetBSD and OpenBSD.

Update to latest (really :) ospf6d from Zebra repository and some tiny
fixes.

Forgot these.

Ospf6d merge from Zebra repository with added privs stuff and merged
zclient changes.

Start of new ospf6d merge from Zebra.

Router advertisement cleanup. Removed compatibility command because it's
very annoying to have both obsolete "prefix-advertisement" and "prefix" in
completion. We will break configuration anyway with secondary cleanup, so
just remember mention it in release notes.

2004-05-13 Paul Jakma <paul@dishone.st>

* HACKING: Add comment about updating ChangeLogs (which this
  commit, ironically, hasnt neglected to do).

2004-05-11 Paul Jakma <paul@dishone.st>

        * ripng_nexthop.h: SUNOS_5 does not define s6_addr32
          catch ifndef(s6_addr32) && defined(SUNOS_5) and define
          it ourselves based on the S10 _KERNEL guarded define.

2004-05-11 Paul Jakma <paul@dishone.st>

        * ioctl.h: Add if_ioctl_ipv6 prototype and AF_IOCTL for SOLARIS_IPV6
        * kernel_socket.c: Fix SAROUNDUP, compiler doesnt like do..while
          RHS in assignments :)
        * redistribute.c: (zebra_interface_delete_update) only used
          if RTM_IFANNOUNCE and NETLINK is available.

2004-05-11 Paul Jakma <paul@dishone.st>

        * Makefile.am: support for IOCTL_METHOD, as per Sowmini's patch.
        * if_ioctl_solaris.c: Fixup some erroneous privilege changes and
          add privs.h header.
        * ioctl_solaris.c: ditto

2004-05-11 Paul Jakma <paul@dishone.st>

        * configure.ac: Add solaris support for the zebra/*_solaris
          method's, based on Sowmini's patches.
        * zebra/Makefile.am: support for IOCTL_METHOD, as per Sowmini's
          patch.

* Make updating redistributions work if prefix list is changed and it's
  used in route-map used to filter redistributions.
* Move ospf_prefix_list_update() function from ospfd.c to ospf_zebra.c.

Workaround for Bugzilla #58.

2004-05-09 Paul Jakma <paul@dishone.st>

* if_ioctl_solaris.c: Add zprivs support.
* ioctl_solaris.c: ditto.

2004-05-09 Sowmini Varadhan <sowmini.varadhan@sun.com>

* if_ioctl_solaris.c: New file, Solaris interface ioctl methods.
* ioctl_solaris.c: New file, Common solaris ioctl methods.

2004-05-09 Sowmini Varadhan <sowmini.varadhan@sun.com>

* ipforward_solaris.c: fix typo of ND variable.

2004-05-09 Paul Jakma <paul@dishone.st>

* ioctl.c: (if_get_mtu) set mtu6 to mtu
* mtu_kvm.c: (if_kvm_get_mtu) set mtu6 to mtu
* rt_netlink.c: (netlink_interface) set mtu6 to mtu
  (netlink_link_change) ditto

2004-05-09 Sowmini Varadhan <sowmini.varadhan@sun.com>

* interface.c: (if_delete_update) only used with HAVE_NETLINK
  and RTM_IFANNOUNCE.
  (if_flag_dump_vty) Solaris IFF_IPV4 and IFF_IPV6 if flags
  (if_dump_vty) print mtu6 if not same as mtu

2004-05-09 Paul Jakma <paul@dishone.st>

* zserv.c: (zsend_route_multipath) Set the nexthop_num
  field correctly. Add NEXTHOP_TYPE_IPV6_IFNAME for v6.
  Conditionally set ZAPI_MESSAGE_METRIC flag - only for adds.
  (zsend_ipv4_add) cruft, deleted.
  (zsend_ipv4_delete) ditto.
  (zsend_ipv6_add) ditto.
  (zsend_ipv6_delete) ditto.

2004-05-09 Sowmini Varadhan <sowmini.varadhan@sun.com>

* zserv.c: (zsend_ipv{4,6}_{add,delete}_multipath) collapsed
  into single zsend_route_multipath function.
  (zsend_interface_{up,down}) collapsed into zsend_interface_update.
  (zsend_interface_address_{add,delete}) collapsed into
  zsend_interface_address.
  (zsend_interface_add) send mtu6.
  (zsend_interface_delete) ditto.
  (zebra_write) remove unused function.
  (various) Apply static qualifier. Add comments.
* zserv.h: Definitions changed as per above.
* redistribute.c: Changes as per zserv.c.