[ospfd] fix rare leak of struct connected, in an error path.

2006-01-10 Juris Kalnins <juris@mt.lv>

* ospf_zebra.c: (ospf_interface_address_delete) fix rare leak of
  struct connected in an error case.

[ospfd] trim redundant strings

2006-01-10 Paul Jakma <paul.jakma@sun.com>

* ospf_vty.c: (config_write_ospf_distribute) trim down
  redundant strings.

[ospfd] fix automatic router-id and network enable bug.

2006-01-10 Paul Jakma <paul.jakma@sun.com>

* ospfd.c: (ospf_network_run) checking to see if router-id
  is set should be on ospf->router_id, not router_id_static.
  This was causing ospfd to not start if router-id had not
  been configured statically.
  (ospf_if_update) ditto.

[tests] Add small unit test for the new stream functions

2006-01-10 Paul Jakma <paul.jakma@sun.com>

* test-stream.c: new file, small unit test for new
  resize and {put,get}-quad stream functions.
* Makefile.am: build teststream unit test.

[stream] Add quad-word support and stream_resize

2006-01-10 Paul Jakma <paul.jakma@sun.com>

* stream.c: (stream_new) Allocate stream data as seperate object.
  (stream_free) free the data.
  (stream_resize) new function, resize stream to new size.
  (stream_{get,put}q*) new functions to get/put quad word size
  types.
* stream.h: (struct stream) make data seperate from the stream.
  Export new stream_resize and quad-word get/put functions.

remove dead code (from David Young)

2005-12-29  Greg Troxel  <gdt@fnord.ir.bbn.com>

        * vty.c (vty_hello): add cast to quiet lint (from David Young)

(patch-lint)

2005-12-29  Greg Troxel  <gdt@fnord.ir.bbn.com>

        * rt_socket.c (kernel_rtm_ipv4): Use AF_INET rather than AF_UNSPEC
        for mask.  From David Young.

shell script and hints for building on NetBSD.  (gdt deleted private
copy by accident when getting a fresh checkout, and on recovery would
like to have it safe, and also it may be useful to others.)

[ospfd] rename graceful to deferred, fix a tiny compile warning.

2005-11-26 Paul Jakma <paul.jakma@sun.com>

* (general) s/graceful/deferred/ in all files, the former term
   is confusing wrt OSPF Graceful-Restart.
* ospfd.c: (ospf_deferred_shutdown_check) dont return
  a function which returns void. SOS complains about this.
  (ospf_finish)

[c99] change gcc zero-length array to C99 flexible array declaration

2005-11-26 Paul Jakma <paul.jakma@sun.com>

* buffer.c: (struct buffer_data) change gcc zero array
  declaration to C99 incomplete array.
* stream.h: (struct stream) same
* ospf_api.c: (struct opaque_lsa) same

[zebra] fix connected_add_ipv6 declarations, label should be const.

2005-11-26 Paul Jakma <paul.jakma@sun.com>

* connected.{c,h}: (connected_add_ipv6) label should have
  const qualifier, fix declarations.

[build] Set default CFLAGS for SunPro, rationalise CFLAGS defaults.

2005-11-26 Paul Jakma <paul.jakma@sun.com>

* configure.ac: Collect together CFLAGS based on compiler
  detected a bit. Recognise and set default CFLAGS for SunPro /
  SOS10.

2005-11-25 Paul Jakma <paul.jakma@sun.com>

* texinfo.tex: update to newer version, seems to now properly
  scale the diagramme images in PDF output.

[zebra] Fix mistake in previous commit and further compile warnings/errors.

2005-11-24 Paul Jakma <paul.jakma@sun.com>

* kernel_socket.h: New header for functions exported to sysctl
  methods.
* kernel_socket.c: include previous.
  Remove static qualifier from couple of functions which are
  used by sysctl methods, incorrectly added in previous commit.
  Add a workaround for a bogus gcc warning to the RTA_ macros.
* Makefile.am: Add kernel_socket.h to noinst_HEADERS
* if_sysctl.c: include rt.h and kernel_socket.h and remove
  redundant prototypes.
* rtread_sysctl.c: ditto.
  (route_read) fix mismatch of return values.
* {rt,zserv,rib}.h: Include lib headers depended on.

[lib/zebra.h] fix Linux compile error

2005-11-24 Paul Jakma <paul.jakma@sun.com>

* zebra.h: s/u_int/unsigned int/, u_int is a BSD type, defining
  __USE_BSD on Linux pulls in further things from netinet/ip.h
  which dont preprocess for some reason. There is no C99
  shorthand type directly equivalent to u_int afaict, so don't
  use it.

[privs/solaris] Fix unused variable and incorrect format string.

2005-11-24 Paul Jakma <paul.jakma@sun.com>

* privs.c: (zcaps2sys/solaris) remove unused variable.
  (zprivs_state_caps/solaris) Format string missing a
  specifier.

[ospfclient] add static qualifier

2005-11-24 Paul Jakma <paul.jakma@sun.com>

* ospf_apiclient.c: add static qualifier to relevant functions.
* ospfclient.c: ditto

[zebra] fix some small compile errors, mark several functions static

2005-11-23 Paul Jakma <paul.jakma@sun.com>

* (general) fix some small compile errors, and mark several
          functions as static.
        * kernel_socket.c: (ifan_read) should be static.
          fix missing brackets.
          (ifm_read,ifam_read,rtm_read_mesg,kernel_read) Make static
          (ifam_read_mesg) make static. fix incorrect variable name.
          (rtm_read) make static. Fix call to rib_delete_ipv4 which
          should be rib_delete_ipv6.
          (routing_socket,kernel_init) should be static. Void argument
          should be specified as such, not left incomplete.
        * rt_netlink.c: rt.h should be included, contains prototypes of
          exported functions.
          (kernel_delete_ipv6_old) fix sign of index argument.
        * rt_socket.c: Exact same as previous. Also, make various
          functions static.
        * rtread_getmsg.c: Include zserv.h, which prototypes
          route_read. Make static.
        * rtread_sysctl.c: zserv.h and rt.h should be included.
          fix definition of route_read.

[tests] Add empty-path and sequence+1ASN test data to aspath_test.c

2005-11-23 Paul Jakma <paul.jakma@sun.com>

