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

* command.c: (install_element) be more robust. Eg, cmd_init
  need not have been called, some applications may use other
  library subsystems, which call install_element, without the
          application wanting commands and hence not calling cmd_init.

* *.c: Try to be less verbose by default (without any debug options
  on).
* isis_lsp.c (lsp_build_nonpseudo): Use stream_reset() instead of
  touching endp directly.
* isis_lsp.c (lsp_build_pseudo): Ditto.

* isis_lsp.c: Yet another regression introduced by stream cleanup.
  Similar fix to the one commited to lsp_build_nonpseudo() some days
  ago.

* isis_lsp.c: Less TLV leaking.

* random.c, spgrid.[ch]: Fix warnings with hope that I didn't broke
  anything. These floats to longs and vice versa casts are starnge
  indeed.
* isis_pdu.c: As we don't use %z for size_t for now because we support
  older compilers, cast them to unsigned long.

Also fix previous changelog entry. Isisd compiles cleanly now again.

* isis_adjacency.c, isis_lsp.c, isisd.c: Replace XMALLOC && memset
  with XCALLOC.
* isis_lsp.c (lsp_build_pseudo): Fix adding ES neighbour.
* isis_tlv.c: More compact free_tlvs() function.
* isis_lsp.c (lsp_build_nonpseudo) : Try to fix one more regression
  introduced by stream cleanup. Seek enp to the right place before
  starting to fill stream with TLVs.

* ripng_zebra.c, ripngd.[ch]: Pass metric info to the zebra daemon.

Forwardport from stable branch.

* zebra_rib.c, rib.h: Add distance and metric arguments to the
  rib_add_ipv6() function so that IPv6 routes in RIB can have correct
  metric. No IPv6 routing daemon uses distance yet though.
* zserv.c, connected.c, kernel_socket.c, rt_netlink.c,
  rtread_proc.c,zserv.c: Pass metric and distance info to the
  rib_add_ipv6().

Forwardport from stable branch.

* bgp_route.c: Third (?) attempt to fix best selection breakage
  introduced long time ago with route server patch. Hopefully
  it's last case to fix - route-server client not in peer group.

[backport candidate]

2005-08-25 Paul Jakma <paul@jakma.org>

* configure.ac: Add -fno-omit-frame-pointer after -Os in default
  cflags, just to be sure.
  Fedora's readline library does not itself link to termcap, hence
  we must pass the result of termcap tests in via OTHER-LIBRARIES
  argument, otherwise the test of main in readline will fail due to
  missing termcap systems. On systems like Debian, -ltermcap
  is not needed for the readline test, because libreadline already
  links to it.

* configure.ac, vtysh/Makefile.am: Only vtysh needs to be linked
  against libreadline and friends.

2005-08-22 Hugo Santos <hsantos@av.it.pt>

* vtysh.c: Add support for BGP_IPV6M_NODE

2005-08-22 Hugo Santos <hsantos@av.it.pt>

* bgp_vty.c: (general) Add support for BGP IPv6 Multicast SAFI
  commands and BGP_IPV6M_NODE.

2005-08-22 Hugo Santos <hsantos@av.it.pt>

* command.h: (enum node_type) Add BGP_IPV6M_NODE
* command.c: (node_parent) Handle BGP_IPV6M_NODE node
  (config_exit, config_end) ditto
* vty.c: (vty_end_config) Handle BGP_IPV6M_NODE node

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

* bgp_route.h: (struct bgp_info) add a new flag, BGP_INFO_REMOVED.
  BGP_INFO_VALID is already overloaded, don't care to do same thing
  to STALE or HISTORY.
* bgpd.h: (BGP_INFO_HOLDDOWN) Add INFO_REMOVED to the macro, as a
  route which should generally be ignored.
