all: remove build_host and build_date to enable reproducible building

The hostname and date/time varies from system to system, so it is
not possible to reproduce the binary when this data is embedded.

Signed-off-by: Reiner Herrmann <reiner@reiner-h.de>
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

all: make sure all make filelists are sorted

Originally by Reiner, adjusted by Ferry to sort all filelists.

Signed-off-by: Reiner Herrmann <reiner@reiner-h.de>
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Merge pull request #49 from wiedi/osx-clock-10.12

mach: clock_gettime and friends appeared in Mac OSX 10.12

mach: clock_gettime and friends appeared in Mac OSX 10.12

Signed-off-by: Sebastian Wiedenroth <wiedi@frubar.net>

info: fix the previous commit: windows doesn't support MSG_DONTWAIT

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: do not block when sending data

Even though checks are done that we can actually send, the send
can still block.

Reported-By: Joe Ayers <joe@ayerscasa.com>
Signed-off-by: Henning Rogge <hrogge@gmail.com>
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: upgrade gradle wrapper

The magical incantation is:
  gradle wrapper --gradle-version 4.4

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

process_routes: fix potential null pointer dereference warnings on gcc 7

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

src: fix format truncation errors on gcc 7

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

olsr_spf: fix potential null pointer dereference warnings on gcc 7

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

nameservice: fix potential null pointer dereference warnings on gcc 7

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

all: link with -lm where <math.h> is included.

Otherwise we'll get crashes on OpenWRT

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

hashing: fix implicit fall-through warning on gcc 7

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: use Java 1.8

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: upgrade gradle wrapper

The magical incantation is:
  gradle wrapper --gradle-version 4.3

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: upgrade gradle wrapper

The magical incantation is:
  gradle wrapper --gradle-version 4.1

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: upgrade workspace

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Merge pull request #32 from stspdotname/fix-sendto-ipv4

Fix olsr_sendto() for IPv4 on BSD.

Fix olsr_sendto() for IPv4 on BSD.

Sending multicast packets requires the IP_MULTICAST_IF socket option.
Without this socket option, packets sent to 255.255.255.255 are dropped.

With this change, olsr_sendto() can become a direct call to sendto() and
the -DSPOOF workaround which relies on libnet to send packets is no
longer required.

Tested on OpenBSD.

Signed-off-by: Stefan Sperling <stsp@stsp.name>

Merge pull request #37 from stspdotname/bcast-addr-len-and-type

Fix initialization of a struct sockaddr_in.

Fix initialization of a struct sockaddr_in.

While reading an IPv4 broadcast address from the configuration file,
do not forget about initializing the family sockaddr_in struct field.
Otherwise the sockaddr_in struct is invalid, and a BSD kernel will reject it.

Signed-off-by: Stefan Sperling <stsp@stsp.name>

info: java: fix tests by letting mock bind to localhost

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Merge pull request #31 from stspdotname/add-ipv6-routes-correctly

Fix adding IPv6 routes on BSD.

Merge pull request #35 from stspdotname/icmp6-rediraccept

Do not toggle net.inet6.icmp6.rediraccept on OpenBSD.

Fix adding IPv6 routes on BSD.

The KAME hack (storing the interface index inside the prefix) is only used
for link-local addresses. So only apply it in case the gateway is link-local.

For gateway routes (reachable on-link), install a cloning route which tells
the kernel to perform NDP address resolution to find the gateway.

The previous code was installing a bogus route to fe:80::1 for any destination.
I don't see how this could ever have worked.

Tested in Berlin's Freifunk mesh.

Signed-off-by: Stefan Sperling <stsp@stsp.name>

Merge pull request #36 from stspdotname/rtm-delete-flags-v6

Use correct RTM flags while deleting an IPv6 route on BSD.

Use correct RTM flags while deleting an IPv6 route on BSD.