* aspath_test.c: Add an empty aspath to test segments, and to
  compare tests.
  Add a segment identical to seq1, but with one extra asn.
  Fix bogus free of stream in make_aspath for case where
  no stream was allocated (empty path data).

[bgp] Fix SEGV if empty path is confed-compared ([quagga-users 5968])

2005-11-23 Paul Jakma <paul.jakma@sun.com>

* bgp_aspath.c: (assegments_parse) should be static

2005-11-23 Juergen Kammer <j.kammer@eurodata.de>

* bgp_aspath.c: (aspath_cmp_left_confed) fix SEGV for case
  where one or both paths are empty.

2005-11-20 Paul Jakma <paul.jakma@sun.com>

        * ospf_abr.c: (ospf_abr_announce_network_to_area) check
          returned LSA of ospf_summary_lsa_refresh and print warning if
          it failed.
          (ospf_abr_announce_network_to_area) similar
          (ospf_abr_announce_rtr_to_area) similar
        * ospf_lsa.c: (ospf_router_lsa_new) check LSA returned is valid.
          (ospf_router_lsa_originate) similar
          (ospf_router_lsa_refresh, ospf_network_lsa_new) similar
          (ospf_summary_lsa_new) Check ID is valid.
          (ospf_summary_lsa_originate) ditto, and check returned LSA from
           previous function is !NULL.
          (ospf_summary_lsa_refresh) check ospf_summary_lsa_new return
           is !NULL.
          (ospf_summary_asbr_lsa_new) ID valid check.
          (ospf_summary_asbr_lsa_originate) similar.

2005-11-20 Paul Jakma <paul.jakma@sun.com>

        * ospfd.h: remove the OSPF_ROUTER_ID_UPDATE_DELAY define
          (struct ospf) remove the router_id timer thread.
          remove export of ospf_router_id_update_timer.
        * ospfd.c: (ospf_router_id_update) call ospf_if_update to
          poke interfaces into action after ID has been configured.
          (ospf_router_id_update_timer) removed.
          (ospf_finish_final) t_router_id_update timer is gone.
          (ospf_network_run) router-id update timer gone.
          call ospf_router_id_update directly if ID not configured.
          In the per-iface loop, don't ospf_if_up interfaces if
          ID is still not configured. The update function will call
          ospf_if_update anyway.
          (ospf_if_update) ID update timer is gone. Just return if no
          ID is set.
        * ospf_vty.c: (ospf_router_id) call ospf_router_id_update, no
          timer needed.
        * ospf_zebra.c: (ospf_router_id_update_zebra) call
          ospf_router_id_update directly, not via timer.

[isisd] remove includes of common system headers.

2005-11-20 Paul Jakma <paul.jakma@sun.com>

        * (general) remove includes of very common system headers,
          these are already picked up by zebra.h. Further, including
          them before zebra.h messes up all our lovely (sarcasm)
          autoconf'd detecting of platform specific things. Recent
          addition of stdint.h to configure.ac and zebra.h appears
          particularly to throw up this problem.

- quagga.spec.in: Release sub-rev now release_rev macro, for rpmbuild -D

[ospfd] Standardize buffer sizes used for displaying timers.
2005-11-16 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* ospf_dump.h: Define OSPF_TIME_DUMP_SIZE as appropriate buffer size
  for use with ospf_timer_dump and ospf_timeval_dump.
* ospf_vty.c: Change all buffer sizes used with ospf_timer_dump and
  ospf_timeval_dump to have size OSPF_TIME_DUMP_SIZE.
  (show_ip_ospf_interface_sub) Fix possible buffer overflow in
  call to ospf_timer_dump.

[ospfd] Make OSPF_ISM_TIMER_OFF macro safer.
2005-11-16 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* ospf_ism.h: (OSPF_ISM_TIMER_OFF) Improve macro syntax by enclosing
  in 'do {...} while(0)'.

[workqueue] Update workqueue users callbacks to additional arguments

2005-11-14 Paul Jakma <paul.jakma@sun.com>

* (general) pass struct work-queue to callback functions.
* workqueue.h: (struct work_queue) move the state flag
  variables to end.
  Add an opaque pointer to spec, for user-data global to the
  queue.
  Pass reference to work_queue to all callbacks.
* workqueue.c: (work_queue_item_remove) pass ref to workqueue
  to user callbacks.
  (work_queue_run) ditto.

2005-11-14 Paul Jakma <paul.jakma@sun.com>

* zebra_rib.c: (rib_process) convert to new workqueue specs and
  shut up gcc, which complains about cast from void via
  function parameters, for some dumb reason. Do the cast
  inside the function instead.
  (rib_queue_qnode_del) ditto.
  (rib_queue_init) no need for the casts anymore.