* bgp_route.c: (bgp_info_delete) Just set the REMOVE flag, rather
  than doing actual work, so that bgp_process (called directly,
  or indirectly via the scanner) can catch withdrawn routes.
  (bgp_info_reap) Actually remove the route, what bgp_info_delete
  used to do, only for use by bgp_process.
  (bgp_best_selection) reap any REMOVED routes, other than the old
  selected route.
  (bgp_process_rsclient) reap the old-selected route, if appropriate
  (bgp_process_main) ditto
  (bgp_rib_withdraw, bgp_rib_remove) make them more consistent with
  each other. Don't play games with the VALID flag, bgp_process
  is async now, so it didn't make a difference anyway.
  Remove the 'force' argument from bgp_rib_withdraw, withdraw+force
  is equivalent to bgp_rib_remove. Update all its callers.
  (bgp_update_rsclient) bgp_rib_withdraw and force set is same as
  bgp_rib_remove.
  (route_vty_short_status_out) new helper to print the leading
  route-status string used in many command outputs. Consolidate.
  (route_vty_out, route_vty_out_tag, damp_route_vty_out,
   flap_route_vty_out) use route_vty_short_status_out rather than
  duplicate.
  (route_vty_out_detail) print state of REMOVED flag.
  (BGP_SHOW_SCODE_HEADER) update for Removed flag.

* ospf_vty.c: Make "show ip ospf neighbor xxx" commands work.
  Interface should be specified by name now.

[backport candidate]

* ospf_vty.c: Check carefully if interface exists before trying to
  print info about it.

Fixes bugzilla #213. [backport candidate]

* isis_misc.[ch]: Fix some warnings, making some strings const.

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

* README.txt: point to the opensolaris.org sources for the SMF
  manifest class scripts. Make step 3 slightly more explicit about
  what 'this directory' is.

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

* ripd.c: (general) Fix previous commit, broke multicast bind and
  hence setting of source port, which broke communication with
  non-borken ripd. Fix removes more stuff from rip_interface.c
  than it adds to ripd.c ;)
  (rip_create_socket) the to argument really is a from argument,
  rename it. Set the source port to RIP port unconditionally, it's
  required.
  (rip_send_packet) Set from address correctly for multicast.
  (rip_output_process) trivial: num can be BSS specified, rather
  than in body.
* rip_interface.c: (rip_interface_multicast_set) strip out
  redundant stuff related to bind, which rip_create_socket does.
  Just make it set the multicast socket option, as per the
  interface concerned, no more.

2005-08-13 Paul Jakma <paul@jakma.org>

* Makefile.am: (EXTRA_DIST) Add the trailing slash back in which
  greg left out - tools bits weren't being included in dist,
  which broke rpm builds :).

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

* ospf6_asbr.c: (ospf6_asbr_redistribute_add) Fix sense of testing
  of route_map_apply return code, it can return many things other
  than RMAP_MATCH which do not indicate DENY. Should test explicitly
  for equality to RMAP_DENYMATCH instead.

[backport candidate]

add INSTALL.quagga.txt to the distribution, so that people who apply
patches and lose because of old autoconf/automake versions have the
opportunity to find tout what to do.

* topology/spgrid.c: MAXLONG is deprecated, use LONG_MAX instead.
  values.h, where MAXLONG is defined, is deprecated as well.

Thanks to Greg for noticing and to Rivo for fix.

rework

clarify make (BSD make not working is a bug, and GNU make not working
is a really serious bug)

declare POSIX.2 as a semi-requirement, with notion that workarounds
for non-posix must be clean.

list OS versions for which not working is a bug.

add instructions for using info and emacs/info, and remove admonition
to use pinfo

add note about GNU awk (really needed?  BSD awk)

Note that NetBSD 2.99.15 has texinfo 4.7

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

        * getopt.h: Don't declare getopt (rather than getopt_long), since
        quagga doesn't need it.
        * getopt.c (getopt): Don't define getopt.

Fixes build breakage on NetBSD, and seems likely to work on most
platforms since it avoids the entire issue of system getopt
declarations and whether they conform to POSIX.2.  Note that this
change doesn't address system getopt_long declarations, but also
doesn't change anything about getopt_long.

* ospf_zebra.c: Don't assert/stop before type == ZEBRA_ROUTE_MAX if
  dealing with routemaps. There is ospf->route_map[ZEBRA_ROUTE_MAX]
  for default-information.

Fixes bugzilla #208.
[backport candidate]

* bgp_routemap.c: Revert part of leaking communities fix commited in
  2005-05-27. While ecommunity fix seems to be correct, community case
  isn't.

Fixes bugzilla #209.
[backport candidate]

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

* interface.c: (if_delete_update) should always be available, not
  just on RTM_IFANNOUNCE/NETLINK systems.
