| Age | Commit message (Collapse) | Author | 
|---|
|  | 2006-06-15 Paul Jakma <paul.jakma@sun.com>
	* (general) The key fixes are actually Andrew Schorr's.
	* interface.c: (ip_address_uninstall) Unset the configured flag.
	* connected.c: (connected_same) new helper, check whether
	  two connected are same.
	  (connected_implicit_withdraw) new helper, consolidation of
	  existing code in connected_add_ipv{4,6}.
	  Try filter out unneeded Zserv address delete/adds when
	  address is exact same.
	  Where old address is implicitely removed, be sure to preserve
	  the IFC_CONFIGURED flag if set, fixes bug where configured
	  addresses were being lost on FreeBSD. | 
|  | 2006-06-15 Paul Jakma <paul.jakma@sun.com>
	* lib/if.c: (if_flag_dump) remove the whitespace indentation, callers
	  should provide.
	* zebra/interface.c: (if_flag_dump_vty) redundant code, remove.
	  (if_dump_vty) use libzebra if_flag_dump. | 
|  | 2006-06-15 Paul Jakma <paul.jakma@sun.com>
	* linklist.c: (list_delete) call list_delete_all_node, don't
	  duplicate it. | 
|  | 2006-06-15 Paul Jakma <paul.jakma@sun.com>
	* memory.h: Experimental, have XFREE macro NULL out the freed
	  pointer. | 
|  | 2006-06-15 Paul Jakma <paul.jakma@sun.com>
	* command.c: (cmd_describe_command_real) Fix leak, CID #38. | 
|  | 2006-05-31 Paul Jakma <paul.jakma@sun.com>
	* ospf_lsdb.c: (ospf_lsdb_delete) robustify against NULL arguments,
	  print warning.
	* ospf_lsa.c: (ospf_discard_from_db) ditto.
	  (ospf_maxage_lsa_remover) Check lsa->lsdb for validity, possible
	  mitigation (but not solution) for bug #269. | 
|  | 2006-05-30 Paul Jakma <paul.jakma@sun.com>
	* ospf_packet.c: (ospf_read) Debug message about packets
	  received on unenabled interfaces should be conditional on
	  debug being set. | 
|  |  | 
|  | 2006-05-28 Paul Jakma <paul.jakma@sun.com>
	* test-sig.c: (main) configure logging, there's no terminal
	  to write to for one thing, but we do want stdout, and we
	  don't want a test to spam syslog. | 
|  | 2006-05-28 Paul Jakma <paul.jakma@sun.com>
	* configure.ac:
	  Check for Sun libc printstack(), add a general HAVE_STACK_TRACE
	  define for lib/log.c, if any supported stack symbol dumping
	  function is found (glibc backtrace/sun libc printstack).
	* log.c: (general) Add support for Sun libc printstack().
	  (hex_append) make the cpp conditional on general HAVE_STACK_TRACE
	  define.
	  (zlog_backtrace_sigsafe) Ditto. Add printstack() version of the
	  the DUMP macro in this function. | 
|  | 2006-05-28 Paul Jakma <paul.jakma@sun.com>
	* vty.c: (vty_log/vty_log_fixed) dont crash if called when vty
	  hasn't been initiliased. | 
|  | 2006-05-28 Paul Jakma <paul.jakma@sun.com>
	* memory.c: malloc.h is deprecated in favour of stdlib.h, however
	  we still need it on GNU Libc for mallinfo(). | 
|  | 2006-05-28 Paul Jakma <paul.jakma@sun.com>
	* zebra.h: Include inttypes.h rather than stdint.h, best practice
	  according to the autoconf manual.
	  Add UINT*_MAX defines for older platforms lacking these (FBSD 4) | 
|  | 2006-05-28 Paul Jakma <paul.jakma@sun.com>
	* configure.ac: Update SOS CFLAGS, Xt shouldn't be used and
	  enable debug options.
	  Add a check for GNU Make and warn the user if it does not appear
	  to be the make used. | 
|  | 2006-05-24 Paul Jakma <paul.jakma@sun.com>
	* vtysh.c: (general) Add 'show memory' command. | 
