summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * *: fix signedness mix-upsDavid Lamparter2015-04-215-11/+12
| | | | | | | | Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
| * *: remove stray extra semicolonsDavid Lamparter2015-04-214-7/+7
| | | | | | | | | | | | Some places had extra semicolons where none belong. Remove them. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
| * *: use void * for printing pointersDavid Lamparter2015-04-2118-84/+103
| | | | | | | | | | | | | | | | | | On higher warning levels, compilers expect %p printf arguments to be void *. Since format string / argument warnings can be useful otherwise, let's get rid of this noise by sprinkling casts to void * over printf calls. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
| * *: use long long to print time_tDavid Lamparter2015-04-1911-43/+49
| | | | | | | | | | | | | | | | | | | | | | | | Since we can't assume time_t to be long, int, or even long long, this consistently uses %lld/long long (or %llu/unsigned long long in a few cases) to print time_t/susecond_t values. This should fix a bunch of warnings, on NetBSD in particular. (Unfortunately, there seems to be no "PRId64" style printing macro for time_t...) Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* | bgpd: Display of configured dampening parametersBalaji2015-04-142-0/+35
| | | | | | | | | | | | | | | | Function to display configured bgp dampening parameters. Signed-off-by: Balaji.G <balajig81@gmail.com> [DL: formatting adjustments] Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* | bgpd: Configured suppress value cannot be less than the reuse value in bgp ↵Balaji2015-04-141-0/+8
| | | | | | | | | | | | | | | | | | dampening RFC 2439, Section 4.2; the values pair up for hysteresis. Signed-off-by: Balaji.G <balajig81@gmail.com> Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* | Fix alignment assumptions on non-RT_ROUNDUP platforms.Greg Troxel2015-03-231-2/+2
|/ | | | | | | The comment said that apple uses int and BSD traditionally used long, but the code was backwards. This fixes apple to be int, and otherwise long. That should make FreeBSD, which aligns to long, work correctly, even without using SA_SIZE.
* build: list actual release procedureDavid Lamparter2015-03-071-5/+25
| | | | | | | | As in a few other places in HACKING.tex, the text doesn't quite reflect reality. Add the actual release procedure including a few more steps, and warn about autoconf's subdirectory behaviour. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* zebra: don't print uninitialized string (3b02fe8)David Lamparter2015-03-071-1/+1
| | | | | | | | | This crept in as part of the MRIB improvements and I missed the compiler warning between other noise. Unfortunately, printing an uninitialised variable can in fact make zebra crash, so this is not trivial. Fixes: 3b02fe8 ("zebra: add "show ip rpf" to get result of RPF lookup") Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* lib: Fix POSIX capabilities on SunOS platformsBrian Bennett2015-03-041-27/+49
| | | | | | | | | | | | | | When using POSIX capabilities on SunOS the capabilities are too restricitve resulting in quagga processes not being able to read their own config files. Credit goes to Oracle where this patch was originally authored and included in OpenSolaris. lib/privs.c: Include additional capabilities, better checking of missing capabilities. Fixes: #820 Acked-by: Greg Troxel <gdt@ir.bbn.com> Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* build: tag version as 0.99.25-devDavid Lamparter2015-03-041-1/+1
| | | | Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* release: 0.99.24quagga-0.99.24David Lamparter2015-03-032-1/+31
|
* zebra: print "no link-detect"David Lamparter2015-03-021-0/+2
| | | | | | | The default for this is slated to change, so let's print the current default value for preexisting configurations. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* solaris: fix SMF manifest dependency model and start methodBrian Bennett2015-03-022-31/+7
| | | | | | | | | | | | | | | | | | | Resolves an issue where quagga daemons restart in an infinite loop. Quagga daemons declare a dependency on zebra that requires a restart of the daemon when zebra restarts and they explicitly restart zebra, which again triggers their own restart. Restarting zebra when other daemons are started is explicitly removed, leaving dependency management up to SMF rather than handling it in the start method. solaris/quagga.init.in: Remove calls to routeadm_zebra_enable, and the routeadm_zebra_enable function. solaris/quagga.xml.in: Set dependency zebra grouping to require_all. Fixes: #818 Signed-off-by: Greg Troxel <gdt@ir.bbn.com> Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* build: Extend ip_mreq hack to DragonFlyBSD and SunOSBrian Bennett2015-03-021-1/+1
| | | | | | | | | | | | This extends the ip_mreq hack to DragonFlyBSD and SunOS. This has been in pkgsrc for some time. I've cleaned up the pkgsrc patch a little and am submitting it upstream. Credit is due to pkgsrc maintainers. Tested on SmartOS (illumos). Fixes: #819 Signed-off-by: Greg Troxel <gdt@ir.bbn.com> Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* build: enable AM_SILENT_RULESDavid Lamparter2015-02-141-0/+1
| | | | | | | | | | | This shuts up make by default (can be reversed with "make V=1" or --disable-silent-rules). This is useful since warnings and error messages become more visible with less noise. Tested on Linux with GNU make and FreeBSD with system's BSD make. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Paul Jakma <paul@jakma.org>
* build: enable isisd by defaultDavid Lamparter2015-02-141-4/+3
| | | | | | | | Most distributors enable it anyway, and it's not THAT broken anymore to mandate disabling it by default. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Paul Jakma <paul@jakma.org>
* doc: fix some warningsDavid Lamparter2015-02-143-5/+5
| | | | | Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Paul Jakma <paul@jakma.org>
* build: track config argsDavid Lamparter2015-02-143-0/+7
| | | | | | | Record the ./configure arguments used and make them user-visible. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Paul Jakma <paul@jakma.org>
* tests: remove --disable-ipv6David Lamparter2015-02-143-30/+10
| | | | | | | | With --disable-ipv6 gone, the IPv6 detection logic in the tests is not needed anymore either. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Paul Jakma <paul@jakma.org>
* build: remove --disable-ipv6David Lamparter2015-02-142-31/+7
| | | | | | | | | | | | | | | | Building with IPv6 disabled tends to break rather often and sprinkles ugly #ifdefs around the code. All that only to support systems where the C library doesn't have IPv6 capability. The year now being 2015, if this is a problem the thing to fix is the C library. The implication of this patch is that future patches need not care about HAVE_IPV6 = 0 and may remove ifdefs gratuitously. This patch doesn't remove these ifdefs to not create unneccessary churn. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Paul Jakma <paul@jakma.org>
* build: get rid of INCLUDES, use AM_CPPFLAGSDavid Lamparter2015-02-1416-20/+15
| | | | | | | | | | | | INCLUDES in configure.ac was not used at all, and INCLUDES in Makefile.am is supposed to be AM_CPPFLAGS these days. Reduces warnings spewed during bootstrap/autoreconf. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Greg Troxel <gdt@ir.bbn.com> Acked-by: Feng Lu <lu.feng@6wind.com> Acked-by: Paul Jakma <paul@jakma.org>
* build: harmonize configure help stringsDavid Lamparter2015-02-141-42/+42
| | | | | | | Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Greg Troxel <gdt@ir.bbn.com> Acked-by: Feng Lu <lu.feng@6wind.com> Acked-by: Paul Jakma <paul@jakma.org>
* build: remove --enable-solaris parameterDavid Lamparter2015-02-141-9/+3
| | | | | | | | | | | This switch controlled descending into the solaris/ subdirectory, which contains package descriptions and init scripts. If they're not appropriate, they'd better be removed outright. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Greg Troxel <gdt@ir.bbn.com> Acked-by: Feng Lu <lu.feng@6wind.com> Acked-by: Paul Jakma <paul@jakma.org>
* zebra: remove rt_ioctl kernel interfaceDavid Lamparter2015-02-143-548/+2
| | | | | | | | | None of the BSDs uses ioctls to set routes anymore. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Greg Troxel <gdt@ir.bbn.com> Acked-by: Feng Lu <lu.feng@6wind.com> Acked-by: Paul Jakma <paul@jakma.org>
* build: remove ancient Linux/BSD IPv6 cruftDavid Lamparter2015-02-142-73/+14
| | | | | | | | | IPv6 functions in a separate library... yeah, right. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Greg Troxel <gdt@ir.bbn.com> Acked-by: Feng Lu <lu.feng@6wind.com> Acked-by: Paul Jakma <paul@jakma.org>
* zebra: remove kernel_delete_ipv6_old()David Lamparter2015-02-145-39/+0
| | | | | | | | | | The only user of this was rib_bogus_ipv6(), which was removed in the previous commit. Good riddance. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Greg Troxel <gdt@ir.bbn.com> Acked-by: Feng Lu <lu.feng@6wind.com> Acked-by: Paul Jakma <paul@jakma.org>
* zebra, ripngd: remove ::/64 special-casingDavid Lamparter2015-02-142-43/+0
| | | | | | | | | | | | | | | | | In the 90ies, IPv4 was believed to exist within IPv6, with some kernels implementing this belief in code... Our code here is keyed to "#ifdef LINUX", yet no Linux from the past 10 years had this, making the code completely useless. FreeBSD 10.0 does in fact have a "::/96 via ::1 dev lo0 reject" route. IMHO we shouldn't mess with that, the admin can filter as neccessary anyway. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Greg Troxel <gdt@ir.bbn.com> Acked-by: Feng Lu <lu.feng@6wind.com> [DL: slightly adjusted commit message to remove misunderstanding] Acked-by: Paul Jakma <paul@jakma.org>
* build: remove INRIA, NRL and MUSICA IPv6 quirksDavid Lamparter2015-02-1410-77/+15
| | | | | | | | | Valar dohaeris. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Greg Troxel <gdt@ir.bbn.com> Acked-by: Feng Lu <lu.feng@6wind.com> Acked-by: Paul Jakma <paul@jakma.org>
* build: remove AIX, NEC EWS and IRIXDavid Lamparter2015-02-148-272/+5
| | | | | | | | | Valar morghulis. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Greg Troxel <gdt@ir.bbn.com> Acked-by: Feng Lu <lu.feng@6wind.com> Acked-by: Paul Jakma <paul@jakma.org>
* build: remove Linux non-netlink configDavid Lamparter2015-02-144-472/+8
| | | | | | | | | | This path is deprecated, completely untested, likely broken and will not be maintained. Kill it with fire. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Greg Troxel <gdt@ir.bbn.com> Acked-by: Feng Lu <lu.feng@6wind.com> Acked-by: Paul Jakma <paul@jakma.org>
* build: remove bogus/deprecated inet_* testsDavid Lamparter2015-02-141-7/+0
| | | | | | | | | | | | | | | These actually break configure on FreeBSD very subtly, because inet_aton and __inet_aton are both detected, and then later other tests get warnings about HAVE_INET_ATON being defined twice. That said, they're incorrect to begin with since they detect alternative functions but there is nothing in place to actually use these alternates. Signed-off-by: David Lamparter <equinox@opensourcerouting.org> Acked-by: Greg Troxel <gdt@ir.bbn.com> Acked-by: Feng Lu <lu.feng@6wind.com> Acked-by: Paul Jakma <paul@jakma.org>
* build: Quagga 0.99.24-rc1quagga-0.99.24-rc1David Lamparter2015-02-101-1/+1
| | | | | | | | this is not a full release version, so neither release notes nor documentation are updated yet. Also, signing the tag with my private GPG key instead of the Quagga one. Signed-off-by: David Lamparter <equinox@diac24.net>
* build: enable pimd in test scriptDavid Lamparter2015-02-061-1/+1
| | | | Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
* pimd: merge pimd as of 2015-01-19David Lamparter2015-02-06109-9/+23919
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Welcome pimd to the Quagga daemon zoo! This is a merge of commit 77ae369 ("pimd: Log ifindex found for an interface when zebra lib reports a new connected address."), with the intermediate "reconnect" changes removed (c9adf00...d274381). d274381 is replaced with b162ab7, which includes some changes. In addition, 4 reconnect-related changes and 1 cosmetic one have been bumped out. The rebase command used to produce the branch that is merged here is: git rebase --onto b162ab7 c9adf00 77ae369 Note that 3 patches had their author rewritten from "Anonymous SR#108542 <>" (which is not a valid git author ID) to: "Savannah SR#108542 <nbahr@atcorp.com>" (which is the e-mail address listed in the associated Savannah ticket) Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
| * doc: list pimd.8 in EXTRA_DISTDavid Lamparter2015-02-041-1/+1
| | | | | | | | Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
| * vtysh: add missing pimd defineDavid Lamparter2015-02-041-0/+1
| | | | | | | | Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
| * Revert "pimd: clear zclient-update: Reset zclient update connection to zebra ↵David Lamparter2015-02-043-14/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | daemon" This reverts commit 3456a80f5f8e6e44c30453bd92eabf5faf7ab25b. Conflicts: pimd/pim_zebra.c This depends on the zebra reconnect changes, which we're not picking up at this point. This revert is partial, only bumping out the reconnect-related changes. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
| * Revert "pim: Remove connected addresses on loss of zebra connection."David Lamparter2015-02-041-3/+1
| | | | | | | | | | | | | | | | | | This reverts commit 96b6dfe98793549aca6a7cc77eaf0957b1168ed2. This depends on the zebra reconnect changes, which we're not picking up at this point. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
| * Revert "pimd: Explicitly restart zclient update connection."David Lamparter2015-02-041-4/+1
| | | | | | | | | | | | | | | | | | This reverts commit 8150beed9a4f50a72696a65c1f40889ab65ad7ff. This depends on the zebra reconnect changes, which we're not picking up at this point. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
| * Revert "pimd: Revert: Explicitly restart zclient update connection."David Lamparter2015-02-041-0/+2
| | | | | | | | | | | | | | | | | | This reverts commit 199f85ade39f751dd493fe011107736c9b168953. This depends on the zebra reconnect changes, which we're not picking up at this point. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
| * redhat: revert non-pim changes in .specDavid Lamparter2015-02-041-4/+4
| | | | | | | | | | | | | | No idea what weird Fedora magic this does... if it's needed, it can be pushed separately from pimd. Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
| * pimd: Log ifindex found for an interface when zebra lib reports a new ↵Everton Marques2015-02-043-10/+10
| | | | | | | | connected address.
| * pimd: Fix configuration file reading upon startupDonald Sharp2015-02-041-6/+5
| | | | | | | | | | | | | | | | Without the fix, qpimd issues this error message: pim_if_add_vif: ifindex=0 < 1 on interface swp1 It happens because in pim_main.c:main() we are initializing zebra with pim_zebra_init() after we read in the configuration with vty_read_config(). See also: https://github.com/udhos/qpimd/issues/3
| * pimd: Addition of Hello & Join-Prune message debug commandsBalaji.G2015-02-045-5/+68
| | | | | | | | | | Separate "debug pim packets hello and Join-Prune" added to enable hello and Join-Prune debugs specifically
| * pimd: show ip pim lan-prune-delay: Cosmetic.Everton Marques2015-02-041-2/+2
| |
| * pimd: Report del_oif() failure within igmp_source_forward_stop().Everton Marques2015-02-042-10/+23
| |
| * pimd: sh ip multicast: Display zclient socket fail counter.Everton Marques2015-02-041-2/+4
| |
| * pimd: Revert: Explicitly restart zclient update connection.Everton Marques2015-02-041-2/+0
| |
| * pimd: Update lookup zclient counter for connection failures.Everton Marques2015-02-041-18/+25
| |