spelling: s/supress/suppress/

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

* vty.h: Remove fields in struct vty that were related to VTY_CONTINUE
  capabilities (that were used only in bgpd/bgp_route.c and are now
  removed).  Also remove some other fields that were not being
  used at all.
* vty.c: (vty_execute) Do not test for obsolete status values VTY_START
  and VTY_CONTINUE.
  (vty_read) Remove calls to vty->output_func since that was part
  of the VTY_CONTINUE infrastructure that has been removed.
  (vty_flush) Remove code to support VTY_START and VTY_CONTINUE.
  (vty_close) Remove code to cancel vty->t_output thread, since that
  thread was never actually used.
* bgp_route.c: Remove all code related to VTY_CONTINUE; this feature
  is deprecated because the output did not represent a single point
  in time.  All output needs to be generated inline and buffered
  by the library code.
  (route_vty_out,route_vty_out_tag,damp_route_vty_out,
   flap_route_vty_out) Remove code to count number of lines of output,
   since this was only useful for VTY_CONTINUE behavior.
  (bgp_show_callback) Removed.
  (bgp_show_table) Remove hooks for VTY_CONTINUE callback support.
  As a result, there's a new output_arg argument to this function.
  Make function static.
  (bgp_show) Make function static and add a new output_arg argument.
  Change all functions that call bgp_show or bgp_show_table to
  pass the new output_arg argument (that used to be passed inside
  vty->output_arg).
* bgp_mplsvpn.c: Remove declarations of functions defined in
  bgp_route.c; these declarations belong in bgp_route.h.
* bgp_route.h: Declare 3 global functions used in both bgp_route.c
  and in bgp_mplsvpn.c.

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

* ospfd.c: (ospf_network_match_iface) revert to previous network
          statement match behaviour.

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

* vty.c: Vtysh connections to daemons should use buffering.
  (vty_out) Remove exception for vty_shell_serv, just use buffer_write.
  (vty_new) Increase output buffer size to 4096 rounded up to a
  multiple of pagesize.
  (vtysh_read) After command has been executed and all output buffered,
  call buffer_flush_available and schedule further writes if the
  buffers are not yet empty.
  (vtysh_write) New function to flush output to vtysh when the socket
  is writeable.
  (vty_event) Added new VTYSH_WRITE event for flushing buffers to vtysh
  clients.  Also, should save read thread in vty->t_read so the
  thread can be cancelled in vty_close.
* buffer.h: In struct buffer_data, remove unused "parent" field.
  Convert "unsigned char *data" to "unsigned char data[0]" to save
  a malloc.  Declare new function buffer_flush_available that works
  with non-blocking sockets.
* buffer.c: (buffer_data_new) Use a single malloc now that data is
  a variable-size array at end of structure.
  (buffer_data_free) Just a single free now that data is part of the
  structure.
  (buffer_write) Simplify the logic to make behavior more transparent.
  (buffer_flush) Decrease b->length as data is written out.
  (buffer_flush_vty_all) Decrease b->length as buffers are freed.
  (buffer_flush_vty) Decrease b->length as data is written out.
  (buffer_flush_available) New function to flush non-blocking sockets.

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

* ospf_packet.c: (ospf_write_frags) remove iov arg, msg already points
  to it. Add convenience pointer to msg->msg_iov[1], and use this,
  fixing the unfortunate borkenness introduced in moving of this code
  to a function.
  (ospf_write) remove iovp and fix up call to previous.
  (ospf_ls_upd_packet_new) cast size to long int - unfortunately
          glibc's size_t format modifier is not portable.

- Add .arch-ids and .arch-inventory to cvsignore, to allow one to use arch
  with CVS trees.

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

* sockopt.c: (setsockopt_pktinfo) remove, its unused.

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

* {bgpd,bgp_attr}.c: size_t printf format should be ld.

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

* ospf_packet.c: (ospf_ls_upd_packet_new) Format for size_t should
          be ld.
* ospf_zebra.c: (ospf_distribute_list_update_timer) Ugly misuse of
  THREAD_ARG to store an integer, but it should at least use same
          same type to retrieve the value. Assert value is sane.

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

* thread.c: Use XCALLOC and sizeof the type, not the pointer.

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

* smux.c: fix int to size_t compile warnings

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

* memory.h: Add MTYPE_THREAD_FUNCNAME and MTYPE_THREAD_STATS
* thread.c: Update stats and funcname alloc/free to use previous
  specific memory type defines

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