[bug #231] check and include stdint.h, also add std=c99 to gcc CFLAGS

2005-11-14 Paul Jakma <paul.jakma@sun.com>

* configure.ac: Tell gcc we like C99.
  [bug #231] Check and test for stdint.h.
* lib/zebra.h: [bug #231] include stdint, if its there.

2005-11-14 Paul Jakma <paul.jakma@sun.com>

* (general) Add state to detect queue floods.  There's no sense
  trying to be sparing of CPU resources, if the queue is
  flooding and using ever more memory resources. we should just
  get on with clearing the queue.
  The sense of delay and hold were wrong way around, fix.
* workqueue.h: (struct work_queue) Add status bitfield.  Add
  'flood' integer to workqueue spec.  Add runs_since_clear
  counter to workqueue.
* workqueue.c: (work_queue_new) set defaults for delay, hold
  and flood.
  (work_queue_add) initial schedule should use delay, not hold.
  (show_work_queues) Print flood field, conserve whitespace.
  (work_queue_unplug) use delay, not hold.
  (work_queue_run) consecutive runs should be seperated by hold
  time, not delay.
  Keep track of number of consecutive runs, go into 'overdrive'
  if queue is being flooded, we can't avoid making heavy use of
  resources, better to use CPU than ever more RAM.

[ospfd] misc small fixes. Fix default stub router setting. no opaque compile.

2005-11-14 Paul Jakma <paul.jakma@sun.com>

* ospfd.c: (ospf_new) stub-shutdown should just default to
  unconfigured, too strange otherwise.
  (ospf_finish_final) t_opaque_lsa_self TIMER_OFF should be
  preprocessor conditional on HAVE_OPAQUE_LSA.
* ospfd.h: (struct ospf) remove the SHUTDOWN_DEFAULT define.
  no longer used, plus it wasn't in range that the command
  accepted.
* ospf_zebra.h: Depends on vty.h, include it.

[solaris] Extra debug and cross-checks for kernel_socket, two Sol9 fixes.

2005-11-12 Alexander Gall <gall@switch.ch>

* See [quagga-dev 1815]
* kernel_socket.c: (rtm_write) Use SAROUNDUP when HAVE_SIN_LEN
  is not available.
* rt_socket.c: (kernel_rtm_ipv6(_multipath)) set family to
  AF_INET6 on ipv6 routes.

2005-11-12 Paul Jakma <paul.jakma@sun.com>

* kernel_socket.c: Add RTA_NAME_GET macro to extract name from
  sockaddr_dl. Add some more RTF_ flags.
* (ifan_read) Add some debug messages.
* (ifm_read) Add more debug messages. More robust cross-checks
  of index against name.
  Fall back to by-name lookup if the index lookup fails, future
  proofing more than anything else.
  (ifam_read_mesg) Read RTA_IFP. Add debug messages.
  (ifam_read) More debug. If there's an RTA_IFP and it isn't
  the name of the interface, save it as the label.
  (rtm_read_mesg) Read RTA_IFP.
  (rtm_read) allow name to be retrieved.
  (rtmsg_debug) expand on the debug message.

[lib] 'show route-map' should print call action seperate from exit policy

2005-11-12 Paul Jakma <paul.jakma@sun.com>

* routemap.c: (vty_show_route_map_entry) call action is
  seperate from exit action, latter should still be printed
  regardless of whether a call is specified.

[ospfd] SPF ospf_canonical_nexthops_free bugfix.

[doc] quagga.info updated /again/ due to version bump, d'oh.

2005-11-11 Paul Jakma <paul.jakma@sun.com>

* NEWS: Update.
* configure.ac: Bump to 0.99.2

- commit the actual file..

[trivia] update quagga.info, auto-built file

2005-11-11 Paul Jakma <paul.jakma@sun.com>

* quagga.info: update auto-built file. Methinks it's time to
  remove from CVS..

[zebra] Fix warning and small connected-label changes merge error.

2005-11-11 Paul Jakma <paul.jakma@sun.com>

* kernel_socket.c: (ifm_read) arithmetic on void pointer
  warning.
  (ifam_read) Fix error from connected-with-label merge,
  something crept in from the pending Solaris kernel_socket.c
          patch which shouldn't have.

2005-11-10 Paul Jakma <paul.jakma@sun.com>

* HACKING: Add recommendation to provide a single Subject
  style description to the commit message.
  Add some recommendations for ChangeLog.

[doc] List snmptrap.texi as a source, so it gets included in dist

2005-11-07 Paul Jakma <paul.jakma@sun.com>

* Makefile.am: snmptrap.texi needs to be listed as a source,
  not disted otherwise.

- Makefile.am: quagga.pam.stack should be in dists
- quagga.spec.in: introduce dist variable to allow for differences
  e.g in buildreqs, pam config files. etc.

2005-11-04 Paul Jakma <paul.jakma@sun.com>

* quagga.info: Update auto-built file
* ospf6d.texi: Add example config
* bgpd.texi: Add example configs. Couple of cleanups of format
  and macros.
* routemap.texi: Add an explanation of how route-maps work.
  Document the call and exit-policy commands.

2005-11-04 Paul Jakma <paul.jakma@sun.com>

* snmptrap.texi: Contributed documentation, contributors name
  is lost (please get in touch). Configuring SNMP for logging
  traps.
* snmp.texi: Minor formatting changes.
* quagga.info: Update auto-built file

2005-11-04 Paul Jakma <paul.jakma@sun.com>

* prototype.dev.in: lib/md5-gnu.h is now lib/md5.h

2005-11-04 Paul Jakma <paul.jakma@sun.com>

* ospf_{dump,spf,vty}.c: Oops, use the internal tv_sub
  function rather than unportable timersub.

- quagga.pam: pam_stack.so module is deprecated, use 'include' instead.
- quagga.pam.stack: the old pam_stack way, kept to allow spec file to
  backwards compatible (changes to spec file pending local testing)

- bah, fix likkle typo.

2005-11-03 Paul Jakma <paul.jakma@sun.com>

* memtypes.h: Update auto-built file.

2005-11-03 Paul Jakma <paul.jakma@sun.com>

* bgp_damp.c: (bgp_reuse_timer) struct bgp can be retrieved via
  the struct bgp_damp_info, no need to guess by using
  bgp_get_default().

2005-11-03 Paul Jakma <paul.jakma@sun.com>

* connected.{c,h}: Include memory.h
  (connected_add_ipv4) Use MTYPE for ifc label.
  (connected_add_ipv6) Also should accept label. Store it in ifp.
  (connected_del_ipv4) Taking label as argument is pointless.
* rt_netlink.c: (netlink_interface_addr) update label usage
  for connected_{add,delete} functions.
* if_ioctl.c: (if_getaddrs) NULL label for connected_add_ipv6.
* if_ioctl_solaris.c: (interface_list_ioctl) Pass LIFC_NOXMIT
  so we also find out about NOXMIT interfaces like VNI.
  Bit of hackery to turn interface names into the primary
  interface name, later with routing socket messages we only
  will about primary interfaces anyway, so we must normalise
  the name.
  (if_get_addr) take label as argument, so it can
  be passed to connected_add.
  If label is provided, then it is interface name to issue the
  ioctl for address information on, not the ifp name.
  (interface_list) List AF_UNSPEC too, just in case.
* if_proc.c: (ifaddr_proc_ipv6) label for connected_add_ipv6.
* interface.c: (if_addr_wakeup) Some very bogus code - sets
  IFF_RUNNING - add comment.
  (if_refresh)
  (ip_address_install) Use MTYPE for ifc label.
* ioctl_solaris.c: (if_mangle_up) New function. Hackery to make
  IFF_UP reflect whether any addresses are left on the
  interface, as we get signalled for IFF_UP flags change on the
  primary interface only. Logical interfaces dont generate
  IFINFO, but we do get an RTM_DELADDR.
  (if_get_flags) Call if_mangle_up before return.
* kernel_socket.c: (ifam_read) Fixup calls to
  connected_{add,delete} to match above changes. Rename gate
  variable to brd, less confusing.
  Pass the interface name as a label, if it is not same name
  as ifp->name.

2005-11-03 Paul Jakma <paul.jakma@sun.com>

* ospf_packet.c: Change level of some warnings to
  informational.

2005-11-03 Paul Jakma <paul.jakma@sun.com>

* if.c: (connected_free) use MTYPE for connected label.
  memtypes.c: Add MTYPE_CONNECTED_LABEL

2005-11-03 Paul Jakma <paul.jakma@sun.com>

* ospf_apiserver.c: (apiserver_sync_callback) stray semi-colon
* ospf_packet.c: include checksum.h, remove the in_cksum extern
* prototypes.
* ospf_te.h: Add braces, quell warning.

2005-11-03 Paul Jakma <paul.jakma@sun.com>

* zebra.h: BSD BYTE_ORDER define isn't available everywhere,
  define if needs be.
* checksum.h: new file. checksum.c exports in_cksum, provide
  a header for it.
* checksum.c: (in_cksum) callers shouldn't have to know it uses
  a u_short internally, change to void *.
* Makefile.am: Add checksum.h
* command.h: remove bogus trailling slash.
* md5.c: (general) Update it for the twentieth century. ANSI
  declarations are widely supported now.. Don't include system
  headers, only include zebra.h. Use POSIX types (the
  alternative is to define u_int64_t in a portable way - rest
  of Quagga needs same cleanup).
  Make endian-conditional code be compiler conditional rather
  than preprocessor conditional, so that breakage gets noticed
  quicker.
* md5.h: POSIX types. Get rid of the odd __P() non-ANSI capable
  compiler compatibility hack.

2005-10-30 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* ripd.c: (rip_response_process) Instead of calling
  rip_interface.c:if_valid_neighbor(), call the equivalent
  library function if_lookup_address().
* rip_interface.c: (if_valid_neighbor) Remove function, since it is
  essentially equivalent to the if_lookup_address() library function.
* ripd.h: (if_valid_neighbor) Remove function declaration.

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

* ospfd.texi: Document the new spf and max-metric commands, and
  the additional form of dead-interval. Add documentation for
  various other commands. Cleanup misc stuff, citations, etc.
  Add some example configurations.
* overview.texi: RFC3137 support added.
* Makefile.am: Make quagga.pdf depend on the _TEXINFOS
  variable, this still doesn't fix the dependency though,
  sadly.

2005-10-29 Paul Jakma <paul.jakma@sun.com>

* (general) RFC3137 stub-router support
* ospfd.h: Add OSPF_OUTPUT_COST_INFINITE define.
  (struct ospf_master) Add a OSPF_MASTER_SHUTDOWN flag for
  options, to allow shutdown to distinguish between complete
  shutdown and shutdown of a subset of ospf instances.
  (struct ospf)
  Add stub_router_{startup,shutdown_}time, configuration of startup
  and shutdown time for stub-router.
  Add t_graceful_shutdown struct thread, timer for graceful
  shutdown, if needed.
  (struct ospf_area) Add stub_router_state - run time state of
  stub-router for an area. Add flags for ADMIN, IS and WAS
  states.
  Add t_stub_router, timer thread to resend router-lsa for an
  area.
* ospf_lsa.c: (ospf_link_cost) new simple function to spit out
  either the given lnks cost or infinite cost if stub-router is
  in effect.
  (lsa_link_{ptop,broadcast,virtuallink,ptomp}_set) use
  previous function for transit-links.
  (ospf_stub_router_timer) timer thread for end of startup stub
  router. Change state as required for the area and setup
  re-origination of router-lsa.
  (ospf_stub_router_check) Check/do whether stub-router should be
  enabled, and whether it requires timer to be setup.
  (ospf_router_lsa_new) call previous function at top.
  (ospf_router_lsa_originate) no external callers, made static.
* ospf_lsa.h: (ospf_router_lsa_originate) removed.
* ospf_main.c: (sigint) make static.
  remove call to exit, as ospf_terminate now deals with
  exiting.
* ospf_route.c: (ospf_terminate) removed, now in ospfd.c.
* ospf_vty.c: (show_ip_ospf_area) print out state of
  stub-router, if active.
  (show_ip_ospf) print out configuration of stub-router
  support, and details of graceful-shutdown if the timer is
  active.
  ((no)?ospf_max_metric_router_lsa_{admin,startup,shutdown}) new
  commands to (de-)?configure stub-router support.
  (config_write_stub_router) write out config of stub-router.
  (ospf_config_write) call previous.
  (ospf_vty_init) install the new stub-router commands.
* ospfd.c: various functions made static.
  (ospf_new) Set defaults for stub-router. Graceful shutdown
  is made to default on, just to be adventerous.
  (ospf_graceful_shutdown_finish) new function, final part of
  shutdown.
  (ospf_graceful_shutdown_timer) timer thread wrapper for
  graceful-shutdown.
  (ospf_graceful_shutdown_check) check whether to setup timer
  for shutdown or proceed directly to final shutdown.
  (ospf_terminate) moved here from ospf_route.c, call
  ospf_finish for each instance.
  (ospf_finish) renamed to ospf_finish_final and made static.
  (ospf_finish) new function, exported wrapper around
  ospf_graceful_shutdown_check.
  (ospf_finish_final) complete shutdown of an instance.
  Add missing TIMER_OFF's of two timer threads.
  (ospf_area_free) opaque self lsa timer should be turned off.

2005-10-28 Paul Jakma <paul.jakma@sun.com>

* Makefile.am: Add rip_interface.h, or else it doesn't get
  included in dists.

2005-10-26 Paul Jakma <paul.jakma@sun.com>

* command.c: Use MTYPE_HOST, MTYPE_STRVEC. Some other fixups,
  including fixing some likely leaks in config_write_file.
* vty.c: memory macro usage fixes.
  (vty_read_config) fix leak where relative config file is
  specified.

2005-10-26 Paul Jakma <paul.jakma@sun.com>

* (general) Cleanup a some calls to XFREE,strdup, etc. to use
  the memory.h macros.
* memtypes.c: Add MTYPE_IF_RMAP_NAME, MTYPE_PQUEUE,
  MTYPE_PQUEUE_DATA and MTYPE_HOST.
* memtypes.h: update auto-built file.
* if_rmap.c: Use MTYPE_IF_RMAP_NAME.
* pqueue.c: Use the two MTYPE_PQUEUE mtypes for allocations.

2005-10-26 Paul Jakma <paul.jakma@sun.com>

* ripd.c: Update couple more functions to specify void
  explicitely.

2005-10-26 Paul Jakma <paul.jakma@sun.com>

* (general) static/extern functions and definitions.
* rip_interface.h: new file, export the public functions from
  rip_interface.c

2005-10-23 Paul Jakma <paul.jakma@sun.com>

* ospf_apiserver.c: (ospf_apiserver_term) This function should
  not have side-effects (eg segv) if no apiserver instances are
  active, ie be robust.

2005-10-23 Paul Jakma <paul.jakma@sun.com>

* ospf_vty.c: (show_ip_ospf) fix display of SPF timer if it
  has not yet been run.

2005-10-21 Paul Jakma <paul.jakma@sun.com>

* ospf_dump.c: (ospf_timeval_dump) fix ms adjustment, thanks to
  Andrew Schorr.
* ospf_vty.c: (ospf_config_write) fix write out of spf timers
  configuration.

2005-10-21 Paul Jakma <paul.jakma@sun.com>

* (general) SPF millisecond resolution timer with adaptive,
  linear back-off holdtime. Prettification of ospf_timer_dump.
* ospf_dump.c: (ospf_timeval_dump) new function. The guts of
  ospf_timer_dump, but made to be more dynamic in printing out
  the relative timeval, sliding the precision printed out
  according to the value.
  (ospf_timer_dump) guts moved to ospf_timeval_dump.
* ospf_dump.h: export ospf_timeval_dump.
* ospf_flood.c: (ospf_flood) remove gettimeofday, use
  the libzebra exported recent_time instead, as it's not
  terribly critical to have time exactly right - the dropped
  LSA will be retransmited to us if we don't ACK it.
* ospf_packet.c: (ospf_ls_upd_timer) Ditto, but here we're
  not transmitting, just putting LSA back on update transmit list.
* ospfd.h: delay and holdtimes should be unsigned.
  Add spf_max_holdtime and spf_hold_multiplier.
  Update default defines for delay and hold time to be in msec.
  (struct ospf) change the SPF timestamp to a struct timeval.
  Remove ospf_timers_spf_(un)?set.
* ospfd.c: (ospf_timers_spf_{set,unset}) removed.
  (ospf_new) initialise spf_max_holdtime and spf_hold_multiplier
* ospf_spf.c: (ospf_spf_calculate) SPF timestamp is a timeval
  now, update with gettimeofday.
  (ospf_spf_calculate_schedule) Change SPF timers to millisecond
  resolution.
  Make the holdtime be adaptive, with a linear increase in
  holdtime ever consecutive SPF run which occurs within holdtime
  of previous SPF, bounded by spf_max_holdtime.
* ospf_vty.c: Update spf timers commands.
  (ospf_timers_spf_set) trivial helper.
  (ospf_timers_throttle_spf_cmd) new command to set SPF delay,
  initial hold and max hold times with millisecond resolution.
  (ospf_timers_spf_cmd) Deprecated. Accept the old values,
  convert to msec, truncate to new limits.
  (no_ospf_timers_throttle_spf_cmd) set timers to defaults.
  (no_ospf_timers_spf_cmd) deprecated form, same as previous.
  (show_ip_ospf_cmd) Display SPF parameters and times.
  (show_ip_ospf_neighbour_header) Centralise the 'sh ip os ne'
  header.
  (show_ip_ospf_neighbor_sub) Fix the field widths. Get rid of
  the multiple spaces which were making the lines even longer.
  (show_ip_ospf_neighbor_cmd) Use show_ip_ospf_neighbour_header
  (show_ip_ospf_neighbor_all_cmd) ditto and fix the field
  widths for NBMA neighbours.
  (show_ip_ospf_neighbor_int) Use header function.
  (show_ip_ospf_nbr_nbma_detail_sub) use sizeof for timebuf,
  local array - safer.
  (show_ip_ospf_neighbor_detail_sub) ditto
  (ospf_vty_init) install the new SPF throttle timer commands.

2005-10-20 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* sockopt.c: (setsockopt_multicast_ipv4) If IP_ADD_MEMBERSHIP
  fails with errno equal to EADDRINUSE, then issue an info
  message and try IP_DROP_MEMBERSHIP followed by IP_ADD_MEMBERSHIP.

2005-10-21 Paul Jakma <paul.jakma@sun.com>

* (general) OSPF fast, sub-second hello and 1s dead-interval
  support. A warning fix. Millisec support for ospf_timer_dump.
  Change auto-cost ref-bandwidth to add a comment to write out
  of config, rather than printing annoying messages to vty on
  startup.
* ospf_dump.c: (ospf_timer_dump) Print out milliseconds too.
  Callers typically specify a length of 9, so most see
  millisecs unless they specify the additional length.
* ospf_interface.h: (struct ospf_interface) new interface param,
  fast_hello.
* ospf_interface.c: (ospf_if_table_lookup) add brackets,
  gcc warning fix.
  (ospf_new_if_params) Initialise fast_hello param.
  (ospf_free_if_params) Check whether fast_hello is configured.
  (ospf_if_new_hook) set fast_hello to default.
* ospf_ism.h: Wrap OSPF_ISM_TIMER_ON inside do {} while (0) to
  prevent funny side-effects from its if statement when this
  macro is used conditionally by other macros.
  (OSPF_ISM_TIMER_MSEC_ON) new macro, set in milliseconds.
  (OSPF_HELLO_TIMER_ON) new macro to set hello timer according
  to whether fast_hello is set.
* ospf_ism.c: Update all setting of the hello timer to use
  either OSPF_ISM_TIMER_MSEC_ON or OSPF_HELLO_TIMER_ON. The
  former is used when hello is to be sent immediately.
* ospf_nsm.c: ditto
* ospf_packet.c: (ospf_hello) hello-interval is not checked
  for mismatch if fast_hello is set.
  (ospf_read) Annoying nit, fix "no ospf_interface" to be debug
  rather than a warning, as it can be perfectly normal to
  receive packets when logical subnets are used.
  (ospf_make_hello) Set hello-interval to 0 if fast-hellos are
  configured.
* ospf_vty.c: (ospf_auto_cost_reference_bandwidth) annoying
  nit, don't vty_out if this command is given, it gets tired
  quick.
  (show_ip_ospf_interface_sub) Print the hello-interval
  according to whether fast-hello is set or not.
  Print the extra 5 millisec characters from (ospf_timer_dump)
  if fast-hello is configured.
  (ospf_vty_dead_interval_set) new function, common to all
  forms of dead-interval command, to set dead-interval and
  fast-hello correctly. If a dead-interval is given, unset
  fast-hello, else if a hello-multiplier is set, set
          dead-interval to 1 and fast-hello to given multiplier.
  (ip_ospf_dead_interval_addr_cmd) use
  ospf_vty_dead_interval_set().
  (ip_ospf_dead_interval_minimal_addr_cmd) ditto.
  (no_ip_ospf_dead_interval) Unset fast-hello.
  (no_ip_ospf_hello_interval) Bug-fix, unset of hello-interval
  should set it to OSPF_HELLO_INTERVAL_DEFAULT, not
  OSPF_ROUTER_DEAD_INTERVAL_DEFAULT.
  (config_write_interface) Write out fast-hello.
  (ospf_config_write) Write a comment about
  "auto-cost reference-bandwidth" having to be equal on all
  routers. Hopefully just as noticeable as old practice of
  writing to vty, but less annoying.
  (ospf_vty_if_init) install the two new dead-interval
  commands.
* ospfd.h: Add defines for OSPF_ROUTER_DEAD_INTERVAL_MINIMAL
  and OSPF_FAST_HELLO_DEFAULT.

2005-10-20 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* memory.c: (show_memory_vty) Omit zero statistics.  Print separators
  only if some non-zero stats have been printed in between.
  (show_separator) New function to print a separator.
  (show_memory_all) Keep track of whether a separator is needed
  between the different memory statistics groups.

2005-10-17 Vincent Jardin <vincent.jardin@6wind.com>

        * ripd.c: rip_create_socket() for each packet, it does not bind to the
          proper interfaces because we forget to use the from address when
          it is specified.

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

* memtypes.h: Add MTYPE_OSPF_VERTEX_PARENT (to match memtypes.c).

2005-10-18 Paul Jakma <paul.jakma@sun.com>

* (general) SPF memory management cleanup and fix for rare
  double-free bug.
* ospf_spf.h: (struct vertex_parent) New struct to hold parent
  specific data, eg the backlink and the parent vertex pointer,
  and point to the appropriate general struct vertex_nexthop.
  (struct vertex_nexthop) remove parent vertex pointer, so
  this struct can be shared across vertices.
  (struct vertex) rename list child to list children. Remove
  list of nexthops, replace with list of vertex_parents.
* ospf_spf.c: (update_stat) trivial, remove cast from void *.
  (vertex_nexthop_new) remove init of parent - field is gone
          from struct vertex_nexthop.
          (ospf_canonical_nexthops_free) Remove the canonical
          vertex_nexthop memory objects. These are the vertex_nexthops
          attached to the first level of router vertices from the root.
          (vertex_parent_new) new function, create a vertex_parent.
          (vertex_parent_free) ditto, but free it.
          (ospf_vertex_new) Update to match changes to struct vertex.
          (ospf_vertex_free) Recursively free a struct vertex and its
          children. The parent list is used as a reference count.
          vertex_nexthops must be free seperately, if required.
          (ospf_vertex_dump) update to match struct vertex changes.
          Print out backlink of parents too.
          (ospf_vertex_add_parent) ditto.
          (ospf_lsa_has_link) update comment.
          (ospf_nexthop_add_unique) removed, not needed anymore.
          (ospf_nexthop_merge) ditto.
          (ospf_spf_consider_nexthop) renamed to ospf_spf_add_parent.
          Simplified to just create vertex_parent and add it.
          (ospf_spf_flush_parents) new function, flush out the parent
  list.
  (ospf_nexthop_calculation) Take the relevant route_lsa_link
  as an argument, which simplifies things and removes the need
  for the hack in ospf_nexthop_add_unique - ospf_spf_next
  already knew exactly which link the cost calculated was for.
  Update to match struct vertex changes too.
  (ospf_spf_next) Don't create a vertex for W unnecessarily, if
          it's there's a vertex already created for W, use it, and
          hence there's no need to free it either.
          Update some manipulation/comparisons of distance to match.
          Flush the parent list if a lower cost path is found.
          (ospf_spf_route_free) unused, removed.
          (ospf_spf_dump) match the struct vertex changes, and dump the
          ifname if possible.
          (ospf_spf_calculate) At end of SPF, free the canonical nexthops
          and call ospf_vertex_free on the root vertex to free the
  entire tree.
* ospf_interface.c: (ospf_vl_set_params) match struct vertex
          changes.
        * ospf_route.c: (ospf_intra_route_add) ditto
          (ospf_route_copy_nexthops_from_vertex) ditto
* memtypes.c: (memory_list_ospf) Add MTYPE_OSPF_VERTEX_PARENT.

2005-10-11 Paul Jakma <paul.jakma@sun.com>

* ospf_api.c: sign warnings.
* ospf_apiserver.c: sign warning and convert all the struct
  in_addr initialisations so as not to make assumptions about
  how this struct is organised, initialise the s_addr member
  explicitely.
* ospf_packet.c: Add const qualifier to auth_key.

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

* quagga.texi: Rejiggle the copyright sections a bit, seems to
  make makeinfo happier and generates info pages again on FC4.
* quagga.info: Update this auto-built file.

2005-10-11 Paul Jakma <paul.jakma@sun.com>

* connected.{c,h}: (connected_{add,delete}_ipv4) label should
  be const qualified.

2005-10-11 Paul Jakma <paul.jakma@sun.com>

* test-privs.c: Privileges unit test.
* Makefile.am: build testprivs
* aspath_test.c: fix a few sign warnings

2005-10-06 Alain Ritoux <alain.ritoux@6wind.com>

        * ospf_snmp.c: Avoid mixing interface and ospf_interface objects
          which now allows snmpwalk to work with ospfIfTable and
          also with ospfIfMetricTable

2005-10-06 Alain Ritoux <alain.ritoux@6wind.com>

        * rip_interface.c: Now the command "no ip rip split-horizon
          poisoned-reverse" just inhibates the poisoned-reverse effects
          but keep spli-horizon activ.

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

* zebra.h: Declare new functions zebra_route_string() and
  zebra_route_char().
* log.c: (zroute_lookup,zebra_route_string,zebra_route_char) New
  functions to map zebra route numbers to strings.
* zebra_vty.c: (route_type_str) Remove obsolete function: use new
  library function zebra_route_string() instead.  Note that there
  are a few differences: for IPv6 routes, we now get "ripng" and
  "ospf6" instead of the old behavior ("rip" and "ospf").
  (route_type_char) Remove obsolete function: ues new library function
  zebra_route_char() instead.  Note that there is one difference:
  the old function returned 'S' for a ZEBRA_ROUTE_SYSTEM route,
  whereas the new one returns 'X'.
  (vty_show_ip_route_detail,vty_show_ipv6_route_detail) Replace
  route_type_str() with zebra_route_string().
  (vty_show_ip_route,vty_show_ipv6_route) Replace route_type_char()
  with zebra_route_char().
* bgp_vty.c: (bgp_config_write_redistribute) Use new library function
  zebra_route_string instead of a local hard-coded table.
* ospf6_asbr.c: Remove local hard-coded tables zroute_name and
  zroute_abname. Change the ZROUTE_NAME macro to use new library
  function zebra_route_string().  Remove the ZROUTE_ABNAME macro.
  (ospf6_asbr_external_route_show): Replace ZROUTE_ABNAME() with
  a call to zebra_route_char(), and be sure to fix the format string,
  since we now have a char instead of a char *.
* ospf6_zebra.c: Remove local hard-coded tables zebra_route_name and
  zebra_route_abname.  Note that the zebra_route_name[] table
  contained mixed-case strings, whereas the zebra_route_string()
  function returns lower-case strings.
  (ospf6_zebra_read_ipv6): Change debug message to use new library
  function zebra_route_string() instead of zebra_route_name[].
  (show_zebra): Use new library function zebra_route_string() instead
  of zebra_route_name[].
* ospf_dump.c: Remove local hard-coded table ospf_redistributed_proto.
  (ospf_redist_string) New function implemented using new library
  function zebra_route_string().  Note that there are a few differences
  in the output that will result: the new function returns strings
  that are lower-case, whereas the old table was mixed case.  Also,
  the old table mapped ZEBRA_ROUTE_OSPF6 to "OSPFv3", whereas the
  new function returns "ospf6".
* ospfd.h: Remove extern struct message ospf_redistributed_proto[],
  and add extern const char *ospf_redist_string(u_int route_type)
  instead.
* ospf_asbr.c: (ospf_external_info_add) In two messages, use
  ospf_redist_string instead of LOOKUP(ospf_redistributed_proto).
* ospf_vty.c: Remove local hard-coded table distribute_str.
  (config_write_ospf_redistribute,config_write_ospf_distribute): Use
  new library function zebra_route_string() instead of distribute_str[].
* ospf_zebra.c: (ospf_redistribute_set,ospf_redistribute_unset,
  ospf_redistribute_default_set,ospf_redistribute_check)
  In debug messages, use ospf_redist_string() instead of
  LOOKUP(ospf_redistributed_proto).
* rip_zebra.c: (config_write_rip_redistribute): Remove local hard-coded
  table str[]. Replace str[] with calls to new library function
  zebra_route_string().
* ripd.c: Remove local hard-coded table route_info[].
  (show_ip_rip) Replace uses of str[] with calls to new library
  functions zebra_route_char and zebra_route_string.
* ripng_zebra.c: (ripng_redistribute_write) Remove local hard-coded
  table str[].  Replace str[i] with new library function
  zebra_route_string(i).
* ripngd.c: Remove local hard-coded table route_info[].
  (show_ipv6_ripng) Use new library function zebra_route_char() instead
  of table route_info[].

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

* rt_netlink.c: (netlink_request) Use memset to clear structure
  before calling sendto (eliminates a valgrind error message about
  uninitialized data).

* isis_adjacency.c: Stop expire timer while deleting adjacency.
* isis_events.c: Stop pseudo LSP thread while resigning circuit from
  level.
* isis_route.c: Fix compiling with EXTREME_DEBUG. Mark route as not
  in sync with zebra if it's changed.
* isis_spf.c: Schedule route validating etc even if tent was empty.
  It's probably because we just don't have any adjacencies.
* isisd.c: Write minimum spf interval into configuration.

2005-09-30 Vincent Jardin <vincent.jardin@6wind.com>

* isis_route.c: fix EXTREME_DEBUG compilation

2005-09-30 Vincent Jardin <vincent.jardin@6wind.com>

* ospf_dump.c, ospf_ia.c, ospf_spf.c, ospf_ase.c:
  remove unused DEBUG

2005-09-30 Vincent Jardin <vincent.jardin@6wind.com>

* bgp_packet.c: fix compilation when DEBUG is used.

2005-09-29 Alain Ritoux <alain.ritoux@6wind.com>

    * lib/smux.[ch]: allow to retreive global OID (identified by <0
      namelen).
    * ospf_ism.c: generate SNMP traps on Interface state change
    * ospf_nsm.c: generate SNMP traps on Neighbour state change
    * ospf_snmp.[ch]: support for SNMP traps for interface and neighbours.

2005-09-29 Paul Jakma <paul.jakma@sun.com>

* configure.ac: Add the test for Solaris least-privileges. Set
  defines for whether capabilities are supported and whether of
  the linux or solaris variety.
  Add missing-prototypes, missing-declarations, char-subscripts
  and cast-qual warnings to default cflags, cause Hasso enjoys warnings,
  and we really should clean the remaining ones up. (ie isisd..).
* (*/*main.c) Update the zebra_capabilities_t arrays in the various
  daemons to match the changes made in lib/privs.h.
* zebra.h: Solaris capabilities requires priv.h to be included.
* privs.{c,h}: Add support for Solaris Least-Privileges.
  privs.h: Reduce some of the abstract capabilities, which do
  not have rough equivalents on both systems. Rename the net
  related caps to _NET, as they should have been in first
  place.
  (zprivs_terminate) should take the zebra_privs_t as argument so
  that it can update change pointer.
  Add an additional privilege state, ZPRIVS_UNKNOWN.
* privs.c: (various capability functions) Add
  Solaris privileges variants.
  (zprivs_state) Use privs.c specific generic types to
          represent various capability/privilege related types, so that
          each can be typedef'd as appropriate on each platform.
  (zprivs_null_state) static added, to hold the state the null
  method should report (should be raised by default, and
          LOWERED if zprivs_terminate has been called)
          (zprivs_state_null) Report back the zprivs_null_state.
          (cap_map) Make it able to map abstract capability to multiple
          system capabilities.
          (zcaps2sys) Map to abstract capabilities to multiple system
          privileges/capabilities.
          (zprivs_init) move capability related init to seperate
  function, zprivs_caps_init.
          (zprivs_terminate) ditto, moved to zprivs_caps_terminate.
          Set the change_state callback to the NULL state, so the
          user can continue to run and use the callbacks.

2005-09-29 Alain Ritoux <alain.ritoux@6wind.com>
   * Of course ISM_SNMP MACRO mist be defined

2005-09-29 Alain Ritoux <alain.ritoux@6wind.com>

    * ospfd/ospf_vty.c: forece default route LSA to be re_issued whenever
      cost is changed ( [no] ip ospf area XXX default-cost YYY)
      Support ignore-mtu option
    * ospfd/ospfd.h: define OSPF_MTU_IGNORE_DEFAULT
    * ospfd/ospf_packet.c: support ignore-mtu option
    * ospfd/ospf_interface.h: field added for skipping MTU check
    * ospfd/ospf_interface.c: fix memory leak in ospf_crypt_key_delete()
      Set mtu_ignore field to default value
    * ospfd/ospf_abr.[ch]: export ospf_abr_announce_network_to_area()
    * ospfd/ospf_ism.h: add MACRO to convert internal ISM status into SNMP
      correct values
    * ospfd/ospf_snmp.c: add sanity check on LSA type in lsdb_lookup_next()
      convert OSPFIFSTATE internal status into SNMP values

2005-09-29 Alain Ritoux <alain.ritoux@6wind.com>

    * lib/filer.c: show protocol name in filter_show()
    * lib/plist.c: show protocol name in vty_show_prefix_entry()
    * routemap.c: show protocol name in vty_show_route_map_entry()
    * lib/vty.c: in vty_command(), show protocol name if command unknown

    * zebra/zserv.c: Always provide distance fo route add

    * ripd/rip_snmp.c: rip2IfConfReceive() sends values in conformance
      with RFC. Also PeerDomain is now set to a STRING type.
    * ripd/ripd.h: rip_redistribute_add() API includes metric and distance
    * ripd/ripd.c: rip_redistribute_add() API i.e. stores metric and distance
      Now allows a RIP-route to overcome a redistributed route coming
      from a protocol with worse (higher) administrative distance
      Metrics from redistribution are shown in show ip rip
    * ripd/rip_zebra.c: adapt to the rip_redistribute_add() API, i.e.
      provide distance and metric
    * ripd/rip_interface.c: adapt to the rip_redistribute_add() API
    * ripd/rip_routemap.c: no RMAP_COMPILE_ERROR on (metric > 16) usage
      rather a CMD_WARNING, because set metric ius shared with other
      protocols using larger values (such as OSPF)
      The match metric action takes first external metric if present
      (from redistribution) then RIP metric.

* *.c: Massive cleanup of lists loops. Stop abusing ALL_LIST_ELEMENTS.
  Replace XMALLOC + memset with XCALLOC. Fix some indentation issues.
  The only really significant change is simplified isis_delete_adj
  function in isis_adjacency.c.

* isis_dynh.c, isisd.h: Implement dynamic hostname cache cleanup.
* isis_lsp.c: Refresh dynamic hostname in the cache while refreshing
  topology LSP.

* isis_lsp.c: Make topology generator generate TE TLVs if necessary.
  Also take care of inserting dynamic hostname of topology TLVs into
  cache.

2005-09-28 Alain Ritoux <alain.ritoux@6wind.com>

    * lib/md5-gnu.h: removed
    * lib/md5.h: replaces md5-gnu.h
    * lib/Makefile.am: use correct md5.h
    * lib/md5.c: import from WIDE
    * ospfd/ospf_packet.c: use new md5 API
    * ripd/ripd.c: use new md5 API

* connected.c: flag connected_up_ipv6() and connected_down_ipv6()
      usage with HAVE_IPV6

* isis_spf.c: Changing cost from uint16_t to uint32_t. Unset
  ISIS_ROUTE_FLAG_ACTIVE flag before running SPF.
* isisd.[ch]: Separate route tables for different levels. SPF is done
  separately, but in case of L1L2 area they have to be merged.
* isis_zebra.c: Set/unset ISIS_ROUTE_FLAG_ZEBRA_SYNC flag correctly in
  case of adding/removing IPv4 routes.
* zebra_route.c: Rework route validating process. Merging L1 and L2
  tables in case of L1L2 area.

        In short - many changes to make SPF work more correctly, add/remove
        to/from RIB also works now. It's still very far from perfect though.

* isis_spf.[ch]: Added TE TLVs to the SPF process. It seems to work
  mostly, but needs certainly much more testing, especially transition
  situation.

* isis_lsp.c: Put correct metric info into TE TLV's. Wide metric is
  used only if only wide metric style (extended TLV's) is in use.

* isis_tlv.[ch]: New function tlv_add_in_addr() to put just one IPv4
  address into TLV. Used for IPv4 address TLV (in case of LSP) and TE
  router ID TLV.
* isis_lsp.c: Use tlv_add_in_addr() and include router ID in LSP.