* kernel_socket.c: (ifan_read) only call if_delete_update when
  interface departs, dont if_delete, because we wish to retain
  interface configuration state even when interfaces are removed.
  (ifm_read) If we dont have RTM_IFANNOUNCE, then transitioning
  to down state is only chance we have to clean up interface in case
  it is deleted (eg Solaris down -> unplumb -> plumb up).
* redistribute.c: (zebra_interface_delete_update) should always be
  available, we /will/ call it now on all systems, via
  if_delete_update.
* zserv.c: (zsend_interface_delete) ditto
  (zsend_interface_address) Update the call-flow diagramme, to
  reflect that if_delete_update /is/ now called on all systems,
  potentially.
* zserv.h: (zsend_interface_delete) unconditionally exported, as
  above.

2005-07-26 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
* prefix.c: (prefix_ipv4_new, prefix_ipv6_new): Call prefix_new
  to allocate the memory to make sure that all struct prefix pointers
  point to objects of the same length (avoids memory overruns
  on struct prefix assignments).
  (prefix_ipv4_free, prefix_ipv6_free): Simply call prefix_free.
  It is interesting to note that these functions are never actually
  called anywhere in the code.  Instead prefix_free was already
  being called directly, despite the previous MTYPE incompatibility.

[backport candidate]

2005-07-26 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* prefix.c: (ip_masklen) While loop should test that 'pnt' pointer is
  in range before dereferencing it.

[backport candidate]

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

* ospf_abr.c: (ospf_abr_announce_network_to_area) SET_FLAG
  should be on lsa not old, which may be freed for one thing,
  obviously.

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

* ospfd.h: add OSPF_ABR_DEFAULT for convenience, make
  OSPF_ABR_CISCO be the default ABR type.
* ospfd.c: (ospf_new) initialise abr_type to OSPF_ABR_DEFAULT
* ospf_vty.c: (no_ospf_abr_type_cmd) add standard as a negatable
  abr_type. default abr_type should be OSPF_ABR_DEFAULT.
  (ospf_config_write) test whether default abr_type against
  OSPF_ABR_DEFAULT, rather than any specific ABR_TYPE.

- Fix ChangeLog entry to properly refer to problem being Linux/ARM, not
  ARM in general.

2005-07-12 Christian Hammers <ch@debian.org>

* vtysh_user.c: rename struct user to struct vtysh_user to avoid
  clashes with sys/user.h on ARM.

* configure.ac: Actually test whether libc has IPv6 support.

[backport candidate]

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

* kernel_socket.c: (RTA_{ADDR,ATTR}_GET) fix mistake, NULL check
  should be on DEST argument

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

* (global) Extern and static'ification, with related fixups
  of declarations, ensuring files include their own headers, etc.
  if_ioctl.c: (interface_info_ioctl) fix obvious arg mis-order in
  list loop

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

* (global) The great bgpd extern and static'ification.
* bgp_routemap.c: remove unused ROUTE_MATCH_ASPATH_OLD code
  (route_set_metric_compile) fix u_int32_t to ULONG_MAX comparison
  warnings.
* bgp_route.h: (bgp_process, bgp_withdraw, bgp_update) export these
  used by various files which had their own private declarations,
  in the case of mplsvpn - incorrect.

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

* INSTALL.quagga.txt: GNU make is required now, because of manual
  automatic rules in solaris/Makefile.am. (If someone knows how
  to do these in a better way..).
  GNU AWK is required for CVS checkout builds.

* ospf6_abr.[ch], ospf6_area.[ch]: Add area filter-list (in|out)
  support and area import and export lists support.

* ospf6_message.c: Changed to be insensitive to changes of neighbors'
  IP source address in the OSPF packets. It was sometimes problematic
  in actual operation (needed some operational cost: restarting
  all-neighbor routers when I/F NIC was changed). Due to this change,
  a previously safe case, attaching multiple interface to the same
  link will now be dengerous and will not work. Remedy to that should
  be applied later.

[port from GNU Zebra]

* ospf6_interface.c: fix the way inactivity_timer is called. Because
  inactivity_timer() deletes the neighbor from the neighbor_list, it
  cannot be called by thread_execute() from inner side of the
  neighbor_list for-loop.
  (Although crash was already fixed in Quagga, it's better follow the
  GNU Zebra logic).

[port from GNU Zebra]

2005-06-24 Pawel Worach <pawel.worach@gmail.com>

