2005-01-28 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* lib/command.h: Document behavior of argv_concat function.
* lib/command.c: (argv_concat) Calculate total string length first so
  we can call malloc just once (instead of realloc'ing to add each
  string element).
  (do_echo,config_logmsg) Allow for possible NULL return value from
  argv_concat.

2005-01-25 Paul Jakma <paul@dishone.st>

* ospf6_asbr.c: Add "hsls" to the zroute_name const char array.

2005-01-24 Paul Jakma <paul@dishone.st>

* configure.ac: Bump version to 0.99.0
* doc/quagga.info: Version bump (autogenerated)

* bgp_route.c: Fix showstopper bug. New route must be selected also
  if old one is flaged as BGP_INFO_ATTR_CHANGED.

2005-01-24 Martin Pot <mpot at martybugs.net>

* zebra/rt_netlink.c: ignore wireless newlink netlink messages.

    * lib/command.[ch]: Make node_parent() function nonstatic. vtyh.c will
          use it as well.
        * vtysh/vtysh.c: Implement walkup in node tree for vtysh as it already
          works in vty.

* vtysh.conf.sample: Fix typo "integrated-vtysh-conf" ->
  "integrated-vtysh-config".

2005-01-18 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* log.h: Test for SA_SIGINFO to see whether zlog_signal takes final
  two args (siginfo and program_counter).
* log.c: (hex_append) Include this function only if SA_SIGINFO or
  HAVE_GLIBC_BACKTRACE is defined.
  (zlog_signal) Final two args (siginfo and program_counter) now
  depend on whether SA_SIGINFO is defined on this platform.
* sigevent.c: (program_counter) Do not include this function if
  SA_SIGINFO is not defined on this platform.
  (exit_handler,core_handler) Test for SA_SIGINFO to decide whether
  2nd & 3rd arguments are present and to decide how to invoke
  zlog_signal.
  (trap_default_signals) Test for SA_SIGINFO and invoke sigaction
  appropriately.

* *.c: Changed many functions to static. Some commented out
  functions and some tiny related fixes. No functional changes.

* interface.c: Better statistics output in "show interface" command in
  case of /proc being used. I don't have others to test with at the
  moment.

2005-01-17 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* log.h: Change prototype for zlog_backtrace_sigsafe to take additional
  program_counter argument.
* log.c: (zlog_backtrace_sigsafe) Add additional program_counter
  argument.  If it is non-NULL, use backtrace_symbols_fd to resolve
  the address.
  (zlog_signal) Call zlog_backtrace_sigsafe with additional
  program_counter argument.

[pullup candidate]

* bgp_route.c: Clear peer's routing table regardless whether it's
  configured or not. Being not configured is even better reason to
  do it.

* main.c: With --nl-bufsize argument is required.

[pullup candidate]

* command.[ch], vty.c: cmd_execute_command() function must not attempt
  to walk up in the node tree if called from vtysh. Different daemons
  might have commands with same syntax in different nodes (for example
  "router-id x.x.x.x" commands in zebra/ospfd/ospf6d daemons).

* vtysh.c: Reflect changes in lib. cmd_execute_command() should know
  now that it's called from vtysh and must not attempt to walk up in
  the node tree.

[pullup candidate]

2005-01-15 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* redhat/quagga.spec.in: Fix postun script to avoid misleading error
  message saying the postun scriptlet failed when watchquagga
  is not running.

          (pullup candidate)

2005-01-14 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* command.c (print_version): Do not bother even to examine host.name,
  since it is always NULL when this function is called from main.

2005-01-14  Greg Troxel  <gdt@fnord.ir.bbn.com>

        * command.c (print_version): Don't print host.name if it is NULL.
        Fixes segfault on Solaris reported by Goetz von Escher <goetz@open.ch>

(pullup candidate)

Ignore <dist tarball>.asc - i always seem to have one hanging around from
last release..

2005-01-12 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* configure.ac: Test for header file <ucontext.h> (for use in
  signal processing).
* sigevent.c: (trap_default_signals) Use the SA_SIGINFO flag to
  pass additional siginfo_t and ucontext_t arguments to core_handler
  and exit_handler.
  (core_handler,exit_handler) Now invoked with 3 arguments (using
  SA_SIGINFO).  Pass additional info to zlog_signal.
  (program_counter) New function to find program counter in ucontext_t,
  needs to be enhanced to support more platforms (currently works only
  on Linux/x86).
