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

Initialize access lists here as well.

Fix critical bugzilla #113. Make CMD_ERR_NOTHING_TODO nonfatal.

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

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

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

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

Some compiler warnings fixes and fix for bugzilla #119.

Unbreak "show ip route" command.

Unbreak ospfclient compilation.

ospf6d supports SNMP as well now.

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

* bump version to 0.97.1, release imminent.

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

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

Better common router-id handling.

Update NEWS - apiserver is disabled by default now.

Disable ospfapi init by default.

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

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

Fix warnings here.

Sync isisd with changes in lib.

Sync with changes in lib. Make more strings const.

Some more const string fixes.

Commas -> dots.

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Merge svn revisions 1065 and 1066 from Zebra CVS.

Compiler warnings fixes round 1.

Argh, fix typo.

There is no warnings here any more.

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

More const strings.

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

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

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

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

Vtysh compiles cleanly as well.

Even more const strings and fix to bugfix.

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

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

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

No warnings here any more.

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

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

Make more strings const.

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

* bump version to 0.97.0, release imminent.

- make specfile actually work

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

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

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

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

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

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

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

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

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

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

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

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.