* getopt.h: add further tests for full getopt declaration on
  various systems.

Fix date.

* ospf_nsm.c: Make database exchange for NSSA database work.

[backport candidate]

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

* memtypes.h: update autobuilt file to match memtypes.c changes

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

* bgpd.c: (bgp_terminate) workqueue's are lazy allocated and its
  possible to terminate bgpd before workqueues were setup, causing
  an abort/crash. Reported by Ashish Mehta of Sun.

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

* README.txt: updated to match changed SMF manifest and
  its changed service, instance and property group names.

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

* quagga.xml.in: Pass the FMRI to the method, removing need for
  daemon_name property and deducing the FMRI.
  remove the empty config_file and pid_file definitions, they just
  make things difficult in the method script, cause it then has to
  deal with svcprop returning "" for defined empty properties.
  Remove daemon_name everywhere.
* quagga.init.in: Take the FMRI as an argument
  Remove deducing the FMRI from the defunct daemon_name property.
  Use svcprop -q to test for presence of a property first.
  Default vty_port to 0 and vty_addr to 127.1 if equivalent
  SMF properties are not set.
  Deduce the pid_file, we can infer it from @quagga_statedir@ I
  think, it's probably not useful to allow it to be configurable
  as a property anyway.
  /var/run/ is on tmpfs on Solaris, so we probably will need
  to create @quagga_statedir@ first run after boot.
  Use @sbindir@, not /usr/local/sbin.

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

* kernel_socket.c: consolidate the IFAM{ADDR,MASK}GET and
  RTM{ADDR,MASK}GET macros into generic rta_addrs macros,
  RTA_{ADDR,ATTR}_GET.
  (af_check) could use 'inline' attribute
  (ifam_read_mesg) remove IFAM{ADDR,MASK}GET macro, change to
  generic macro.
  (rtm_read_mesg) similar

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

* ospf_spf.c: Try get more information on a SEGV under
  ospf_spf_vertex_add_parent.
  (ospf_vertex_free) NULL out the child and nexthop lists
  (ospf_vertex_add_parent) nexthop and child can not be NULL
  vertex_nexthop's parent->child list can not be NULL
  (ospf_spf_next) w and cw are per-loop iteration variables, move
  declarations into loop body.

* rt_netlink.c: Remove unused netlink-addr socket declaration.
* rt_netlink.c (netlink_parse_info): Fix debug messages - nlmsg_pid is
  unsigned and one zlog call had swapped arguments.
* rt_netlink.c (netlink_route_multipath): Fix compile with disabled
  IPv6 support.

[backport candidate] - with stuff commited to rt_netlink.c before to
fix logging in netlink_route_multipath().

* ospf_apiserver.c: Fix obvious error in notifying clients about ISM
  changes - oi->ifp->status doesn't give to us info about ISM,
  oi->state does.

[backport candidate]

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

* ripd.c: (rip_create_socket) Make it static.
  Remove the getservbyname stuff, as RFC2453 3.9.2 says non-RIP
  port messages should be discarded, quagga doesnt accept them,
  no need to lookup port.
  Take a 'to' argument, if socket should be bound to something else.
  setsockopt_so_recvbuf might need privs, move it to the raised
  privileges section.
  dont forget to close the socket if bind fails.
  (rip_send_packet) use strncpy, just in case (address is under
  our control anyway, but still).
  dont duplicate rip_create_socket - just use it.
  (rip_create) rip_create_socket takes an argument now, modify.

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

* ripd.c: (rip_create_socket) move it up so rip_send_packet
  can use it too.

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

* quagga.xml.in: Fix mistake in bgpd definition. Change dependency
  on zebra to optional. Remove the duplicated stability statements.
* quagga.init.in: svcprop should check PACKAGE_TARNAME, not _NAME.
  no need to check for config file either, there's already a
  dependency in the manifest.

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

* NEWS: bgpd work queues and ripd auth-mode change

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

* rip_interface.c: Fix authentication, no-auth impossible to specify
  (rip_interface_new) default to RIP_NO_AUTH
  (rip_interface_reset) ditto
  (rip_interface_config_write) write out config for simple

2005-06-01  Akihiro Mizutani  <mizutani@net-chef.net>

* ospf_ism.c (ospf_elect_bdr/ospf_elect_dr): Fix DR election bug.

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