Same problem as fixed for IPv4 in 65978bd6996c8f62ac3f0adf971692fc8f2dc9cb.
I forgot to include the IPv6 code path in that commit.

Signed-off-by: Stefan Sperling <stsp@stsp.name>

Merge pull request #34 from stspdotname/rtm-delete-flags

Use correct RTM flags while deleting a route on BSD.

Merge pull request #25 from neeels/master

fix segfault when printing "You must be root"

Merge pull request #26 from neeels/segfault_cleanup

fix: don't attempt to cleanup uninitialized multiGwTunnels

fix: don't attempt to cleanup uninitialized multiGwTunnels

If olsr_cnf->smart_gw_use_count is 1, no multi-gw tunnels are initialized, and
the sgwTunnel4InterfaceNames and sgwTunnel6InterfaceNames are actually NULL. Do
not attempt to cleanup those if NULL.

Symptom: stopping olsrd e.g. via SIGINT leads to a segfault during cleanup.
The reason is that smart_gw_use_count is 1, which attempts to then dereference
one of sgwTunnel{4,6}InterfaceNames[0], amounting to <NULL>[0].

Signed-off-by: Neels Hofmeyr <neels@hofmeyr.de>

fix segfault when printing "You must be root"

When olsr_cnf is NULL, always print OLSR_PRINTF messages to avoid segfault.

I compiled olsrd and ran ./olsrd --help, resulting in a segfault.
That's because main.c prints an error message even before the config is loaded,
so that OLSR_PRINTF tries to lookup the logging level in a NULL olsr_cnf.

Signed-off-by: Neels Hofmeyr <neels@hofmeyr.de>

Do not toggle net.inet6.icmp6.rediraccept on OpenBSD.

OpenBSD no longer supports the net.inet6.icmp6.rediraccept sysctl.
It was removed in 2014: http://marc.info/?l=openbsd-cvs&m=140914827732101&w=2

Signed-off-by: Stefan Sperling <stsp@stsp.name>

Use correct RTM flags while deleting a route on BSD.

If a route to a given destination already exists, the add_del_route()
function attempts to delete this route before inserting a new one.
While doing so, it asks olsr_rt_flags() for route message flags to use.

It should ask for route message flags that apply to a deletion. However,
it only did so if the 'add' parameter is false. This looks like a case of
badly copy-pasted code, since this section of code is obviously trying
to delete a route regardless of the value of 'add'.

Signed-off-by: Stefan Sperling <stsp@stsp.name>

Merge pull request #30 from stspdotname/route-deletion-needs-netmask

Fix route deletion on BSD.

Fix route deletion on BSD.

Specify a netmask when deleting routes. If no netmask is passed
in the routing message then deletion of the route can fail.

This fix covers both IPv4 and IPv6 routes.
Tested on OpenBSD only, but this change should not hurt other BSDs.

Signed-off-by: Stefan Sperling <stsp@stsp.name>

Merge pull request #29 from stspdotname/detect-wifi-on-bsd

Implement a simpler way of detecting wifi interfaces on BSD.

Merge pull request #28 from stspdotname/missing-olsr-random-include

Add a missing include for olsr_random() on BSD.

Implement a simpler way of detecting wifi interfaces on BSD.

The media type of an interface can be used to tell wireless interfaces
apart. The SIOCGIFMEDIA ioctl works the same way across all BSDs,
whereas the net80211 ioctls used previously differ.

Signed-off-by: Stefan Sperling <stsp@stsp.name>

Add a missing include for olsr_random() on BSD.

Signed-off-by: Stefan Sperling <stsp@stsp.name>

info: java: upgrade gradle wrapper

The magical incantation is:
  gradle wrapper --gradle-version 4.0

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Merge branch 'release-0.9.6'

* release-0.9.6:
  Update version after release of v0.9.6.2
  Release v0.9.6.2

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Update version after release of v0.9.6.2

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Release v0.9.6.2

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Merge branch 'release-0.9.6'

