2003-06-19 Paul Jakma <paul@dishone.st>

* ospfd/ospf_packet.c: Update Hasso's hack slightly, add comment
  elaborating on Hasso's, use the SET/CHECK_FLAG macros.

2003-06-15 Paul Jakma <paul@dishone.st>

* lib/vty.{c,h}: Remove vty layer depending on a 'master' global,
  pass the thread master in explicitly to vty_init. Sort out some
  header dependency problems with lib/command.h
* zebra/: Move globals to struct zebrad. Update vty_init().
* (.*)/\1_main.c: update call to vty_init().

2003-06-12 Paul Jakma <paul@dishone.st>

* zebra/ioctl.c: Properly zero-out struct ifreq before passing it
          to the SIOCSIFFLAGS ioctl(2), so that it doesn't
          occasionally trigger promisc mode on interface.
          Reported/tested by: Vladimir B. Grebenschikov <vova@sw.ru>

          (patch snarfed taken from FreeBSD ports)

2003-06-11 Paul Jakma <paul@dishone.st>

* bgpd/ChangeLog: Update (wasnt commited with the fix)

2003-06-11 Paul Jakma <paul@dishone.st>

* lib/privs.h (zprivs_init): do not use zlog() as it depends on
  vty having been initialised (zlog calls vty_log). print to stderr
  directly instead.

2003-06-09 Paul Jakma <paul@dishone.st>

        * bgp_clist.c (community_list_delete): honour deny statements

2003-06-08 Paul Jakma <paul@dishone.st>

Compile and warning fixes for the ripd 'passive-interface default' patch.

2003-06-07 Paul Jakma <paul@dishone.st>

* (bgp_clist.c): Run it through indent -nut (all indentation to
   be spaced, get rid of mix of tabs and spaces)

From: Andrew J. Schorr <aschorr@telemetry-investments.com>
Subject: [zebra 12403] patch for ripd to accept any version of RIP
by default

The default Cisco IOS behavior is to send RIP version 1 packets and receive
version 1 and version 2 packets.  But zebra version 0.92a sends and receives
only version 2 packets by default.

I have patched the code to change zebra's default behavior to sending
version 2 packets (same as before) but receiving both versions.  While
this is still not identical to Cisco's behavior, it does now accept
packets of both versions and retains backwards compatibility with
zebra configurations.

From: Andrew J. Schorr <aschorr@telemetry-investments.com>
Subject: [zebra 12406] patch for ripd to support "passive-interface
default" behavior

In Cisco IOS, the router rip passive-interface subcommand can accept
an interface name of "default".  In that cases, all interfaces are set
to be passive by default, and you must explicitly make it non-passive
by using the "no passive-interface <ifname>" command if you want updates
to go to that interface.

<command implemented for zebra>

2003-06-07 Paul Jakma <paul@dishone.st>

* (zebra/rtadvc.): can not set checksum for ICMPV6, see:

From: Jun-ichiro itojun Hagino <itojun@iijlab.net>
Subject: [zebra 11052] setsockopt(IPV6_CHECKSUM) does not work for ICMPv6
socket

setsockopt(IPV6_CHECKSUM) does not work for ICMPv6 socket.
RFC2292 may not be clear enough about it (it talks about raw socket
and ICMPv6 socket mixed up), but the next revision
(draft-ietf-ipngwg-rfc2292bis-02.txt) is very clear about it.
>   An attempt to set IPV6_CHECKSUM for an ICMPv6 socket will fail.

2003-06-07 Paul Jakma <paul@dishone.st>

* (ospf_spf.c): Fix indentation - primarily the mix of tabs and spaces.
Ran through indent -nut (GNU style, but only spaces for indentation)

2003-06-07 Paul Jakma <paul@dishone.st>

* Revert Cougar's sort interface names patch, causes problems with
enabling of interfaces for OSPF in ospfd.

2003-06-7 kamatchi soundaram <kamatchi@tdd.sj.nec.com>