* keychain.c: Convert some more strtoul users to VTY_GET_INTEGER.

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

* ospf_packet.c: (ospf_write_frags) Add debug output
  (ospf_write) set type early, so we can pass it to
  ospf_write_frags.
  (ospf_ls_upd_packet_new) print size in debug output when too large
  packet is encountered.

- update cvs ignore, ignore quagga.dvi, quagga.ps, etc.

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

* vty.c: As per Andrew's suggestions..
  (vty_serv_un) remove flags.
  (vtysh_accept) close socket if we cant set NONBLOCK. Add flags.

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

* vty.c: Move setting of sock to O_NONBLOCK from vty_serv_un
  to vtysh_accept, where sock is the actual fd we wanted to set to
          O_NONBLOCK, ie the /connected/ vtysh unix socket.

- lib/ dir shouldnt be conditional, remove bad dir attribute from dev files

"show running-daemons" -> "show daemons".

Remove dead "ip tunnel" command.

Fix help of "show debugging ripng" command.

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

* Update with fix in debian bug id 222930.
* bgp_main.c: Add ZCAP_RAW, needed to bind to interfaces.
  bgp_network.c: (....) raise/lower privs around call to
          SO_BINDTODEVICE sockopt.

Yet another breakage introduced in May with zserv cleanup.

- andrew's service adding change to detect existing entries reliably

- update with andrew's modifications
- enable irdp by default again

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

* configure.ac: bump version to 0.97.2, release imminent.

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

* zebra.h: reserve ZEBRA_ROUTE_HSLS

- credit rix for fixing openbsd

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

* ospfd/ospf_packet.c: setsockopt_pktinfo -> setsockopt_ifindex
* zebra/irdp_main.c: ditto

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

* sockopt.h: Fix elif that should be an else.

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

* command.h: Cleanup the defines a bit, add helper defines and
  collapse all defines to use those. Add an attribute field to
  cmd_element to support, eg hidden or deprecated commands, add
          defun defines for such. All that's left to do is add logic
          to command.c to check these attributes... ;)

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

* ospf_network.c: (ospf_sock_init) call neutral setsock_ifindex()
          function.
        * ospf_packet.c: (ospf_read) manually look up ifindex
          if system could not have returned one, eg openbsd.

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

        * sockopt.c: (setsockopt_pktinfo) unexported
* sockopt.h: Cleanup SOCKOPT_CMSG defines a bit. Add a throwaway
          define for SOPT_SIZE_CMSG_IFINDEX_IPV4 for systems which have
          neither IP_RECVIF nor IP_PKTINFO (eg openbsd).

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

* sockopt.c: (getsockopt_ipv4_ifindex) no ifindex should be 0, not
          -1.

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

* ripd.c: Collapse redundant passing of various address structs,
          struct interface and struct connected as arguments to functions
          down to two key arguments, namely struct connected and, possibly,
          address of source/destination. Testing for RIPv1 would be useful.
          (rip_read) lookup struct connected for the received packet, pass
          it on.
        * rip_interface.c: With previous changes, we no longer have to tread
          carefully with struct connected, as it will always be there and
          valid.

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

* configure.ac: fix up enable help alignment slightly
          Add --enable-gcc-ultra-verbose to set various gcc warnings which
          should one day be fixed but are not serious problems or which
          could be false-positives.

Don't popen pager if command is "exit". Fixes "vtysh screws up my terminal
if I exit" issue.

Small copyright string and hostname related cleanup.

OK. Here it is - PtP patch from Andrew J. Schorr. No problems with ospfd,
ripd might need some more testing though.

- From Andrew Schorr, fixup logrotate to use correct path to killall
  and use the pid file.

Fix regression. It seems to me that this code still works only by accident
though. Needs serious cleanup.

Missing comma.

Make dump configuration appear in vtysh.

Ladies and Gentlemens. This file is dead for years, from 1999 to be exact.

- fix date in changelog of my last commit..

* zserv.c: Remove useless warnings "forwarding is already on".

Make initializing smux connection configurable - "smux peer OID" command
initializes connection, and "no smux peer" command terminates it. Fixes
bugzilla #47 and #112.

Unbreak compilation with ospfapi disabled. We should seriously think about
reducing various configure options.

All access lists (IPv4 and IPv6) are used by all daemons to limit access to
vty.