* release-0.9.6:
  link_set: fix typo in f8d4e24

link_set: fix typo in f8d4e24

This broke LinkQualityMult

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

link_set: fix typo in f8d4e24

This broke LinkQualityMult

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

pud: only check gpsd status; no clear of info on bad fix or invalid signal

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: gradle: upgrade wrapper

The magical incantation is:
  gradle wrapper --gradle-version 3.5

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: gradle: upgrade wrapper

The magical incantation is:
  gradle wrapper --gradle-version 3.4.1

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: update jackson bundles

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: update workspace

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: gradle: upgrade wrapper

The magical incantation is:
  gradle wrapper --gradle-version 3.4

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Merge branch 'release-0.9.6'

* release-0.9.6:
  Update version after release of v0.9.6.1
  Release v0.9.6.1

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Update version after release of v0.9.6.1

Signed-off-by: Henning Rogge <hrogge@gmail.com>

Release v0.9.6.1

Signed-off-by: Henning Rogge <hrogge@gmail.com>

Merge branch 'release-0.9.6'

* release-0.9.6:
  hello reception: only remove UNSPEC duplicates
  hello reception: remove duplicate neighbors
  lq_packet: do not report duplicate neighbours in HELLO messages
  hello reception: use link status ordering
  lq_packet: make the hello link order a define
  info: automatically detect whether the reply should have HTTP headers
  info: java: upgrade gradle wrapper

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

hello reception: only remove UNSPEC duplicates

Only UNSPEC links can be duplicates, so only try to remove those.
This is quite a bit faster at the cost of some complexity.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

hello reception: remove duplicate neighbors

See the explanations in the descriptions of commits 2f38613 and
5089a79.

To fix the issue more thoroughly we need to ensure that no duplicate
neighbours are sent or received because that confuses the receiver.
This is especially true when the HELLO messages are fragmented, and
even more so when the fragments have some delay between them since
that causes link instability.

This commit fixes that for the receiving side.

Signed-off-by: Iwan G. Flameling <iwanovich@gmail.com>
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

lq_packet: do not report duplicate neighbours in HELLO messages

See the explanations in the descriptions of commits 2f38613 and
5089a79.

To fix the issue more thoroughly we need to ensure that no duplicate
neighbours are sent or received because that confuses the receiver.
This is especially true when the HELLO messages are fragmented, and
even more so when the fragments have some delay between them since
that causes link instability.

This commit fixes that for the sending side.

Signed-off-by: Iwan G. Flameling <iwanovich@gmail.com>
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

hello reception: use link status ordering

See the explanation in the description of commit 2f38613.
This commit fixes the receive side of the described problem.

From this commit on 'broken' senders and 'fixed' receivers can
interoperate correctly.

The only case left unfixed is 'broken' senders and 'broken' receivers
(which is obviously unfixable), all other combinations are fixed.

'broken' = v0.6.6.1-136-g3d2fd73 <= olsrd < v0.9.5-82-g2f38613
'fixed'  = olsrd <  v0.6.6.1-136-g3d2fd73
           OR
           olsrd >= v0.9.5-82-g2f38613 (senders  )
           olsrd >= this commit        (receivers)

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

lq_packet: make the hello link order a define

This will be re-used later

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: automatically detect whether the reply should have HTTP headers

This is the case when a HTTP request is done.
The request can still override whether or not HTTP headers are sent
by employing the 'http' and 'plain' request prefixes.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: automatically detect whether the reply should have HTTP headers

This is the case when a HTTP request is done.
The request can still override whether or not HTTP headers are sent
by employing the 'http' and 'plain' request prefixes.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: upgrade gradle wrapper

The magical incantation is:
  gradle wrapper --gradle-version 3.3

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

info: java: upgrade gradle wrapper

The magical incantation is:
  gradle wrapper --gradle-version 3.3

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Merge branch 'release-0.9.6'