* bgpd/(general) refcount struct peer and bgp_info, hence allowing us
  add work_queues for bgp_process.
* bgpd/bgp_route.h: (struct bgp_info) Add 'lock' field for refcount.
  Add bgp_info_{lock,unlock} helper functions.
  Add bgp_info_{add,delete} helpers, to remove need for
  users managing locking/freeing of bgp_info and bgp_node's.
* bgpd/bgp_table.h: (struct bgp_node) Add a flags field, and
  BGP_NODE_PROCESS_SCHEDULED to merge redundant processing of
  nodes.
* bgpd/bgp_fsm.h: Make the ON/OFF/ADD/REMOVE macros lock and unlock
  peer reference as appropriate.
* bgpd/bgp_damp.c: Remove its internal prototypes for
  bgp_info_delete/free. Just use bgp_info_delete.
* bgpd/bgpd.h: (struct bgp_master) Add work_queue pointers.
  (struct peer) Add reference count 'lock'
  (peer_lock,peer_unlock) New helpers to take/release reference
  on struct peer.
* bgpd/bgp_advertise.c: (general) Add peer and bgp_info refcounting
  and balance how references are taken and released.
  (bgp_advertise_free) release bgp_info reference, if appropriate
  (bgp_adj_out_free) unlock peer
  (bgp_advertise_clean) leave the adv references alone, or else
  call bgp_advertise_free cant unlock them.
  (bgp_adj_out_set) lock the peer on new adj's, leave the reference
  alone otherwise. lock the new bgp_info reference.
  (bgp_adj_in_set) lock the peer reference
  (bgp_adj_in_remove) and unlock it here
  (bgp_sync_delete) make hash_free on peer conditional, just in
  case.
* bgpd/bgp_fsm.c: (general) document that the timers depend on
  bgp_event to release a peer reference.
  (bgp_fsm_change_status) moved up the file, unchanged.
  (bgp_stop) Decrement peer lock as many times as cancel_event
  canceled - shouldnt be needed but just in case.
  stream_fifo_clean of obuf made conditional, just in case.
  (bgp_event) always unlock the peer, regardless of return value
  of bgp_fsm_change_status.
* bgpd/bgp_packet.c: (general) change several bgp_stop's to BGP_EVENT's.
  (bgp_read) Add a mysterious extra peer_unlock for ACCEPT_PEERs
  along with a comment on it.
* bgpd/bgp_route.c: (general) Add refcounting of bgp_info, cleanup
  some of the resource management around bgp_info. Refcount peer.
  Add workqueues for bgp_process and clear_table.
  (bgp_info_new) make static
  (bgp_info_free) Ditto, and unlock the peer reference.
  (bgp_info_lock,bgp_info_unlock) new exported functions
  (bgp_info_add) Add a bgp_info to a bgp_node in correct fashion,
  taking care of reference counts.
  (bgp_info_delete) do the opposite of bgp_info_add.
  (bgp_process_rsclient) Converted into a work_queue work function.
  (bgp_process_main) ditto.
  (bgp_processq_del) process work queue item deconstructor
  (bgp_process_queue_init) process work queue init
  (bgp_process) call init function if required, set up queue item
  and add to queue, rather than calling process functions directly.
  (bgp_rib_remove) let bgp_info_delete manage bgp_info refcounts
  (bgp_rib_withdraw) ditto
  (bgp_update_rsclient) let bgp_info_add manage refcounts
  (bgp_update_main) ditto
  (bgp_clear_route_node) clear_node_queue work function, does
  per-node aspects of what bgp_clear_route_table did previously
  (bgp_clear_node_queue_del) clear_node_queue item delete function
  (bgp_clear_node_complete) clear_node_queue completion function,
  it unplugs the process queues, which have to be blocked while
  clear_node_queue is being processed to prevent a race.
  (bgp_clear_node_queue_init) init function for clear_node_queue
  work queues
  (bgp_clear_route_table) Sets up items onto a workqueue now, rather
  than clearing each node directly. Plugs both process queues to
  avoid potential race.
  (bgp_static_withdraw_rsclient) let bgp_info_{add,delete} manage
  bgp_info refcounts.
  (bgp_static_update_rsclient) ditto
  (bgp_static_update_main) ditto
  (bgp_static_update_vpnv4) ditto, remove unneeded cast.
  (bgp_static_withdraw) see bgp_static_withdraw_rsclient
  (bgp_static_withdraw_vpnv4) ditto
  (bgp_aggregate_{route,add,delete}) ditto
  (bgp_redistribute_{add,delete,withdraw}) ditto