Initialize access lists here as well.

Fix critical bugzilla #113. Make CMD_ERR_NOTHING_TODO nonfatal.

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

* (global) more const'ification.
* sockunion.c: (sockunion_su2str) buffer should be sized
          SU_ADDRSTRLEN.
          (sockunion_log) do not return stack variables, strdup buf before
          return.
        * vty.h: Fix up the VTY_GET_INTEGER macros. Testing caller supplied
          values against ULONG_MAX is daft, when caller probably has passed
          a type that can not hold ULONG_MAX. use a temporary long instead.
          Add VTY_GET_LONG, make VTY_GET_INTEGER_RANGE use it, make
  VTY_GET_INTEGER a define for VTY_GET_INTEGER_RANGE.

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

* (global) more const'ification and fixups of types to clean up code.
* bgp_mplsvpn.{c,h}: (str2tag) fix abuse. Still not perfect,
          should use something like the VTY_GET_INTEGER macro, but without
          the vty_out bits..
        * bgp_routemap.c: (set_aggregator_as) use VTY_GET_INTEGER_RANGE
          (no_set_aggregator_as) ditto.
        * bgpd.c: (peer_uptime) fix unlikely bug, where no buffer is
          returned, add comments about troublesome return value.

Some compiler warnings fixes and fix for bugzilla #119.

Unbreak "show ip route" command.

Unbreak ospfclient compilation.

ospf6d supports SNMP as well now.

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

* bump version to 0.97.1, release imminent.

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

* vtysh.c: (vtysh_client_execute) fix the sync fix. The 4th and
  final sync byte is not ASCII NULL, it's actually a status integer,
  and only ASCII NULL if CMD_SUCCESS is returned by daemon.
  Fix pointed out by Hasso.

Better common router-id handling.

Update NEWS - apiserver is disabled by default now.

Disable ospfapi init by default.

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

* ospfd.texi: reformat the ospf md5 paragraph, add an additional
  way to work around bad clocks.

Fix warnings here.

Sync isisd with changes in lib.

Sync with changes in lib. Make more strings const.

Some more const string fixes.

Commas -> dots.

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

* if.h: mtu's should be unsigned.
* routemap.{c,h}: const char updates
* smux.{c,h}: ditto

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

* (global) Const char update and signed/unsigned fixes.
* (various headers) size defines should be unsigned.
* ospf_interface.h: remove duplicated defines, include the
  authoritative header - though, these defines should probably
          be moved to a dedicated header, or ospfd.h.
* ospf_lsa.h: (struct lsa) ls_seqnum should be unsigned.
* ospf_packet.c: (ospf_write) cast result of shift to unsigned.

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

* ospf_dump.c: (ospf_ip_header_dump) Assume header is in host order
  remove ntohs that should have dissappeared. Take struct ip
  as argument, caller has to know there's an IP header at start of
  stream anyway.
* ospf_dump.h: update declaration of ospf_ip_header_dump.
* ospf_packet.c: (ospf_write) correct call to
  sockopt_iphdrincl_swab_htosys which was munging the header.
  (ospf_recv_packet) ip_len is needed for old OpenBSD fixup.
  (ospf_read) sockopt_iphdrincl_swab_systoh ip header as soon as
  we have it.

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

* thread.c: (funcname_thread_add_timer)
  (funcname_thread_add_timer_msec) Fix mistakes from last change.
  Pointed out by Liu Xin in [quagga-dev 1609].

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

* ospf6_route.c: Add const qualifier to various char arrays of
          constants. signed/unsigned fixes.
          (ospf6_linkstate_table_show) argv is const
        * ospf6_snmp.c: listnode typedef is dead.
          (ospf6_snmp_init) Take struct thread_master arg, needed for
          smux_init.
        * ospf6_snmp.h: update ospf6_snmp_init declaration.
        * ospf6d.c: (ospf6_init) add const qualifier to sargv, pass master
          to ospf_snmp6_init.
* ospf6_asbr.c: const char update.
* ospf6_interface.c: ditto, plus signed/unsigned fixes.
  (ipv6_ospf6_cost) Check whether cost fits in u_int32_t and use
          strtoul.
* ospf6_intra.c: const char update. Parenthesise expression.
* ospf6_lsa.c: signed/unsigned and const char updates.
* ospf6_proto.c: ditto.
* ospf6_message.c: ditto.
* ospf6_lsdb.c: signed/unsigned update.
* ospf6_main.c: const char update.
* ospf6_neighbor.c: ditto.
* ospf6_spf.c: ditto.
* ospf6_top.c: ditto.

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