* log.h: Change the zlog_signal prototype to add new arguments
  siginfo_t * and program_counter.
* log.c: (zlog_signal) Add new arguments siginfo and program_counter.
  Include si_addr and program counter (if non-NULL) in message.
  And remove #ifdef HAVE_GLIBC_BACKTRACE around hex_append, since
  that is now used to render the si_addr and PC pointers.

2005-01-12 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* configure.ac: If configure is invoked with --enable-snmp, but
  the configure script is unable to find SNMP support on the platform,
  then configure should give an error message and exit.

2005-01-12 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* redhat/quagga.spec.in: Pass --enable-gcc-rdynamic to configure
  to get gcc to link with -rdynamic for better backtraces.
  When the rpm is upgraded, the restart logic now works as follows:
  1. stop watchquagga; 2. stop all routing daemons; 3. restart zebra
  if it was running; 4. start all routing daemons that were running;
  and 5. start watchquagga if it was running.

2005-01-12 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* watchquagga.c: (run_job) Fix bug: delay.tv_sec was being used
  without having been initialized when the "force" argument was true.
  (try_restart) When phased restart is stopping the routing daemons,
  it should tell run_job to update the restart interval.
  (phase_check) In final step of phased restart when the routing
  daemons are being started again, tell run_job not to update
  the restart interval (this is now done when the daemons are
  stopped).

2005-01-12 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* zebra.h: If not C99 and no va_copy macro available, fall back to
  memcpy (solves a build problem on FreeBSD 4.x).

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

* texinfo.tex: Updated to a version which understands @{ and @}
* vtysh.texi: Add node name for @section
* quagga.info: updated build of autogenerated file

2005-01-10  Greg Troxel  <gdt@fnord.ir.bbn.com>

        * ospf_packet.h: Remove commented out definition of
        OSPF_MAX_PACKET; neither it or the uncommented one are used any more.

        * ospf_packet.c (ospf_make_ls_upd): Leave room for authentication
        when deciding if an update will fit.
        (ospf_packet_authspace): Factor out calculation of size required
        for authentication.
        (ospf_make_db_desc): Use ospf_max_packet, not OSPF_MAX_PACKET.
        Don't confuse readers that there is a macro.

2005-01-07 Paul Jakma <paul@dishone.st>

* configure.ac: Bump version to 0.98.0
* doc/quagga.info: Version bump (autogenerated)

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

* configure.ac: Bump version to 0.97.5
* doc/quagga.info: autogenerated, version bump.

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

* zserv.c: (zebra_accept) Comment out setting of socket to NONBLOCK
          for now, as we dont actually deal with with resending.... See
          bugzilla #122, fix from wawa@yandex-team.ru (Vladimir Ivanov).
* kernel_socket.c: (routing_socket) ditto.

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

* bgp_packet.c: (bgp_write) set socket to nonblock while writing
          this should be generalised. See bugzilla #102. Fix supplied by
  wawa@yandex-team.ru (Vladimir Ivanov).

minor fix to HAVE_BROKEN_CMSG_FIRSTHDR support: use AC_MSG_CHECKING
etc. so configure output shows the choice we made.

(Andrew: please recheck on a buggy system; it configures/compiles fine
on NetBSD.)

2005-01-04 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* configure.ac: Added test for broken CMSG_FIRSTHDR macro
  (relevant for Solaris 8 and unpatched Solaris 9, don't know
  whether other platforms are affected).
* zebra.h: Define ZCMSG_FIRSTHDR appropriately based on whether
  config.h indicates HAVE_BROKEN_CMSG_FIRSTHDR (as determined
  by the configure test program).
* sockopt.c: (getsockopt_cmsg_data) Use ZCMSG_FIRSTHDR instead
  of CMSG_FIRSTHDR.
* rtadv.c: (rtadv_recv_packet,rtadv_send_packet) Use ZCMSG_FIRSTHDR
  instead of CMSG_FIRSTHDR.
* ripd.c: (rip_recvmsg) Use ZCMSG_FIRSTHDR instead of CMSG_FIRSTHDR.
* ripngd.c: (ripng_recv_packet) Use ZCMSG_FIRSTHDR instead of
  CMSG_FIRSTHDR.

