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_...

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

* sockopt.{c,h}: add sockopt_iphdrincl_swab_{htosys,systoh},
          functions to change byte order between system IP_HDRINCL order
          and host order.

Even my time is limited :).

Remove dead files.

First small part of lib cleanup. Mainly "constification" of arguments and
adding FIXME's.

Every file includes it and warns about it.

Document stuff I broke today.

Big vtysh cleanup. See changelogs for details.

New way to handle secondary addresses from Gilad Arnold.

Common router id.

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

* Update default CFLAGS for gcc to keep Hasso busy.

* Wed Sep 15 2004 Paul Jakma <paul@dishone.st>
- build snmp support by default
- build irdp support
- build with shared libs
- devel subpackage for archives and headers

Merge svn revision 1048 from Zebra cvs. Also add missing sokopt.h include
to ospf6_network.h.

I think that it's better to report bugs into Bugzilla.

CVS ignores symlinks, but I recently updated update-autotools to have
libtool copy files rather than symlink. Ignore these now copied files.

This code didn't survive valgrinding. I wonder how it survived normal run.

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

* ospf_dump.c: (ospf_ip_header_dump) Use HAVE_IP_HDRINCL_BSD_ORDER
  Apply to offset too. Print ip_cksum, lets not worry about
  possible 2.0.37 compile problems.
* ospf_packet.c: (ospf_swap_iph_to{n,h}) Use
  HAVE_IP_HDRINCL_BSD_ORDER.
  (ospf_recv_packet) ditto.
  (ospf_write) Fixup iov argument to ospf_write_frags.

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

* zebra.h: Add WANT_OSPF_WRITE_FRAGMENT for ospfd
  to try to fragment oversized packets. Enabled only for Linux.
  Add HAVE_IP_HDRINCL_BSD_ORDER to define struct ip byte order,
  to consolidate various ad-hoc platform defines for same thing.

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

* update-autotools: libtoolize should copy files, rather than link.
  the whole idea is that dist files should not need auto*, etc.
  installed to be able to compile.

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

* ospf_packet.c: (ospf_write) (struct msghdr).msg_name is caddr_t
  on most platforms.
  (ospf_recv_packet) ditto. And msg_flags is not always there
          memset struct then set fields we care about rather than
          initialise all fields individually.

Brown paperbag please. Fix previous commit.

Compiler warning fix.

Compiler warnings fixes.

Fix compiler warning.

Compiler warnings fixes.

Compiler warnings fixes.

Compile with gcc-4.0.

* isisd.c: thread_master *master is already defined in isis_main.c.
* isis_misc.[c|h], isis_lsp.[c|h]: Move static variables out of header
  files.

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

        * irdp_interface.c: (no_ip_irdp_address_preference_cmd)
          add missing listnode declaration.

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

        * irdp_{interface,main}.c: lists typedef removal cleanup.
          update some list loops to LIST_LOOP. some miscellaneous style
          and indent fixups.
          (no_ip_irdp_address_preference_cmd) Fix delete of referenced node
          in loop.
        * irdp_packet.c: (irdp_recvmsg) Fix buggy assignment of integer
          to pointer.
        * if_ioctl{,_solaris}.c: lists typedef removal cleanup.
          update some list loops to LIST_LOOP.

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

        * ospf_apiserver.{c,h}: (ospf_apiserver_term) loops calling
          ospf_apiserver_free, which deletes referenced nodes from
          apiserver_list, fixed.

forgot to commit changelog..

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

* ospf_packet.c: (ospf_write) ifdef fragmentation support.
          move actual fragmentation out to a new, similarly ifdefed, function.
  (ospf_write_frags) fragmented write support, moved from previous.

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

        * ospf_apiserver.{c,h}: lists typedef removal cleanup.
          update some list loops to LIST_LOOP. some miscellaneous indent
          fixups.
          (ospf_apiserver_unregister_opaque_type) fix listnode_delete of
          referenced node in loop.
        * ospf_interface.h: lists typedef removal cleanup.
        * ospf_opaque.{c,h}: lists typedef removal cleanup. update some list
          loops to LIST_LOOP. miscellaneous style and indent fixups.
        * ospf_te.{c,h}: ditto
* ospf_packet.c: lists typedef removal cleanup.

Remove usage of evil list and listnode typedefs.

2004-09-22 Paul Jakma <paul.jakma@sun.com>

* zserv.c: (zsend_route_multipath) fix nasty bad memset of struct
          in_addr to sizeof(struct in6_addr), causing odd and hard to debug
          crash.

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

* sockopt.c: Add missing bracket

Yet another DIS election fix from LIU Xin - update dis_record of
adjacencies when isisd becomes DIS.