* bgpd/bgp_vty.c: (peer_rsclient_set_vty) lock rsclient list peer
  reference
  (peer_rsclient_unset_vty) ditto, but unlock same reference
* bgpd/bgpd.c: (peer_free) handle frees of info to be kept for lifetime
  of struct peer.
  (peer_lock,peer_unlock) peer refcount helpers
  (peer_new) add initial refcounts
  (peer_create,peer_create_accept) lock peer as appropriate
  (peer_delete) unlock as appropriate, move out some free's to
  peer_free.
  (peer_group_bind,peer_group_unbind) peer refcounting as
  appropriate.
  (bgp_create) check CALLOC return value.
  (bgp_terminate) free workqueues too.
* lib/memtypes.c: Add MTYPE_BGP_PROCESS_QUEUE and
  MTYPE_BGP_CLEAR_NODE_QUEUE

* ospf6d.c: No need for double ';'. Fixes parsing "show ipv6 ospf6
  database ..." commands for vtysh by extract.pl. Remove duplicate
  install_element calls.

Fixes bugzilla #194. [backport candidate]

Ignore generated prototype.smf.

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

* zserv.c: (zsend_route_multipath) Fix bug if route is sent
  with no NEXTHOP_FLAG_FIB nexthops. As ZAPI_MESSAGE_IFINDEX
  and ZAPI_MESSAGE_NEXTHOP are always set, clients would try
  read non-existent nexthop information and hit stream assert.
  Zserv is still broken for multi-nexthop messages, but it always was.

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

* ripd.c: (rip_output_process) fix error which crept in my
  previous rip auth untanglement commit - it had become impossible
  to not have authentication (even for v1).

Sync bgpd and ripd set_metric_addsub_cmd commands. Fixes bugzilla #192.
[backport candidate]

* bgp_routemap.c: Stop leaking communities.

Fixes bugzilla #89. [backport candidate]

* bgpd.c: Deleting bgp->rsclient list needs fix similar to pree-groups
  deleting fix. Avoid leaking bgp->group, bgp->peer and bgp->rsclient
  lists.

[backport candidate]

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

* ospf_abr.c: (ospf_abr_update_aggregate) Fix comment, cost bug itself
  had been fixed long ago by Sowmini.

* isis_dr.c: Fix copy&paste error in isis_dr_resign().

[backport candidate]

* bgpd.c: Don't crash while deleting list of peer-groups.

[backport candidate]

* rip_routemap.c: In case of '0.0.0.0' used as 'nexthop', use sender
  address as nexthop in routemap.

Fixes bugzilla #186. [backport candidate]

* rip_routemap.c: Make "match interface" routemap command match both -
  in and out interfaces.

Fixes bugzilla #185. [backport candidate]

* bgpd.c: Fix obvious (routeserver patch) merge error. This makes "no
  neighbor x.x.x.x routemap [export|import] commands work again.

Fixes bugzilla #184. [backport candidate]

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

* isisd.c: (show_isis_generated_topology) change to _RO version
  of list macro. remove the extra listnode variable. one of the
  macros had had incorrect number of arguments.

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

* memtypes.h: update this auto-built file. (maybe we should just
  remove it, is GNU awk a terrible dependency to have?)

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

* bgp_fsm.h: Add extern qualifier to exported functions
* bgp_nexthop.c: add static to nexthop specific globals
* *.h: Add guard defines

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

* workqueue.h: Add a WQ_QUEUE_BLOCKED item_status return code,
  to allow a queue function to indicate the queue is not
  ready/blocked - rather than any problem with the item at hand.
  Add a notion of being able to 'plug' and 'unplug' a queue.
  Add helpers to plug/unplug a queue.
  Add a completion callback, to be called when a queue is emptied.