2005-01-04 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* NEWS: Note improved logging facilities.

Reverting some int -> unsigned int fixes in command.c for now. Fixes crash
described in [quagga-dev 2292].

More cleanup in isisd.

Move TODO items from isid to the common TODO.

There is no need to keep this directory.

Make authentication of SNPs work correctly - ie. conditionally like it is in
IOS.

2004-12-30 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* ospf_network.c: Improve all setsockopt error messages to give detailed
  information on the arguments.

Add comment explaining about limit of 20 joined groups on a socket.

2004-12-29 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* ospf_packet.c: (ospf_db_desc) Reduce severity of "Negotiation done"
  messages from LOG_WARNING to LOG_INFO, since this seems to be
  normal.

2004-12-29 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* ospf_packet.c: (ospf_read) Always look up the interface if
  ospf_recv_packet returns NULL ifp, since some platforms such
  as Solaris 8 appear to support ifindex retrieval but don't.

add XXX comment that perhaps we need to recover from short headers.

Really fix crashes now. Revert isis_circuit_update_params() part of previous
commit. Replace all if_is_up() calls with if_is_operative().

2004-12-29  Greg Troxel  <gdt@poblano.ir.bbn.com>

* sockopt.c (getsockopt_ipv4_ifindex): Document calling
convention.  Beef up comments.  Handle the case where the cmsghdr
has a zero controllen, or more specifically when the wanted option
is not present.  This is needed for Solaris 8, and in general for
any platform for which configure finds a method and it can fail.
Mark some changes with XXX to be cleaned up post 0.98.

Don't crash during interface up/down events.

2004-12-29  Greg Troxel  <gdt@poblano.ir.bbn.com>

* sockopt.c (getsockopt_ipv4_ifindex): Return 0 when passed a NULL
  cmsghdr pointer.

I believe this will avoid ospfd crashing on Solaris 8, which seems to
define IP_RECVIF but not actually implement it.

2004-12-29 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* configure.ac: Add new option --enable-gcc-rdynamic to link
  with -rdynamic.

2004-12-29 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* watchquagga.c: In several places, cast pid_t to int for printf to
  avoid complaints on Solaris 8.

2004-12-29 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* watchquagga.c: Fix headers: get most stuff from zebra.h.

2004-12-28 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* sockopt.c: (setsockopt_ipv4_ifindex) Improve error message.
  When neither IP_PKTINFO nor IP_RECVIF is defined, make return value
  deterministic (-1).

2004-12-28 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* thread.c: (funcname_thread_add_timer_msec) Reduce overflow risk.

*.c: Change level of debug messages to LOG_DEBUG.

zlog_* cleanup. Level of debug messages to LOG_DEBUG.

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

configure.ac: Bump version to 0.97.4
doc/quagga.info: ditto (autogenerated file)

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

* watchquagga.c: Add missing getopt.h include

2004-12-22 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* redhat/quagga.spec.in: daemonv6_list should contain only IPv6 daemons.

2004-12-22 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* redhat/quagga.spec.in: Add watchquagga, and fix some other
  logic to make sure that all daemons are restarted on upgrades
  and stopped on package removal.

2004-12-22 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* watchquagga.c: Try for gcc 2.95 compatibility (avoid %z and a tricky
  initialization).

2004-12-22 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* connected.c: (connected_add_ipv4) Limit warning about /32 addresses
  with no peer specified to PtP interfaces only.

2004-12-22 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* redhat/quagga.sysconfig: Define some variables to support watchquagga.

2004-12-22 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* watchquagga.c: Add -b option to circumvent difficulties passing
  embedded spaces in command-line arguments.  This is an ugly hack,
  we should probably use a config file instead.

Show sums of checksums in "show ip ospf" output. Okayed by Paul and James
R. Leu (author of original idea).

2004-12-22 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* watchquagga.c: Use new config.h define for DAEMON_VTY_DIR.

2004-12-22 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* configure.ac: Add a define for DAEMON_VTY_DIR in config.h.

2004-12-22 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* watchquagga.c: Get PATH_WATCHQUAGGA_PID from config.h.

2004-12-22 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* {configure.ac,Makefile.am}: Build watchquagga by default.

Fix "show ip ospf" output. If router is configured as "translate-never", we
say so - we can never be translator.

More cosmetical vtysh fixes.

Show debug configuration in vtysh.