|  | 2006-05-23 Paul Jakma <paul.jakma@sun.com>
	* bgp_vty.c: (neighbor_update_source_cmd) Expand tab completion
	  to make it clear it takes both address and ifname. | 
|  | 2006-05-23 Paul Jakma <paul.jakma@sun.com>
	* bgpd.texi: Document the update-source command. | 
|  | 2006-05-21 Paul Jakma <paul.jakma@sun.com>
	* vtysh.c: (vtysh_show_logging_cmd) Suppress output for daemons
	  which are not running (Andrew Schorr). | 
|  | 2006-05-17 Paul Jakma <paul.jakma@sun.com>
	* vtysh.c: (general) Add 'show logging' command. | 
|  | 2006-05-21 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
	* if.h: (struct connected) Document the meaning of the
	  ZEBRA_IFC_REAL and ZEBRA_IFC_CONFIGURED flags.
	* connected.c: (connected_withdraw) Do not delete the connected
	  address if the ZEBRA_IFC_CONFIGURED flag is set.
	  (connected_add_ipv4,connected_add_ipv6) Before calling
	  connected_withdraw, unset the ZEBRA_IFC_CONFIGURED flag
	  on the superseded connected structure. | 
|  | 2006-05-19 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
	* connected.c: (connected_add_ipv4,connected_add_ipv6) If the
	  new struct connected matches an already existing one
	  (that will consequently be removed by connected_withdraw),
	  then be sure to preserve the ZEBRA_IFC_CONFIGURED flag. | 
|  | 2006-05-17 Andrew J. Schorr <ajschorr@alumni.princeton.edu>
	* kernel_socket.c: (ifam_read_mesg) Improve debug message
	  to show the IP address. | 
|  | 2006-05-15 Paul Jakma <paul.jakma@sun.com>
	* zserv.c: (general) Remove the private zebra_command_str
	  in favour of newly added libzebra zserv_command_string. | 
|  | 2006-05-15 Paul Jakma <paul.jakma@sun.com>
	* log.c: (general) Generalise struct zebra_route_desc into
	  struct zebra_desc_table and, similar to route_types, add a
	  command_types table to describe Zserv protocol commands.
	  (route_types[]) use a macro to use designated initialisers
	  while avoiding tedious duplication.
	  (zserv_command_string) lookup string from zebra_desc_table,
	  similar to zebra_route_string
	* zebra.h: Add declaration for zserv_command_string, adjust the
	  comments to reflect zebra_desc_table. | 
|  | 2006-05-13 Paul Jakma <paul.jakma@sun.com>
	* vty.c: (vty_describe_command) CID #39 fix was too hasty, just
	  cause it /can/ leak doesn't mean it always will have, check
	  first. | 
|  | 2006-03-01  Yasuhiro Ohara  <yasu@sfc.wide.ad.jp>
	* ospf6_asbr.c: fix bug of needless forwarding address
	* ospf6d.h: version 0.9.7r | 
|  | 2006-02-22  Yasuhiro Ohara  <yasu@sfc.wide.ad.jp>
	* valgrind check and memory fix
	* route table identification string added
	* ospf6d.h: version 0.9.7q | 
|  | 2005-10-20  Yasuhiro Ohara  <yasu@sfc.wide.ad.jp>
	* ospf6_neighbor.c: add the calling of ospf6_maxage_remove ()
	when the neighbor went out of Exchange/Loading. | 
|  | 2005-10-06  Yasuhiro Ohara  <yasu@sfc.wide.ad.jp>
	* ospf6_asbr.c: fix bug of calculating AS-External routes
	  in other areas. Add debugging function for border-router
	  calculation.
	* ospf6d.h: version 0.9.7p | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* bgp_aspath.{c,h}: (aspath_print_vty) take a format string,
	  so as to reduce burden on callers, all in bgp_route.c
	* bgp_route.c: (route_vty_out{,tmp}) Update to match
	  aspath_print_vty, simplifying checks needed to get spacing
	  right. CID #4,#5.
	  ({damp,flap}_route_vty_out) Ditto, CID #9, #10 | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* bgp_route.c: (bgp_static_update_rsclient) Remove useless NULL
	  check, code already assumes bgp_static can not be NULL,
	  fixes CID #6. | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* bgp_route.c: (bgp_static_update_main) Remove useless NULL
	  check, code already assumes bgp_static can not be NULL,
	  fixes CID #7. | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* bgp_attr.c: (bgp_packet_attribute) Remove dead code, Coverity
	  CID #1 | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* command.c: (cmd_describe_command_real) Fix return of freed
	  pointer when no-match, CID #55. | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* vty.c: (vty_describe_command) fix leak of describe vector in
	  error path, CID #39. | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* command.c: (cmd_complete_command_real) Fix leak of cmd_vector
	  in error case, Coverity CID #37. | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* plist.c: (vty_prefix_list_uninstall) Fix potential NULL deref
	  of prefix and typestr strings, Coverity CID #3. | 