* (ospf_spf.c): Fix consideration of costs for PtP nexthops in
  ospf_nexthop_calculation().

2003-06-06 Paul Jakma <paul@dishone.st>

* Fix up defaults for --enable-{user,group}

2003-06-06 Paul Jakma <paul@dishone.st

* Fix up case of --enable-vty-group not being specified

Patch from Cougar - sort iflist by name.

Unbreak router advertisment feature when using capabilities.

Paul forgot probably to commit privs.[c|h] changes. Had to reorder includes
in zebra.h to make it compile in debian.

2003-06-04 Paul Jakma <paul@dishone.st>

* Merge of zebra privileges

2003-06-04 Paul Jakma <paul@dishone.st>

* (configure.in): removed
* (configure.ac): Fixes/cleanups backported from zprivs branch
   AC_DEFINE templates added,
   Indentation of nested macros,
   IPv6 checks changed from independent if..fi blocks to
     if...elif...fi.,
   Merge piddir and ac_sockdir into one ac_statedir,
   Remove world writeable dirs as candidates for ac_statedir,
   ERROR out if no state dir is found or --localstatedir not
             specified,

2003-06-04 Paul Jakma <paul@dishone.st>

* (Makefile.am): doc should be a SUBDIR. Specify DIST_SUBDIRS to
remove dependence of make dist on configure options. Add doc/mpls
to EXTRA_DIST. redhat/ does not need a dist-hook, its already in
SUBDIR. Remove CVS/ dir from the dist doc/.

Paul Jakma:

lib/debug.{c,h}: glibc backtrace printing function (from the glibc info page)
configure.ac: check for glibc backtrace and set define
lib/zebra.h: glibc backtrace support

Paul Jakma: acconfig.h is deprecated.

Ignore dist tar file

From: kamatchi soundaram <kamatchi@tdd.sj.nec.com>
Subject: [zebra 19263] Another memory leak!! is zebra OSPF

   This memory leak gets into picture whenever any interface goes down.

 Problem found and desctription: Whenever the interface goes down, the
 "def_params" (member of ospf_if_info) structure memory is not freed.

 Fix made: added the following line to free the "def_params" memory of
 ospf_if_info before calling the  "XFREE(MTYPE_OSPF_IF_INFO, ifp->info);"
 The added line is:
       ospf_del_if_params ((struct ospf_if_params *) IF_DEF_PARAMS (ifp));

From: kamatchi soundaram <kamatchi@tdd.sj.nec.com>
Subject: [zebra 19262] Re: Memory leak in OSPF

Fix applied for Kamatchi's original report was to remove the auth_crypt
initialisation from the wrong function. This should fix that.

From: kamatchi soundaram <kamatchi@tdd.sj.nec.com>
Subject: [zebra 19259] Memory leak in OSPF

The detail of the memory leak follows:
 File name: ospf_interface.c
 Function name: ospf_if_new_hook().
 Type of leak: Overwriting the already allocated memory.
 Problem found and description:
 The ospf_new_if_params() fn allocates a memory for "auth_crypt" which
 is overwritten, down in the fn (ospf_if_new_hook) by explicitely allocating
 a memory for the same "auth_crypt".

 Fix: remove allocation from ospf_new_if_params()

Apply redistribution route-map's correctly.

Commit my hack to fix options mismatch in NSSA area DD packets. It shouln't
harm, but of course ... it is hack.

ospfd/ospfd.h: Change delay and holdtime to 1s. More appropriate values for
the class of machines which zebra runs on than the original defaults which
were taken from IOS (and even IOS uses much lower times these days). Lower
values greatly help with convergence.

Ideally, we'd use even lower times, but thread_add_timer() api will need to
be modified to use milliseconds. (eg JunOS uses 100ms for delay, and 700ms
for hold time, iirc from Hasso).

lib/table.c: Fix the SET_LINK macro. (not that its of any worth)

ripngd/Makefile.am: missing ripng_nexthop.h from _HEADERS line. breaks
make dist.

IPv4 part of "make reject|blackhole routes work without nexthop|interface".