* version.h.in: (pid_output*) add const qualifier.
* command.h: Change DEFUN func to take const char *[] rather
          than char **, to begin process of fixing compile warnings in lib/.
          Nearly all other changes in this commit follow from this change.
        * buffer.{c,h}: (buffer_write) pointer-arithmetic is gccism, take
          const void * and cast an automatic const char *p to it.
          (buffer_putstr) add const
        * command.c: (zencrypt) const qualifier
          (cmd_execute_command_real) ditto
          (cmd_execute_command_strict) ditto
          (config_log_file) ditto.
          Fix leak of getcwd() returned string.
        * memory.{c,h}: Add MTYPE_DISTRIBUTE_IFNAME for struct dist ifname.
        * distribute.{c,h}: Update with const qualifier.
          (distribute_free) use MTYPE_DISTRIBUTE_IFNAME
          (distribute_lookup) Cast to char *, note that it's ok.
          (distribute_hash_alloc) use MTYPE_DISTRIBUTE_IFNAME.
          (distribute_get)  Cast to char *, note that it's ok.
        * filter.c: Update with const qualifier.
        * if.{c,h}: ditto.
        * if_rmap.{c,h}: ditto.
          (if_rmap_lookup) Cast to char *, note that it's ok.
          (if_rmap_get) ditto.
        * log.{c,h}: Update with const qualifier.
        * plist.{c,h}: ditto.
        * routemap.{c,h}: ditto.
        * smux.{c,h}: ditto. Fix some signed/unsigned comparisons.
        * sockopt.c: (getsockopt_cmsg_data) add return for error case.
        * vty.c: Update with const qualifier.

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

* Makefile.am: fix listing of ospf_snmp.c in noinst_HEADERS

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

* version.h.in: (pid_output*) add const qualifier.

Merge svn revisions 1065 and 1066 from Zebra CVS.

Compiler warnings fixes round 1.

Argh, fix typo.

There is no warnings here any more.

Only warning left is the known lvalue problem in ripng_nexthop.c.

More const strings.

Although we don't look at current directory for config any more, it's nice
to have modified config file here for development.

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

* version.h.in: print_version declaration is here, not in autoconf
  generated version.h.

- ospfclient should be in -devel package, if anywhere.

Vtysh compiles cleanly as well.

Even more const strings and fix to bugfix.

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

* ChangeLog: commit this too...
* bump version to 0.97.0, release imminent.

Fix warnings. Didn't even look at files not compiled in Linux though.

No warnings here any more.

2004-10-07  Greg Troxel  <gdt@claude.ir.bbn.com>

        * ospf_apiserver.c (ospf_apiserver_unregister_opaque_type): Don't
        use of variable names 'node' and 'nextnode' to avoid possible
        conflict with list macros.  Move variable declaration inside for
        loop after a statement to top of function.

Make more strings const.

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

* bump version to 0.97.0, release imminent.

- make specfile actually work

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

* ospf_snmp.c: Missed list typedef update
* ospf_dump.c: Include sockopt.h for header swab functions.

2004-10-07  Greg Troxel  <gdt@sunpal7.mit.edu>

        * configure.ac: remove -Wpacked; 2.95.3 doesn't support it.

Number of warnings is down to 3 again in lib directory. A lot of const's
added to strings and a lot of int -> unsigned int changes.

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

* thread.c: (funcname_thread_add_timer_timeval) new function, add
          timer at specified timeval.
  (funcname_thread_add_timer) use funcname_thread_add_timer_timeval.
  (funcname_thread_add_timer_msec) ditto

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

* irdp_packet.c: (parse_irdp_packet) style issues.
  Use sockopt_iphdrincl_swab_systoh.
          Try unbork the code. Checksum the ICMP data and actually
          compare it to received checksum. Check data length against
          claimed length in header.
  Always use ntoh.. when accessing addresses, even when the
          comparison happens to be endian-safe.
  (send_packet) minor style isues. Use
          sockopt_iphdrincl_swab_htosys.
  (irdp_iph_hton/ntoh) IP header to/from network/host order.

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

* ospf_packet.c: replace ospf_swap_iph_to... with
          sockopt_iphdrincl_swab_...