Make "C" vendor routers happy - put correct prefix addresses into
reachability TLV's. Somehow it wasn't problem for "J" vendor routers.

2004-09-20 LIU Xin <lx at ns.6test.edu.cn>
* isis_dr.c, isis_events.c: Remove hello multiplier usage while scheduling
  DIS election.
* isis_pdu.c: Don't call isis_event_dis_status_change() whenever
  l[1|2]_desig_is is different from hdr.lan_id.

Rework of SPF threads schedule logic. I'm not 100% sure whether it's right
thing to do to have completely separate threads for IPv4 and IPv6 SPF
though. But it works and it's now possible to have both IPv4 and IPv6
enabled in interface. One bug fixed in LSP regeneration scheduling as well.

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

        * ripngd/ripngd.c: move setsockopt_so_recvbuf to lib.
        * lib/sockopt.{c,h}: Add setsockopt_so_recvbuf, for ripd and ripngd.
        * ripd/ripd.c: set receive buffer to a decent size, some systems
          have low defaults. Problem noted and fix suggested by Stephan
          Schweizer in [zebra 20967].

Another great little fix from LIU Xin.

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

       * vtysh.c: (vtysh_client_execute) trailling NULLs can be arbitrarily
         split across reads, dont get confused by this and block forever.

There is several new RFCs ...

Second part of fixes from Laurent Rabret.

There is no such option any more.

Isisd is now able to remove addresses from circuit and trigger LSP updates
if it's done. Some random fixes as well and update to sample configuration.

isis_pdu.c: Update l1_desig_is only if neighbor really is DIS.