* workqueue.c: (work_queue_new) remove useless list_free.
  (work_queue_schedule) new internal helper function to schedule
  queue, if appropriate.
  (work_queue_add) use work_queue_schedule
  (show_work_queues) Print 'P' if queue is plugged.
  (work_queue_plug) new API function, plug a queue - ie prevent it
  from 'drained' / processed / scheduled.
  (work_queue_unplug) unplug a queue, allowing it to be drained
  / scheduled / processed again.
  (work_queue_run) Add support for WQ_QUEUE_BLOCKED.
  Add comment for RETRY_NOW case.
  Make hysteris more aggresive in ramping up granularity, improves
  performance significantly.
  Add support for calling completion callback when queue is emptied,
  possibly useful for knowing when to unplug a queue.

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

* bgp_routemap.c: add semi-colons to VTY_GET_* to match vty.h change

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

* routemap.c: (rmap_onmatch_goto) fix crash if 'continue' command
  is used, which does not supply an argv[0].
  this is a backport candidate /iff/ the trailing ; is removed
  from VTY_GET_INTEGER_RANGE
* vty.h: fix the VTY_GET macros, do {..} while(0) so they have
  correct function like syntax in usage.

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

* memtypes.awk: use character classes, which work correctly in
  all LC_COLLATE environments, unlike A-Z, which doesnt work in
  eg estonian collate order. Reported by Hasso.

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

* memtypes.c: (memory_list_bgp) add MTYPE_BGP_PEER_HOST

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

* bgp_network.c: (bgp_accept) use XSTRDUP
* bgpd.c: (peer_delete) XFREE the correct memtype, not free.
  (peer_create) use XSTRDUP
* bgp_packet.c: (bgp_stream_dup) deleted, stream_dup should be used
  (various) update -> s/bgp_stream_dup/stream_dup

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

* bgp_fsm.c: (bgp_stop) use sockunion_free, not XFREE..
* bgp_network.c: (bgp_getsockname) ditto
* bgp_routemap.c: (route_match_peer) ditto, als use a ret value and
  remove one sockunion_free.
* bgpd.c: (peer_delete) ditto

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

* sockunion.c: (sockunion_getsockname) use MTYPE_SOCKUNION, not TMP
  (sockunion_getpeername) ditto

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

* thread.c: (thread_cancel_event) the number of pending events
  cancelled is potentially useful information, dont throw it away,
  pass it back to the caller.

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

* ospf_interface.c: (ospf_if_table_lookup) Fix a serious bug
  a less serious one.
  1: this function is supposed to lookup
  entries in the oifs ospf_interface route_table and return either
  an existing oi or NULL to indicate not found, its caller depends
  on this, yet this function uses route_node_get which /always/
  returns a route_node - one is created if none exists. Use
  route_node_lookup instead. This should fix root cause of the
  reports of the (ospf_add_to_if) assert being hit.
  2: oi's are inserted into this table with prefixlength set to
  /32 (indeed, it should be a hash table, not a route_table),
  however prefixlength to lookup was not changed, if no valid entry
  can be inserted other than /32, then nothng but /32 should be
  looked up. This possibly only worked by fluke..
  Fix confirmed by 2 reporters (one list, one IRC), definitely a
  backport candidate once it has been incubated in HEAD for a while.
  Thanks to Patrick Friedel and Ivan Warren for testing.

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

* getopt.h: It's not just __GNU_LIBRARY__ which defines
  getopt, eg __EXTENSIONS__ does too on SunOS. It still seems
  awfully fragile though.
* getopt.c: include zebra.h after config.h, before including
  getopt.h so that things at least are consistent..
* getopt1.c: ditto

- revert the commenting of touch doc/quagga.info, wont work because
  defines.texi is autogenerated every configure time so it will always
  be newer than quagga.info, and hence require quagga.info to be built. drat.

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

* prototype.daemons.in: Move the Solaris 10 SMF specific stuff to
  a seperate package.
* depend.smf: dependencies for smf package
* pkginfo.smf.tmpl.in: pkginfo for smf
* prototype.smf.in: prototype for smf
* Makefile.am: Add smf to pkg_names

- Forgot to commit ChangeLog

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

* quagga.xml.in: Sigh, XML child elements are order-sensitive.
* prototype.doc.in: info/dir doesnt get created always.. comment
  out for now, likely needs to be done with an install method.

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

* configure.ac: Comment out the touch doc/quagga.info hack. I'm
  hoping it's not needed anymore, cause it breaks out of tree builds
  which i've become fond of.. Plus I think we sorted out most of
  the original problems (updated texinfo.tex, moved the version
  info to an autogenerated by automake version.texi). Dist users
  most definitely wont be affected.

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

