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.

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

       * bgp_zebra.c: (bgp_interface_address_add) sync to zclient changes
         (bgp_interface_address_delete) ditto.
         (bgp_zebra_announce) ditto.
         (bgp_zebra_withdraw) ditto.
       * isis_zebra.c: Sync with zclient changes.
       * zclient.c (zapi_ipv4_route) Follow Sowmini's lead and describe
         message format.
       * ospf6_zebra.c: Sync to zclient changes
       * ospf_zebra.c: Sync with lib/zclient changes
       * rip_zebra.c: sync with zclient changes.
       * rip_interface.c: ditto.
       * ripng_{interface,zebra}.c: sync with zclient changes

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

       * zclient.c: (zapi_ipv4_add) collapsed into zapi_ipv4_route
         (zapi_ipv4_delete) ditto.
         (zapi_ipv4_route) add/delete a route by way of cmd arg.
         (zapi_ipv6_add) collapsed into zapi_ipv6_route.
         (zapi_ipv6_delete) ditto.
         (zapi_ipv6_route) add/delete a route by way of cmd arg.
         (zebra_interface_address_delete_read) collapsed into
         zebra_interface_address_read.
         (zebra_interface_address_delete_read) ditto.
         (zebra_interface_address_read) read address add/delete messages
         by way of type argument. Describe command message format.
         (zebra_interface_add_read) Unconditionally read new ifmtu6 field.
         Describe command message format.
         (zebra_interface_state_read) Unconditionally read new ifmtu6 field.
         (zclient_redistribute_set) Collapsed into zclient_redistribute
         (zclient_redistribute_unset) ditto
         (zclient_redistribute) set/unset redistribution.
         (zclient_redistribute_default_set) Collapsed into
         zclient_redistribute_default.
         (zclient_redistribute_default_unset) ditto.
         (zclient_redistribute_default) Redistribute default set/unset.
       * zclient.h: delete zapi_ipv{4,6}_add, zapi_ipv{4,6}_delete. Add
         zapi_ipv{4,6}_route. delete zclient_redistribute_set/unset. Add
         zclient_redistribute. Ditto for
         zclient_redistribute_default_{set/unset}.

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

        * command.c: (sockunion_getsockname) use socklen_t for len.
          (sockunion_getpeername) ditto.

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

        * if.h: Add mtu6 field to struct interface, IPv6 MTU may differ
          from IPv4, and Solaris treats the MTU's differently.
          Add connected_add_by_prefix, for use by later patch.
        * if.c: (connected_add_by_prefix) Add prefix to connected list.
          (if_flag_dump) Solaris: Dump IFF_IPv4/6 flag
          (if_dump) Dump mtu6 flag, for HAVE_IPV6.

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

        * ospf_network.c: (ospf_sock_init) Check whether IP_HDRINCL is
          defined. Warn at compile and runtime. Use
          IPTOS_PREC_INTERNETCONTROL otherwise.

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

* ospf_interface.c: (ospf_vl_set_params) Catch changes of interface
          address for either end of a virtual-link, and hence potential cost
          changes.

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

        * ospf_packet.c:  (ospf_associate_packet_vl) cleanup, move
          some of the checks up to ospf_read, return either a
          virtual link oi, or NULL.
          (ospf_read) Cleanup, make it responsible for checks. Remove
          the nbr lookup - moved to ospf_neighbor. Adjust all nbr
          lookups to use new wrappers exported by ospf_neighbor.
        * ospf_neighbor.h: Add ospf_neigbour_get and ospf_nbr_lookup.
        * ospf_neighbor.c: (ospf_neigbour_get) Index ospf_interface
          neighbour table by router-id for virtual-link ospf_interfaces,
          not by peer_addr (which breaks for asymmetric vlinks)
          (ospf_nbr_lookup) add a wrapper for nbr lookups to deal with
          above.

2004-05-05 Anthony.Golia@morganstanley.com

        * ripd.c: (rip_update_jitter) Bound jitter to a more sensible
          value, eg 1/4 of update time.

2004-05-04 Andrew J. Schorr <aschorr@telemetry-investments.com>

* redhat/quagga.logrotate: fix paths

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

        * ripd.c: (rip_rte_process) fix typo in merge of previous patch
          and run function through indent.

2004-05-03 Daniel Roesen <dr@cluenet.de>

        * bgp_fsm.c: (bgp_stop) Reset uptime only on transition from
          Established so that it reflects true downtime (rather time
          since last transition, eg Active->Idle)

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