Disable isisd compiling by default. It compiles only in Linux and FreeBSD
and works only in Linux ;(.

Merge svn revisions 997 and 1000 from Zebra cvs repository.

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

        * doc/quagga.texi: Update copyright. Contents should be at beginning
          of docs...
* redhat/quagga.spec.in: wee keep_build define, to keep build dir.

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

* Makefile.am: remove zebra/irdp.c from list - dead.
* extract.pl.in:  Substitute in the autoconf srcdir/builddir paths
  for includes to cpp.

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

        * configure.ac: capitalise the package name. autoconf lowercases
          it for PACKAGE_TARNAME.
        * lib/command.c: Update the copyright string in the default motd.

2004-09-13 Jose Luis Rubio <jrubio@dit.upm.es>
           (at Technical University of Madrid as part of Euro6ix Project)

        Enhanced Route Server functionality and Route-Maps:

        * bgpd/bgpd.h: Modified 'struct peer' and 'struct bgp_filter' to
        support rs-clients. A 'struct bgp_table *rib' has been added to the
        first (to mantain a separated RIB for each rs-client) and two new
        route-maps have been added to the last (for import/export policies).
        Added the following #defines: RMAP_{IN|OUT|IMPORT|EXPORT|MAX},
        PEER_RMAP_TYPE_{IMPORT|EXPORT} and BGP_CLEAR_SOFT_RSCLIENT.

        * bgpd/bgpd.c: Modified the functions that create/delete/etc peers in
        order to consider the new fields included in 'struct peer' for
        supporting rs-clients, i.e. the import/export route-maps and the
        'struct bgp_table'.

        * bgpd/bgp_route.{ch}: Modified several functions related with
        receiving/sending announces in order to support the new Route Server
        capabilities.
        Function 'bgp_process' has been reorganized, creating an auxiliar
        function for best path selection ('bgp_best_selection').
        Modified 'bgp_show' and 'bgp_show_route' for displaying information
        about any RIB (and not only the main bgp RIB).
        Added commands for displaying information about RS-clients RIBs:
        'show bgp rsclient (A.B.C.D|X:X::X:X)', 'show bgp rsclient
        (A.B.C.D|X:X::X:X) X:X::X:X/M', etc

        * bgpd/bgp_table.{ch}: The structure 'struct bgp_table' now has two
        new fields: type (which can take the values BGP_TABLE_{MAIN|RSCLIENT})
        and 'void *owner' which points to 'struct bgp' or 'struct peer' which
        owns the table.
        When creating a new bgp_table by default 'type=BGP_TABLE_MAIN' is set.

        * bgpd/bgp_vty.c: The commands 'neighbor ... route-server-client' and
        'no neighbor ... route-server-client' now not only set/unset the flag
        PEER_FLAG_RSERVER_CLIENT, but they create/destroy the 'struct
        bgp_table' of the peer. Special actions are taken for peer_groups.
        Command 'neighbor ... route-map WORD (in|out)' now also supports two
        new kinds of route-map: 'import' and 'export'.
        Added commands 'clear bgp * rsclient', etc. These commands allow a new
        kind of soft_reconfig which affects only the RIB of the specified
        RS-client.
        Added commands 'show bgp rsclient summary', etc which display a
        summary of the rs-clients configured for the corresponding address
        family.

        * bgpd/bgp_routemap.c: A new match statement is available,
        'match peer (A.B.C.D|X:X::X:X)'. This statement can only be used in
        import/export route-maps, and it matches when the peer who announces
        (when used in an import route-map) or is going to receive (when used
        in an export route-map) the route is the same than the one specified
        in the statement.
        For peer-groups the statement matches if the specified peer is member
        of the peer-group.
        A special version of the command, 'match peer local', matches with
        routes originated by the Route Server (defined with 'network ...',
        redistributed routes and default-originate).

        * lib/routemap.{ch}: Added a new clause 'call NAME' for use in
        route-maps. It jumps into the specified route-map and when it returns
        the first route-map ends if the called RM returns DENY_MATCH, or
        continues in other case.

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

        * Makefile.am: extract.pl isnt in srcdir, it's always in the
          builddir.
        * extract.pl.in: match on end of filenames, filename could contain
          various preamble due to out of tree builds.

autoreconf doesnt like top_srcdir var. update-autotools needs to -I m4 too
to ensure it's pulled in. (snapshot host depends on that script).

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

        * ospf_packet.c: Fix bugzilla #107
          (ospf_packet_max) get rid of the magic 88 constant
          (ospf_swab_iph_ton) new function. set ip header to network order,
          taking BSDisms into account.
          (ospf_swab_iph_toh) the inverse.
          (ospf_write) Add support for IP fragmentation, will only work on
          linux though, other kernels make it impossible. get rid of the
          magic 4 constant.
          (ospf_make_ls_upd) Bound check to end of stream, not to
          interface mtu.
          (ospf_ls_upd_packet_new) New function, allocate upd packet
          taking oversized LSAs into account.
          (ospf_ls_upd_queue_send) use ospf_ls_upd_packet_new to allocate,
          rather than statically allocating mtu sized packet buffer, which
          actually was wrong - it didnt take ip header into account, which
          should not be included in packet buffer.
          (ospf_ls_upd_send_queue_event) minor tweaks and remove
          TODO comment.

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

* ospfd.texi: OSPF MD5 auth requires stable time.

Cosmetical fixes in vty help.

Fixes from LIU Xin.

Indentation only. No any functional changes.

Add IS-IS info into array.

Fix aclocal m4/ include for out-of-tree builds - all src relative paths
need to be prefixed with $(top_srcdir)

Merge svn revision 981 from Zebra cvs.

Merge svn revisions 978-979 from Zebra CVS.

Merge svn revision 975 from Zebra repository.

Indentation consensus.

Ignore generated files.

Ignore *.sh.

2004-08-31  Greg Troxel  <gdt@poblano.ir.bbn.com>

        * Makefile.am (EXTRA_DIST): New - just has EXTRA_DIST.  (Using a
        directory in EXTRA_DIST in the parent was causing 'make dist' to fail.)

(Now m4/Makefile* is first-class and included from top level.)

2004-08-31  Greg Troxel  <gdt@poblano.ir.bbn.com>

* Rename scripts to *.sh.in, and use autoconf-style substitution
  for prefix and localstatedir.

Assorted changes from work at BBN.  Most are minor, and several are in
support of more significant changes not in this commit.  The last item
in the ChangeLog below may be needed for p2mp to work correctly.

2004-08-31  David Wiggins  <dwiggins@bbn.com>

* hash.c (hash_iterate): Save next pointer before calling
procedure, so that iteration works even if the called procedure
deletes the hash backet.

* linklist.h (listtail): new macro, not yet used.

2004-08-31  David Wiggins  <dwiggins@bbn.com>

* ospf_spf.c (ospf_spf_calculate): Many more comments and debug
  print statements.  New function ospf_vertex_dump used in debugging.

2004-08-31  David Wiggins  <dwiggins@bbn.com>

* ospf_spf.h (struct vertex): Comments for flags and structure members.

2004-08-31  David Wiggins  <dwiggins@bbn.com>

* ospf_route.c: When finding an alternate route, log cost as well.

2004-08-31  David Wiggins  <dwiggins@bbn.com>

* ospf_interface.c (ospf_lookup_if_params): Initialize af in
struct prefix allocated on stack.

2004-08-31  David Wiggins  <dwiggins@bbn.com>

* ospf_packet.c (ospf_ls_ack_send_delayed): In p2mp mode, send
acks to AllSPFRouters, rather than All-DR.