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.

Some fixes to isisd done by me and Cougar in the spring of 2003. See
changelog for details.

Let's use 2608/tcp for isisd.

Add .cvsignore

Changelog entry about isisd integration. Sorry about previous commit
without message. Fingers worked faster than brain ;(.

2607/tcp is already used by ospfapi.
Let's use 2608/tcp for isisd.

*** empty log message ***

Add comments about isisd integration.

Merge isisd into the Quagga's framework:
  - add privs support
  - use misc quagga's definitions
  - make it compile"able"
  - fix segfault cases related to hostname()
  - add debug isis xxx command

This patch has been approved by Paul Jakma.

Initial revision

Reorder free(f); unlink(f); to unlink before freeing.

2003-12-22 Christian Hammers <ch@lathspell.de>

        * configure.ac (and everywhere a regular file is opened for
          writing): use file permissions from configure rather than
          compiled-in umask.

Revert my "microfix". It was caused by misreading code. Gilad is right as
always.

note that order of = items is undefined, probably

comments about struct list

add comments clarifying the operation of listnode_add_sort

add question about changelog usage

lib/if.c part from [quagga-dev 552] and lib/inklist.c part from [quagga-dev
555]. Okayed by Paul.

rough cut at committing guidelines

{lib,ospfd,ospfapi}/Makefile.am:

Use pkginclude_HEADERS rather than include_HEADERS to place includes
in ${prefix}/include/quaggainstead of polluting ${prefix}/include.

2003-12-08 Mattias Amnefelt <mattiasa@kth.se?

        * ospf_packet.c: (ospf_recv_packet) OpenBSD now leaves iph.ip_len
          network byte order. (bugzilla #67).

2003-12-06 Yasuhiro Ohara <yasu@sfc.wide.ad.jp>

* ospfd/ospf_flood.c: (ospf_flood_through_interface) fix compile
  warning.

* ospfd.c (ospf_network_match_iface): Rewrite code for clarity
while trying not to change semantics.  Add ifdefed-out code to
avoid matching ppp interfaces whose destination address does not
also match the prefix under consideration, to help out people with
problems due to as-yet-unfixed bugs with p2p interfaces coming and
going.

fix exampledir processing bug; it failed when neither exampledir nor
sysconfdir were specified, due to incorrect extra quoting of
sysconfdir.  Thanks to Vincent Jardin for the bug report.

Enable rtadv by default.

Resolves bug #66.

reorder includes to avoid warning.
fixes rest of bug #62

check for v6 header files after we figure out whether we are doing v6
and if so which flavor.

Fixes 99% of bugzilla bug #62.

Add support for --enable-exampledir to specify where to place example
files, defaulting to sysconfdir (matching previous behavior).

This is needed to support (cleanly) NetBSD pkgsrc, which requires that
example config files go in $(prefix)/share/examples/pkgname, rather
than in $(prefix)/etc/pkgname.

Added info about -A and -u command line switches to man pages.

2003-11-17 Jay Fenlason <fenlason@redhat.com>

* zebra/rt_netlink.c: netlink_parse_info() ignore messages which are
  not from kernel. Reported to RH by Herbert Xu. See
  http://rhn.redhat.com/errata/RHSA-2003-307.html and CAN-2003-0858.

2003-11-17 sowmini.varadhan@sun.com

        * ripd/ripd.c: rip_output_process() fixup aspects of split-horizon
          route suppression. See [quagga-dev 429].

2003-11-17 Hasso Tepper <hasso@estpak.ee>

        * ripd/ripd.c: rip_create_socket() privs were not dropped if bind
          failed.

2003-11-17 Hasso Tepper <hasso@estpak.ee>

        * ripngd/ripngd.c: ripng_make_socket() add calls to raise/lower
          privs.

2003-11-10 Claus Endres <claus@endresconsulting.com>

        * ospfd/ospf_interface.c: (ospf_if_table_lookup) grab reference to
          rn->info /before/ calling route_unlock_node() as some systems
          may deliberately poison freed memory, eg FreeBSD 5.1.
          see [quagga-dev 417].

2003-11-10 Lorenzo Colitti <lorenzo@ripe.net>

        * bgpd/bgp_dump.c: (bgp_dump_routes_entry) call
          dump_bgp_routes_attr() with a prefix.

2003-11-03 Paul Jakma <paul@dishone.st>

* bgpd/bgp_route.c: Delete some extraneous whitespace. Credit to
  a person on #quagga whose name I unfortunately can not recall.

2003-11-02 Paul Jakma <paul@dishone.st>

* redhat/quagga.spec.in: dont specify --with-cflags. rpmbuild sets
  CFLAGS.

2003-11-02 Paul Jakma <paul@dishone.st>

* NEWS: Update for 0.96.4 release

2003-11-02 Paul Jakma <paul@dishone.st>

* ChangeLog: Update with logs from last 2 releases.

2003-11-02 Paul Jakma <paul@dishone.st>

        * redhat/quagga.spec.in: Install libzebra headers with -devel
          package.

2003-11-02 Paul Jakma <paul@dishone.st>

        * configure.ac: FreeBSD has net-snmp in /usr/local.

2003-11-02 Krzysztof Oledzki <oleq@ans.pl>

        * zebra/zebra_rib.c: Revert patch (dating from zebra.org) which
          caused zebra to read all routes in all tables, rather than just
          the main table. See [quagga-dev 280].

2003-11-02 Paul Jakma <paul@dishone.st>

        * bgpd/bgp_routemap.c: Fix up 'set ip next-hop A.B.C.D|peer-address'
          route map command so that vtysh can use it. Modified version of
          Hasso Tepper's patch. Fixes bug #52.

2003-10-31 Paul Jakma <paul@dishone.st>

* configure.ac: newline not escaped in CHECK_HEADERS

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

* configure.ac: netinet/in_systm.h is yet another well-known
  header file we really should be checking for.

2003-10-30 sowmini.varadhan@sun.com

        * lib/pid_output.c: (pid_output_lock) undo paulj's mindless edit.

2003-10-29 Paul Jakma <paul@dishone.st>

        * lib/routemap.c: (route_map_apply_index) renamed to
          route_map_apply_match.
          (route_map_apply_match) Take rule list as argument rather than
          index. concentrate solely on match logic.
          (route_map_apply) Properly implement the route map statement
          logic, according to the table, especially the on-match exit
          policy.

2003-10-27 Gilad Arnold <gilad.arnold@terayon.com>

        * zebra/zebra_rib.c: (nexthop_active_update) Check for multipath
          limit when setting changed flag to avoid spurious changes.
          (static_install_ipv{4,6}) dont uninstall by default, might not be
          required - avoid spurious uninstalls.
          (static_uninstall_ipv{4,6}) only uninstall the route if its
  actually FIB route.

2003-10-27 Paul Jakma <paul@dishone.st>

* lib/if.{ch}: remove ifc_pointtopoint() - left over from the
  reverted RFC3021 patch.