* bump version to 0.96.5

2004-03-19 Jean-Yves Simon <lethalwp@tiscali.be>

        * ripd.c: make ripd also check on administrative distance of his
          own links to update routes.

2004-05-01 rivo nurges <rix@estpak.ee>

         * bgp_route.c: fix UNH IOL BGP-4.1.12f

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

        * Revert the attempted clean-up of the dummy peer hack, reverts
          patchsets 435 (see 2004-02-17 below) and 456.

More supported RFCs added.

Don't ignore reject/bh routes. Discussed with Paul in IRC.

2004-04-21 Boris Kovalenko <boris@tagnet.ru>

        * daemon.c: (daemon) fix check for error return from setsid

Some ospfd fixes - [quagga-dev 1032], [quagga-dev 1048], [quagga-dev 1058].

Update cease codes in bgp_debug.c file as well. Just update to patch from
Rivo.

2004-04-16 rivo nurges <rix@estpak.ee>

        * bgpd.h: update cease subcodes to draft-ietf-idr-cease-subcode-05
        * bgpd.h, bgpd.c, bgp_route.c, bgp_route.h: fix UNH IOL BGP-4.1.6a

Fix typo to make it compile.

Fix debug message and make it conditional.

Ooops. Check if area-id format is valid.

****! What else I will discover while documenting this code? Remove useless
warnings and update documentation.

Don't assume anything. I'm tired ...

Document summarizing and filtering ABR commands. Native speakers, please
review and fix my awful english ;).

Fix obviously wrong info in documentation.

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

        * ospf_spf.h: Add backlink field to struct vertex
        * ospf_spf.h: (ospf_vertex_new) initialise backlink
          (ospf_lsa_has_link) return index of link back to
          vertex V from candidate vertex W, or -1 if no link exists.
          (ospf_spf_next) save backlink index for candidate vertex
        * ospf_interface.c: (ospf_vl_set_params) Use the backlink index
          to determine correct address for virtual-link peers. Fall back
          to older "pick first link" method if no backlink index exists.

add $Id:$ line, commented out

require automake 1.6, in case that helps quagga.net
(paul: please try it)

[quagga-dev 1002]. Also fix date of previous commit in Chanelog.

* Fixed lowering privileges in proc ipforward method.
* Fixed "(no) ipv6 forwarding" command logic.
* Added --disable-capabilities switch to configure.

Vtysh fixes:
* replace -e with -c
* don't save command in history if last line is the same one
* doc/help fixes

[quagga-dev 1005] bugzilla #86: ospf_recv_packet will fail on Solaris/x86

Require automake 1.7 or higher.  Fixes problem reported by Gilad
Arnold where Gentoo Linux tries to be too smart and chooses automake
1.5, which doesn't set @PACKAGE@, leading to include files being in
$(prefix)/include rther then $(prefix)/include/quagga.
(Since our makefiles rely on @PACKAGE@, they should have required an
automake new enough to set it.)

Readded SIGTERM handling into sigevent stuff. Fixes bugzilla #85.

Fixing ospf6d as well and move Changelog entry from ospfd to general one.

*** empty log message ***

Fix segfault in ospfd - [quagga-dev 980].

Fix warning.

Amir's fix case where opaque capability is switched "ON -> OFF -> ON".
[quagga-dev 843]

Temorary secondary-label fix. Real fix from Gilad will follow after release.

From Sowmini fix for crash reported in [quagga-dev 929].

Fix bugzilla #82.

2004-03-16 David Young <dyoung@pobox.com>

* (many) reference <lib/version.h> rather than "version.h",
because version.h is a generated file and not present in the
source tree when using objdir builds.

(committed by gdt)
works fine with normal builds; didn't try objdir

Fix bugreport URLs here as well.

Fix help. "ipv6 ..." commands are for IPv6.

Document VTYSH_PAGER and fix bugreporting address.

Fix warning. Approved by Amir.

Many warning fixes from PC Drew ([quagga-dev 940]) and removing using PAGER
from vtysh ([quagga-dev 932]).

[quagga-dev 489] - fix ripd version send by Krzysztof Oledzki.

"show ip rip" fix from Krzysztof Oledzki ([quagga-dev 437]).

Fix typo in comment.

Zebra -> Quagga and reformatting.

Report quagga bugs to quagga bugzilla.

Fix vty address handling and added pid file stuff.

Report quagga bugs to quagga bugzilla.