* release-0.9.6:
  changelog: fixup
  Update version after release of v0.9.6
  Release v0.9.6
  release: insert changelog just before the marker of the previous release
  release: minor improvements to the release script
  changelog: minor layout fixes

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

changelog: fixup

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Update version after release of v0.9.6

Signed-off-by: Henning Rogge <hrogge@gmail.com>

Release v0.9.6

Signed-off-by: Henning Rogge <hrogge@gmail.com>

release: insert changelog just before the marker of the previous release

So that the notes that are added manually stay on top.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

release: minor improvements to the release script

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

changelog: minor layout fixes

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Update version after branch of v0.9.6

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Branch v0.9.6

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

lq_packet: change link status ordering in hello messages

A while ago we noticed that neighbours of nodes with multiple interfaces
on the same medium report infinite costs on their links to those nodes.

This problem is 100% reproducible, and was traced down to
commit 3d2fd73a5528a9c7cdccd088f2dcca80b37e66b9
Author: Henning Rogge <henning.rogge@fkie.fraunhofer.de>
Date:   Mon Mar 31 11:18:30 2014 +0200

    Fix processing error for fragmented hellos

Below is the setup in which we noticed the problem.

       wlan0                                                 wlan0
       172.31.175.97/16                           172.31.175.61/16
    (((*))) ------------------------------------------------- (((*)))
       |                                                         |
       |                                                         |
       |                                                         |
   ____|___   172.29.175.97/15   ________  172.29.175.61/15  ____|____
  |         |-eth1.2580---------|        |--------eth1.2580-|         |
  | Node 97 |                   | Switch |                  | Node 61 |
  |_________|-eth2.2580---------|________|                  |_________|
              172.28.175.97/15

In this setup node 97 will report normal link costs for its (wired) links
to node 61 (see the first table below), while node 61 will report infinite
link costs for both its (wired) links to node 97 (see the second table
below).

  Table: Links (node 97)
  Local IP       Remote IP       Hyst.   LQ      NLQ     Cost
  172.29.175.97  172.29.175.61   0.000   1.000   1.000   0.100
  172.28.175.97  172.29.175.61   0.000   1.000   1.000   0.100
  172.31.175.97  172.31.175.61   0.000   1.000   1.000   1.000

  Table: Links (node 61)
  Local IP       Remote IP       Hyst.   LQ      NLQ     Cost
  172.29.175.61  172.29.175.97   0.000   1.000   0.000   INFINITE
  172.29.175.61  172.28.175.97   0.000   1.000   0.000   INFINITE
  172.31.175.61  172.31.175.97   0.000   1.000   1.000   1.000

Checking the HELLO messages that are received on node 61 from node 97,
we see the following:

  [node 61] # tcpdump -vni eth1.2580 udp port 698
  tcpdump: listening on eth1.2580, link-type EN10MB (Ethernet), capture size 262144 bytes
  06:21:23.528204 IP (tos 0xc0, ttl 1, id 42455, offset 0, flags [DF], proto UDP (17), length 80)
      172.28.175.97.698 > 255.255.255.255.698: OLSRv4, seq 0xf7c0, length 52
     Hello-LQ Message (0xc9), originator 172.31.175.97, ttl 1, hop 0
       vtime 3.000s, msg-seq 0x533d, length 48
       hello-time 1.000s, MPR willingness 3
         link-type Symmetric, neighbor-type Symmetric, len 12
           neighbor 172.29.175.61, link-quality 0.00%, neighbor-link-quality 0.00%
         link-type Unspecified, neighbor-type Symmetric, len 20
           neighbor 172.31.175.61, link-quality 0.00%, neighbor-link-quality 0.00%
           neighbor 172.29.175.61, link-quality 0.00%, neighbor-link-quality 0.00%