* (general) Fix memory leaks in opaque AS-scope LSAs, reported and
  with much debugging done by by scott collins <scollins@agile.tv>.
  (possible backport candidate?)
* ospf_lsa.c: (ospf_discard_from_db) dont call
  ospf_ase_unregister_external_lsa for opaque-lsa's, opaques are
  never registered with ase in the first place.
* ospf_packet.c: (general) Disabuse opaque related code of its
          tendency to try gather up things into temporary lists.
          (ospf_ls_upd) remove the temporary lists opaque uses, call
          opaque functions inline, just like all other types.
          (ospf_ls_ack) ditto.
          (ospf_recv_packet) fixup sign warning.
        * ospf_opaque.c: (general) fix the unneeded use of lists, and
          untwist some of the logic.
          (ospf_opaque_self_originated_lsa_received) take a single LSA
          as argument, not a list of them. Remove the list loop. Logic
          otherwise unchanged.
          (ospf_opaque_ls_ack_received) Mostly ditto. But untwist the logic,
          move the actions up into the switch block, remove the goto's and
          sanitise the logic near the end a bit.
        * ospf_opaque.h: Adjust definitions of aforementioned functions
          in ospf_opaque.c to match.

2005-05-11 Andrew J. Schorr <ajschorr@alumni.princeton.edu>

* rip_interface.c (rip_interface_add): Need to call
  rip_passive_interface_apply (was already calling it in
  rip_interface_up).

[backport candidate]

2005-05-07 Yar Tikhiy <yar@comp.chem.msu.su>

* configure.ac: Check for OSes which support passing ifindex in
  struct ip_mreq.
* lib/sockopt.c: Add support for BSD style ifindex in ip_mreq.
* ospfd/ospf_network.c: Log ifindex on multicast membership leave/join
  events.

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

        * (general) extern and static qualifiers added.
          unspecified arguments in definitions fixed, typically they should
          be 'void'.
          function casts added for callbacks.
          Guards added to headers which lacked them.
          Proper headers included rather than relying on incomplete
          definitions.
          gcc noreturn function attribute where appropriate.
        * ospf_opaque.c: remove the private definition of ospf_lsa's
          ospf_lsa_refresh_delay.
        * ospf_lsa.h: export ospf_lsa_refresh_delay
        * ospf_packet.c: (ospf_make_md5_digest) make *auth_key const,
          correct thing to do - removes need for the casts later.
        * ospf_vty.c: Use vty.h's VTY_GET_INTEGER rather than ospf_vty's
          home-brewed versions, shuts up several warnings.
        * ospf_vty.h: remove VTY_GET_UINT32. VTY_GET_IPV4_ADDRESS and
          VTY_GET_IPV4_PREFIX moved to lib/vty.h.
        * ospf_zebra.c: (ospf_distribute_list_update_timer) hacky
          overloading of the THREAD_ARG pointer should at least use
          uintptr_t.

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

* zserv.h: Remove ZEBRA_PORT definition, its in lib/zebra.h now

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

* (general) extern and static'ification of functions in code and
  header.
  Cleanup any definitions with unspecified arguments.
  Add casts for callback assignments where the callback is defined,
  typically, as passing void *, but the function being assigned has
  some other pointer type defined as its argument, as gcc complains
  about casts from void * to X* via function arguments.
  Fix some old K&R style function argument definitions.
  Add noreturn gcc attribute to some functions, as appropriate.
  Add unused gcc attribute to some functions (eg ones meant to help
  while debugging)
  Add guard defines to headers which were missing them.
* command.c: (install_node) add const qualifier, still doesnt shut
  up the warning though, because of the double pointer.
  (cmp_node) ditto
* keychain.c: (key_str2time) Add GET_LONG_RANGE() macro, derived
  fromn vty.h ones to fix some of the (long) < 0 warnings.
* thread.c: (various) use thread_empty
  (cpu_record_hash_key) should cast to uintptr_t, a stdint.h type
* vty.h: Add VTY_GET_IPV4_ADDRESS and VTY_GET_IPV4_PREFIX so they
  removed from ospfd/ospf_vty.h
* zebra.h: Move definition of ZEBRA_PORT to here, to remove
  dependence of lib on zebra/zserv.h

* sockunion.c: Fix warning message.