2004-02-19 Sowmini Varadhan <sowmini.varadhan@sun.com>

        * ospf_abr.c: (ospf_abr_update_aggregate) UNH 3.12b,c, address range
          should be configured with the highest cost path within the range,
          not lowest.

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

* redhat/quagga.spec.in: remove references to patch files which
  were accidently committed.

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

        * ospf_zebra.c: (ospf_interface_delete) Do not delete the interface
          params, nor the interface structure, if an interface delete
          message is received from zebra.
        * ospf_interface.c: (ospf_if_delete_hook) Delete the interface
          params and interface, ie that which was previously removed in
          (ospf_interface_delete) above.

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

        * bgpd.h: (bgp_peer) add fd_local and fd_accept
          file descriptor's, fd becomes a pointer to one of these.
        * bgpd.c: (global) adjust for fact that fd is now a pointer.
          (peer_create_accept) removed.
        * bgp_route.c: (global) adjust for change of peer fd to pointer
        * bgp_packet.c: (bgp_collision_detect) adjust and remove the
          "replace with other peer" hack.
        * bgp_network.c: (bgp_accept) Remove the dummy peer hack.
          Update peer->fd_accept instead.
          (global) Adjust fd references - now a pointer.
        * bgp_fsm.c: (global) adjust peer fd to pointer.
          (bgp_connection_stop) new function, to stop connection.
          (global) adjust everything which closed peer fd to use
          bgp_connection_stop().

add some /* XXX assert() */ for questionable assumptions (no code
change - just notes to others).

add an XXX at a point where ifindex_num is assumed to be one w/o
checking.

Trivial fix from Bernd Leibing <bernd.leibing@kiz.uni-ulm.de>
[quagga-dev 879].

Added "ipv6 forwarding" command and fixed "no ipv6 forwarding" help.

Don't crash when attempting to read path->oi->ifp if oi doesn't exist any
more. I'm not sure if this read should be attempted at all, but I'm trying
to keep code safe AND stable.

Fix by Vadim Suraev - [zebra 14710] #6.

Threads usage cleanup. Still at least one bug to go.

Yet another "set thread pointer to NULL during thread run" fix.

Fixing previous commit.

Make "(no) ip address x.x.x.x/x" commands work again in vtysh.

2004-01-23 sowmini.varadhan@sun.com

        * rip_interface.c: obsolete unbind code in
        rip_interface_multicast_set, and instead do the more portable
        (though slower) method of creating a socket for each outgoing packet
        and binding the source address on the new socket.
        * rip_interface.c, ripd.c, ripd.h: Modify rip_request_send so that
        source address is determined by the caller of rip_request_send for
        ripv1 packets and non-multicast interfaces (rip_request_send loops
        over all connected address in all other cases).
        * rip_send_packet: don't send packets with source set to
        ZEBRA_IFA_SECONDARY connected addresses; improved debug messages;

add appropriate cvsignore

2004-01-19 Paul Jakma <paul@dishone.st>

        * tests/test-sig.c: New file, regression test for sigevents.
        * lib/Makefile.am: add sigevent.{c,h}
        * (isis|rip|ripng|ospf|ospf6|bgp)d/\1_main.c: modify for sigevents.
        * zebra/main.c: ditto.

2004-01-19 Paul Jakma <paul@dishone.st>

        * tests/test-sig.c: New file, regression test for sigevents.

2004-01-19 Paul Jakma <paul@dishone.st>

        * sigevent.{c,h}: New files, implement event handled signals.
          see signal_init() in sigevent.h.

2004-01-15 Paul Jakma <paul@dishone.st>

        * vtysh_config.c: Fix up strlen to use correct string.

Explain the dual use of struct prefix for CIDR prefixes and
address/netmask pairs, and clarify the two comparison functions.

2004-01-10 Paul Jakma <paul@dishone.st>

        * Makefile.am: redhat/ is a dist subdir too.

Exclude Makefile and Makefile.in

Fix build on FreeBSD 4.7 and FreeBSD 5.1:
[...]
source='isis_main.c' object='isis_main.o' libtool=no  depfile='.deps/isis_main.Po' tmpdepfile='.deps/isis_main.TPo'  depmode=gcc /usr/local/bin/bash ../depcomp  gcc -DHAVE_CONFIG_H -DSYSCONFDIR=\"/home/jardin/bin/etc/\" -I. -I. -I.. -I.. -I.. -I../lib     -g -O2 -Wall -c `test -f isis_main.c || echo './'`isis_main.c
gcc  -g -O2 -Wall   -o isisd  isis_main.o isis_adjacency.o  isis_lsp.o dict.o isis_circuit.o  isis_pdu.o isis_tlv.o isisd.o  isis_misc.o isis_network.o isis_zebra.o  isis_dr.o isis_flags.o isis_dynhn.o  iso_checksum.o isis_csm.o isis_events.o  isis_spf.o isis_route.o isis_routemap.o ../lib/libzebra.a -lm -lcrypt
Making all in redhat
make: don't know how to make all. Stop
*** Error code 1