No delimiter between "ip forwarding" and "ipv6 forwarding".

2004-12-21 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* .cvsignore: Added.

2004-12-21 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* NEWS: Note addition of watchquagga.
* HACKING: Note that watchquagga is in testing phase.

2004-12-21 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* watchquagga: New watchquagga daemon.

2004-12-21 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* redhat/Makefile.am: Added watchquagga.init to EXTRA_DIST.

2004-12-21 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* redhat/watchquagga.init: New file, init script for watchquagga.

2004-12-21 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* {ChangeLog,watchquagga.c,Makefile.am}: Created.

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

* stream.c: Dont allocate streams with 0 sized data buffers

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

* if.h: Add more 'non-generic' IFF_ flags.
* if.c: IFF_NOXMIT/IFF_VIRTUAL interfaces are 'loopback like'

Fix "show ipv6 route <proto>" command.

Add isisd.8 into list of man pages.

2004-12-17 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* command.c: (do_echo) Added new "echo" command, useful for
  watchdog pinging to make sure the daemon is responsive.

2004-12-17 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* pid_output.c: (pid_output_lock) Eliminate static function, and just
  use the #ifdef to decide which version of the function to include.
  This eliminates a compilation problem with gcc4.  And fix the
  non-fcntl version so that it actually compiles.  Exit with
  status 1 instead of -1 on error.

2004-12-16 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* ripng_nexthop.c: Replace NEXTHOP_OUT macro with NEXTHOP_OUT_PTR,
  since it's not really OK to take the address of a ternary expression.
  (_ripng_rte_cmp,ripng_rte_send) Use new NEXTHOP_OUT_PTR macro
  instead of NEXTHOP_OUT.

2004-12-15 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* sigevent.c: (trap_default_signals) Ignore SIGPIPE instead of exiting.

2004-12-15 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* ospf_interface.h: Declare new function ospf_default_iftype.
* ospf_interface.c: (ospf_default_iftype) New function to centralize
  this logic in one place.
* ospf_zebra.c: (ospf_interface_add) Use new function
  ospf_default_iftype.
* ospf_vty.c: (no_ip_ospf_network,config_write_interface) Fix logic
  by using new function ospf_default_iftype.

2004-12-15 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* ripd.c: (rip_read) Improve 2 error messages to show the source of
  the packet when the lookup fails.

2004-12-11 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* ospf_packet.c: (ospf_db_desc) Should be static, not global.
  (ospf_hello,ospf_db_desc,ospf_ls_upd,ospf_ls_ack) Improve warning
  messages to include identifying information (e.g. router id).
* ospf_nsm.c: (nsm_change_state) Improve info message to include
  router id and state names.

2004-12-10 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* log.c: (zlog_signal,_zlog_assert_failed) Change logging level back to
  LOG_ERR instead of LOG_EMERG.

2004-12-09 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* log.c: (hex_append) No need to include this function if
  HAVE_GLIBC_BACKTRACE is not defined.

   * ospf_apiserver.c (ospf_apiserver_term): Obtain struct
        ospf_apiserver * from listnode.  Remove unused variables.  Follows
        suggestion from Jay Fenlason.

2004-12-09 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* vtysh.c: (vtysh_connect) No reason to call geteuid and getegid,
  the results were never used.

add XXX comment about hex_append being unused.
(only used if HAVE_GLIBC_BACKTRACE, so perhaps ifdef it?)

ospf_apiserver_term:
  Remove unused variable.

  Add comment: this is seriously broken, and needs fixing.  It appears
  likely to segfault if ever called.  Comments please - should I make
  the fix I suggest in the comment?  -gdt

add comments about warnings

Add const to char * to quiet warnings.

Note two warnings in comments.  The serious one is that struct
interface->ifindex probably needs to be signed, ssince we store and
compare to -1.

add comment about warning of comparison between signed and unsigned

Add comments with warnings obtained on NetBSD/i386.  Both are due to
size_t being used for streams.  Perhaps cast to unsigned long before
printf?  Surely that's big enough even if size_t could be larger in
theory.

(no changelog, since only comment changes)

2004-12-08 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* *.h: Change level of debug messages to LOG_DEBUG.

2004-12-08 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* *.c: Change level of debug messages to LOG_DEBUG.

2004-12-08 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* *.c: Change level of debug messages to LOG_DEBUG.