Node 61 receives HELLO messages from node 97 that report (amongst others):
1- a  SYMMETRIC   link-type to node 61 (172.29.175.61)
2- an UNSPECIFIED link-type to node 61 (172.29.175.61)

Clearly, this is 'confusing' and the root cause of why node 61 reports
infinite costs for the links, as shown above.

On node 61 - the receiving side - an entry with link type SYMMETRIC is
processed, but then later overwritten with UNSPECIFIED from the duplicate
entry.

For now we fix this problem by letting the sending side order the link
type statuses differently. In a future follow-up patch we'll fix both
the sending and receiving sides more thoroughly.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
Signed-off-by: Henning Rogge <henning.rogge@fkie.fraunhofer.de>

changelog: add a remark about updated plugin versions

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

txtinfo: bump version

Needed because of extensive rework, using the 'info' base code, which
added one or more parameters.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

netjson: bump version

Needed because of extensive rework, using the 'info' base code, which
added one or more parameters.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

jsoninfo: bump version

Needed because of extensive rework, using the 'info' base code, which
added one or more parameters.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

pud: bump version

Needed because of extensive rework, using gpsd

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

nameservice: fix typos of filewrite-interval plugin parameter

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

nameservice: fix plugin versions

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Merge pull request #18 from SvenRoederer/olsr-nameservice-version

nameservice: bump version to 0.4

nameservice: bump version to 0.4

for the new parameter "filewrite-interval" we should bump the version, so init-scripts can give correct parameters.
Automatic generate config-files (like in OpenWrt from UCI-settings) have no knowledge if this parameter can be added to configfile when there is all the same version.

Signed-off-by: Sven Roederer <freifunk@it-solutions.geroedel.de>

Merge pull request #16 from SvenRoederer/rename_sighup-interval

nameservice: rename parameter "sighup_interval" to "filewrite_interval"

nameservice: rename parameter "sighup_interval" to "filewrite_interval"

Playing around with the new option "sighup_interval" I found, that this
also controls the interval of calling the "name-change-script".
Checking the code points out that this is the maintimer for writing
any status-file to disk. So it's more consistent to change the parameter-
name to what it's really doing.

Also update the README to reference this parameter in the others
affected by it.

Signed-off-by: Sven Roederer <freifunk@it-solutions.geroedel.de>

logging: work around bogus Coverity warning

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

cfgparser: use superfasthash for the configuration checksum

Saves space, by request of LEDE/OpenWRT

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

cfgparser: pull in openssl sha256 code to avoid linking to it

Obsoletes/removes the NOCONFIGCHECKSUM build option

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

cfgparser: use correct SHA256_DIGEST_LENGTH

Not SHA_DIGEST_LENGTH, which is for SHA1 checksums

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

Merge pull request #14 from bittorf/master

logging: show interface name during blocked socket error

logging: show interface name during blocked socket error

message was:
OLSR: sendto IPv4 Operation not permitted

message now:
OLSR: sendto IPv4 'Operation not permitted' on interface wlan1

(added also for the IPv6 case)

Signed-off-by: Bastian Bittorf <bittorf@bluebottle.com>

Merge pull request #12 from booo/sighup-interval

nameservice: add new sighup-interval parameter

nameservice: add new sighup-interval parameter

We add the 'sighup-interval' parameter to control the time interval
between sighups to the file specified by 'sighup-pid-file'.

Signed-off-by: Philipp Borgers <borgers@mi.fu-berlin.de>

files: update configuration generation script

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

files: get rid of net-tools usage

net-tools has been deprecated well over a decade now.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

make: remove -Wmissing-include-dirs

It's causing grief for LEDE/OpenWRT and doesn't seem that useful in the
first place.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

gui: clean-ups

Especially the gui olsr_protocol.h file was REALLY out of sync.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>

make: strip CC

Windows can't handle CC with leading/trailing whitespace.
Fixes msys/mingw build on win32.

Signed-off-by: Thorsten Maerz <info@netztorte.de>
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>