Stop in /usr/home/jardin/no_save/quagga.
*** Error code 1

Stop in /usr/home/jardin/no_save/quagga.
quagga$

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

* HACKING: slight changes to make consistent with devel.php on web
  site. remove 'not yet consensus' as lack of any disagreement can
  be interpreted that file does reflect such.

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

        * Makefile.am: as per gdt, specify the redhat dir as a DIST_SUBDIR,
          remove the redhat/... dist targets - instead these now go in..
          redhat/Makefile.am: (new) proper place to describe redhat/ dist
          files, as well as allow quagga.spec to be regenerated properly.
          redhat/quagga.sysconfig: specify conf file location.
          redhat/quagga.spec.in: Add 2 patches to RPM build.

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

        * kernel_socket.c (kernel_read): Use sockaddr_storage in buffer
        for reading kernel messages to ensure enough space (necessary on
        Solaris due to sockaddr_dl being large).  Thanks to Sowmini
        Varadhan for help with this change.

change perror to zlog_err

2004-01-05  Greg Troxel  <gdt@fnord.ir.bbn.com>
        * kernel_socket.c (ifm_read): Major cleanup.  Use Sowmini's code
        to find the sockaddr_dl in all cases, narrowing the Solaris ifdef
        to just the accomodation of broken kernels.  Check sockaddr_dl
        carefully up front, and later assume any non-NULL sdl pointer is
        valid.  Clean up types and variable declarations, and rename
        WRAPUP to SAROUNDUP to make the name fit the behavior.

increase buffer size for kernel reads

move changelog section up higher.

Add notion of commenting for reviewers and taking care to not break
platforms one cannot test on.

Fix incorrect test in previous commit; thanks to Sowmini Varadhan for
catching this.

2004-01-05  Greg Troxel  <gdt@ahi.ir.bbn.com>

        * kernel_socket.c (kernel_read): Look up interfaces by index
        first, so that state changes which do not include a sockaddr_dl
        now work.  Add many sanity checks.  In
        particular, do not assume that a sockaddr_dl follows a message
        without checking the ifm_addrs flags, and do not trust the length
        in a sockaddr_dl.  Add/clarify many comments.

Really ripngd should either leave groups on down interfaces, or keep
track of on which interfaces joins have occurred.  This is complicated
by the possible lack of a clear behavior spec on whether interfaces
keep their joined group lists across a down/up/down transition.  (On
NetBSD, they do.)

2004-01-03  Greg Troxel  <gdt@ahi.ir.bbn.com>

        * ripng_interface.c (ripng_multicast_join): If IPV6_JOIN_GROUP
        returns EADDRINUSE, consider the join successful.  This happens
        when an interface goes down and comes back because
        ripng_multicast_leave does not invoke the LEAVE_GROUP operation if
        the interface is down.  Solves problem of ripng stopping working
        on an interface which goes down and then comes back up (on NetBSD).

Fix some threads related problems [quagga-dev 591].

2003-12-30 Paul Jakma <paul@dishone.st>

        * redhat/isisd.init: new file, init script for isisd.
          redhat/quagga.sysconfig: new file, sysconfig file for quagga
          initscripts.
          redhat/quagga.spec.in: various cleanups, including sysconfig patch
          from RH, fixed UID/GID as per RH EL, shell changed to
          /sbin/nologin, daemon vty's listen to 127.1 only per default and
          isisd packaged.
          redhat/*.init: sysconfig support and runlevels specified.

2003-12-30 Paul Jakma <paul@dishone.st>

        * Makefile.am: put the redhat/ stuff into EXTRA_DIST rather than
          copying via dist-hook. Remove ~ files backup cruft from dists.
          isisd/Makefile.am: sysconf example should go via
          dist_examples_DATA. The include-netbsd/ headers werent mentioned
          as sources and werent being copied into dists.

2003-12-23 Krzysztof Oledzki <oleq@ans.pl>

        * bgp_network.c: drop privs on error cases

(from [quagga-dev 438])

Remove generated file.