|  | 2006-05-13 Paul Jakma <paul.jakma@sun.com>
	* ospf_lsa.c: (ospf_translated_nssa_refresh) fix the sanity
	  check to match the assert, small error in CID #13 fix. | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* ospf_lsa.c: (ospf_translated_nssa_refresh) Add non-assert
	  sanity check, in case DEBUG isn't defined.  Debug message
	  when no type7 exists should print the ID from the type5, not
	  the type7, fixes CID #13. | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* ospf_ism.c: (ism_change_state) NULL check on oi->area is
	  useless, it's always valid. Only possibility where it
 	  couldn't be is if there is a race between abr_task and
	  cleaning up oi's, in which case a NULL check here isn't going
	  to do anything. Fixes CID #15. | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* ospf_ia.c: (ospf_update_router_route) ospf->backbone could be
	  NULL when passed to ospf_find_asbr_route_through_area,
	  check for NULL first, CID #14. | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* ospf_asbr.c: (ospf_redistribute_withdraw) remove ospf_lookup
	  call by taking the struct ospf * as argument, which the
	  caller has, fixing CID #28.
	* ospf_asbr.h: (ospf_redistribute_withdraw) update declaration
	* ospf_zebra.c: (ospf_redistribute_unset) update call to
	  ospf_redistribute_withdraw to match. | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* ospf_interface.c: (ospf_if_exists) Fix missing NULL return
	  check on ospf_lookup, CID #27. | 
|  | 2006-05-12 Paul Jakma <paul.jakma@sun.com>
	* ospf_lsa.c: (ospf_lsa_action) Get rid of the ospf_lookup
	  call, which is not checked for NULL return, by stripping out
	  functionality which is never used, hence fixing Coverity CID
	  #29.
	  (struct lsa_action) remove unused member. | 
|  | 2006-05-11 Paul Jakma <paul.jakma@sun.com>
	* ospf_lsa.c: (ospf_default_originate_timer) Let the thread
	  take (struct ospf *) as thread argument, rather than (struct
	  ospf *)->default_originate, thus avoiding having to call
	  ospf_lookup.
	* ospf_zebra.c: (ospf_redistribute_default_set) change setup
	  of ospf_default_originate_timer thread to match.
	* ospfd.c: (ospf_router_id_update) ditto. | 
|  | 2006-05-11 Paul Jakma <paul.jakma@sun.com>
	* ospf_abr.c: (ospf_area_range_cost_set) Shouldn't create a new
	  range, should just lookup to see if one exists, the new range
	  is just leaked. Fixes CID #46. | 
|  | 2006-05-11 Paul Jakma <paul.jakma@sun.com>
	* ospf_route.c: (ospf_route_delete_same_ext) Fix deref before
	  NULL check by moving into check-protected block, fix CID #49. | 
|  | 2006-05-11 Paul Jakma <paul.jakma@sun.com>
	* ospf_vty.c: (no_ospf_area_filter_list) Check NULL result from
	  ospf_area_lookup_by_area_id, fixes Coverity CID #69 | 
|  | 2006-05-11 Paul Jakma <paul.jakma@sun.com>
	* ospf_vty.c: (general) Audit ospf_lookup calls in commands,
	  ensure check for NULL result, make vty messages consistent.
	  (show_ip_ospf_interface) Missing NULL check on ospf_lookup
	  result, fixes Coverity CID #70. | 
|  | 2006-05-11 Paul Jakma <paul.jakma@sun.com>
	* interface.c: (if_delete_update) route_node_lookup may return
	  NULL, should fix Coverity CID #31. |