Rivo Nurgess: Add cobalt machine type

*BSD compile fix from Rivo Nurges.

Add .cvsignore for redhat

Sync to configure.ac

Change rtadv to default to disabled.

Fix missing else in static_add_ipv4() which was causing case of (!ifname)
to always be considered to be a blackhole route. (if gate is passed in, it
cant be).

Tiny fixes from 6Wind patch.

ospf6d part of 6Wind patch.

Add back support for null0 interface blackhole routes.

Last fixes from 6Wind patch.

Sync with configure.in.

Merged 6Wind patch.

Route (reject|blackhole) support from 6Wind patch.

Update changelog as well.

Ripngd part of 6Wind patch.

Unbroke "set metric" command in vtysh again.

RIP has now "route-map x [in|out] <interface>" as well.

Merge RIP part of 6Wind patch.

Just "write" command without any parameters writes conf to file.

"ip forwarding" command. Patch from Jim Crumpler.

Jim Crumpler:

working ipforward_solaris.c

Revert my http://hasso.linux.ee/zebra/ht-ifrmap-14042003.patch patch. Used
same idea as in lib/distribute.c to allow extract.pl to pick up commands
for vtysh.

From: rivo nurges <rix@estpak.ee>
Subject: solaris compile patch

types defined for SUNOS_5 were already defined in config.h

Fix missing bracket from previous commit.

2003-05-24 Kenji Yabuuchi

        * ospf_interface.c(ospf_if_lookup_recv_if): Use the most specific
          match for interface lookup.

Update changelog  -> fix from Anil Madhavapeddy

Fix incorrect buffer bounds.

Reported by Anil Madhavapeddy to bug-zebra.

Ripd has "no interface ..." command as well.

From: Hasso Tepper <hasso@estpak.ee>

ensure only interfaces that are inactive or do not exist can be deleted

Fix errant ;;

slight addition to hasso's previous no_interface patch.

From: Sergiy Vyshnevetskiy <serg@vostok.net>
Subject: [zebra 18994] Re: zebra-pj compile failure under OpenBSd 3.3
    (gmake)

properly detect rl_completion_matches vs completion_matches.

From: Hasso Tepper <hasso@estpak.ee>

Add the 'no interface' command to all the daemons and vtysh. now it's
possible to delete interface from routeing daemons as well only if it
doesn't exist in os.

http://hasso.linux.ee/zebra/ht-no_interface_fix.patch

Undo Hasso last patch, about to apply a slightly updated patch.

Allow group access to vty socket.

From: Hasso Tepper <hasso@estpak.ee>

http://hasso.linux.ee/zebra/ht-no_interface-23052003.patch

Add the 'no interface' command to all the daemons and vtysh.

Add additional system capabilities

From: Paul Jakma <paulj@alphyra.ie>
Subject: [zebra 19125] vtysh bogus perms check

vtysh checks that the zebra.vty socket uid is == to its own euid, this is
bogus - let the kernel decide whether permissions match, vtysh shouldnt
try second guess. (this becomes noticeable when neither zebra nor vtysh
are run as root :) )

From: Hasso Tepper <hasso@estpak.ee>
Subject: [zebra 19120] [PATCH] Vtysh

I really don't know how I could miss that one. Opsf(6)d have
prefix-lists as well.

Fix stray comment and using pwentry for group info.

Undo revision 1.3, revert back to zebra_org 1.1.1.2.
asm/types.h is required for netlink.
Later glibc's seem to have resolved the warning.

Fix several small caps related errors.

Privilege support files for zebra.
Linux capabilities enabled (if libcap is enabled).

Autogenerate the redhat/zebra.spec file using autoconf
supplied VERSION and CONFDATE.

Cleanup the openpam AC_MSG's slightly.
Add a DATE AC_SUBST.

From: Paul Jakma <paul@clubi.ie>
Subject: [zebra 19097] HAVE_ASM_TYPES

does anyone know why lib/zebra.h has:

#ifdef HAVE_ASM_TYPES_H
#include <asm/types.h>
#endif /* HAVE_ASM_TYPES_H */

There's no need for it that i can see (least not on linux) and it
causes compile warnings. Is it needed? If so, why?

From: Hasso Tepper <hasso@estpak.ee>
Subject: [zebra 19093] [PATCH] Two more fixes

1) From 6Wind patch - show router and interface nodes from ospf6d when
using vtysh and don't show defaults in interface node. Fixes bugzilla
#14.

From: Hasso Tepper <hasso@estpak.ee>
Subject: [zebra 19093] [PATCH] Two more fixes

Show NSSA LSA route info in "show ip ospf database" output

Sync configure.in to configure.ac (sergey's openpam support)

Fix attribution

Update Changelogs

From: Hasso Tepper <hasso@estpak.ee>
Subject: [zebra 19091] [PATCH] NSSA fixes try2

NSSA didn't work without problems even after my previous fixes. Seems
that I tracked down problems finally.

From: Gilad Arnold <gilad.arnold@terayon.com>
Subject: [zebra 19084] Re: suspected memory leakage upon static route

I believe this is a more complete patch, it also addresses
allocating/freeing of nexthop->ifname string buffer (1) using
XSTRDUP/XFREE macros (thus correctly updating memory statistics), and
(2) for all nexthop->type cases that carry an ifname string.

From: Gilad Arnold <gilad arnold at terayon com>
Subject: [zebra 19080] suspected memory leakage upon static route deletion

upon deletion of a static route entry, at the end of
zebra/zebra_rib.c/static_delete_ipv4(), there's a call to XFREE to
deallocate the 'struct static_ipv4'. However, in the case of a static with
ifname as nexthop (type==STATIC_IPV4_IFNAME), this struct holds a pointer to
a dynamically allocated ifname string buffer (si->gate.ifname, see its
allocation within static_add_ipv4() using XSTRDUP). IMO, the attached patch
is required.

Notes: the same applies to IPv6 statics as well (fix included in patch);
and, I admit I haven't yet tested that, but it seems simple enough to be
working... ;->

Fix merge error of Hasso's patch "case case" :)

From: Hasso Tepper

Fix removal of Type-7 LSAs, http://bugzilla.dishone.st/show_bug.cgi?id=18

Hopefully fix the 'bgp dump routes-mrt' command to actually dump routes.
Should recognise the absence of an interval now and schedule one-shot dump.
hopefully.

From: Sergiy Vyshnevetskiy <serg @ vostok.net>
Subject: [zebra 19019] [PATCH] daemons -A option

Works now. Patch attached. Not tested for IPv6, but should work there too.

From: Sergiy Vyshnevetskiy <serg@vostok.net>
Subject: [PATCH] sockdir

"configure.ac" script ignores absent socket directory. So do all demons
(with just a puzzling complaint to _stderr_ only!). I added /tmp and
/var/tmp to the list of possible dirs. They should be present in most
configurations.

Reverting the NSSA maxage walker loop addition for now, until all the other
problems with NSSA are resolved, and we can determine whether this had any
point.

NSSA /might/ be missing this. I'm not sure.
It didnt solve a particular problem of Hasso's, but i'm committing it just
for posterities sakes, so i dont lose it (i'll revert it immediately).

From: Sergiy Vyshnevetskiy <serg@vostok.net>
Subject: [zebra 18947] [PATCH] openpam patch

configure support for openpam (freebsd5.0)

From: Sergiy Vyshnevetskiy <serg@vostok.net>
Subject: [zebra 18941] Re: [PATCH] misc patch

Program shouldn't terminate on pager execution failure. We can also use
"PAGER" environment variable - it's very common.

Import of zebra.org 20030428-18:07 IST

From: Hasso Tepper

This one fixes "router xxx" node commands in vtysh. Don't get "unknown
command" error when entering "passive-interface eth0" command while
ospfd and ospf6d running etc.

vtysh_cmd.c is no longer in repository, ignore it.