hna_set: explicitly cast to (time_t *) in calls to localtime()

To make it build on win64

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

win32: remove unused getpid() function

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

main: do not seed the random number generator with the process id

We now seed it with the current time. This is predictable as well
but getpid() is not compatible with the windows 64bits mingw build.

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

tas: remove plugin

Unmaintained for a long time, no response on removal announcement,
no known users.

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

gateway: simplify stopping the cleanup timer

To make the code of olsr_delete_gateway_tree_entry consistent with
that of olsr_update_gateway_entry

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

gui: linux: fix the string buffer overflow better

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

gui: linux: fix string buffer overflow

Coverity:
CID 742359 (#1 of 1): Copy into fixed size buffer (STRING_OVERFLOW)
At (2): You might overrun the 10 byte fixed-size string "tmp" by copying
        the return value of "g_locale_to_utf8(gchar const *, gssize,
        gsize *, gsize *, GError **)" without checking the length.

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

gui: linux: fix string buffer overflow

Coverity:
CID 742358 (#1 of 1): Copy into fixed size buffer (STRING_OVERFLOW)
At (1): You might overrun the 255 byte fixed-size string "title" by
        copying the return value of "ip_to_string(union olsr_ip_addr *)"
        without checking the length.

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

gui: linux: fix inccorrect sizeof()

Coverity:
CID 742357 (#1 of 1): Incorrect sizeof expression (BAD_SIZEOF)
Taking the size of arithmetic expression "512 + 1" is suspicious.

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

gui: linux: fix some 'shadow' compiler warnings

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

gui: linux: fix some 'may be used uninitialized' compiler warnings

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

Revert "gui: linux: fix some 'may be used uninitialized' compiler warnings"

This reverts commit b27a04dd3106a7123810c066d59362ec6e7a7c64.

gui: linux: fix some 'unused parameter' compiler warnings

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

gui: linux: fix a 'mixed declarations and code' compiler warning

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

gui: linux: fix 'too few arguments to function ipc_connect' compiler warning

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

gui: linux: fix some 'old-style function definition' compiler warnings

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

gui: linux: fix some 'discards const qualifier' compiler warnings

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

gui: linux: fix some 'noreturn' compiler warnings

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

gui: linux: use toplevel make options

But disable some warnings for now.

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

gui: linux: fix some 'may be used uninitialized' compiler warnings

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

gui: linux: add VERBOSE handling

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

gui: linux: clean and install are phony targets

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

gui: linux: minor Makefile fixup

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

Makefile: always pass VERBOSE to the make command

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

Makefile: add gui target

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

Makefile: fixup some forgotten make command prefixes

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

win32: use #ifdef _WIN32

I'm really tired of seeing (windows) compiler errors in Eclipse
on my Linux machine...

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

Merge tag 'v0.6.4'

OLSRd release 0.6.4

* tag 'v0.6.4':
  Update version to 0.6.4
  Fix slightly insulting error message.
  on os x we need to include the proper files in order to have IPv6_PKTINFO
  pud: include nmealib v1.0.3
  pud: force pending buffer out if not enough space for our message
  Pud: Do not use the variable name strlen.

gateway: refactor olsr_set_inet_gateway

To make it easier to understand

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

hna_set: olsr_ip_prefix_to_string outputs a wider string

See 213f9e04ab44d89c0138b0283883eebe2aabc9a5

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

gateway: minor improvement in NULL comparisons

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

gateway: rename a variable

And add a comment

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

gateway: only remove a tunnel when there is one

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

use INET_ADDRSTRLEN and INET6_ADDRSTRLEN instead of magic constants

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

neighbor_table: stub out olsr_print_neighbor_table on #ifdef NODEBUG and fix

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

neighbor_table: stub out olsr_print_neighbor_table on #ifndef NODEBUG

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

hna_set: stub out olsr_print_hna_set on #ifdef NODEBUG and fix

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

hna_set: stub out olsr_print_hna_set on #ifndef NODEBUG

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

two_hop_neighbor_table: stub out olsr_print_two_hop_neighbor_table on #ifdef NODEBUG

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

tc_set: stub out olsr_print_tc_table on #ifdef NODEBUG

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

routing_table: stub out olsr_print_routing_table on #ifdef NODEBUG

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

mpr_set: stub out olsr_print_mpr_set on #ifdef NODEBUG

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

link_set: stub out olsr_print_link_set on #ifdef NODEBUG

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

duplicate_set: stub out olsr_print_duplicate_table on #ifdef NODEBUG

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

gateway: stub out olsr_print_gateway_entries on #ifdef NODEBUG

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

gateway: add a define for the tunnel name

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

gateway: rename gw_mem_cookie to gateway_entry_mem_cookie

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

gateway: improve cleanup

First remove all inactive gateways from the gateway tree
to avoid getting a gateway re-election. Then remove the active
gateways.

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

Update version to 0.6.4

gateway: minor optimisation

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

Remove dead codepath

CID 739641: Logically dead code (DEADCODE)
Execution cannot reach this expression "NULL" inside statement "err = olsr_new_netlink_rout...".

reset link timer if not set

CID 739651: Dereference after null check (FORWARD_NULL)
At (6): Passing "entry" to function "olsr_process_hysteresis(struct link_entry *)", which dereferences null "entry->link_timer"

no hostroute possible in this if() branch

CID 739641: Logically dead code (DEADCODE)
Execution cannot reach this expression "NULL" inside statement "err = olsr_new_netlink_rout...".

Interface will always be found.

CID 739650: Dereference after null check (FORWARD_NULL)
At (10): Dereferencing null pointer "cfg_inter".

Declare olsr_exit() as noreturn

CID 739654: Dereference after null check (FORWARD_NULL)
At (7): Dereferencing null pointer "node".

fail of non-blocking netlink socket is not fatal

CID 739623: Unchecked return value (CHECKED_RETURN)
At (24): Calling function "fcntl(olsr_cnf->rtnl_s, 4, 2048)" without checking return value. This library function may fail and return an error code.

Fail in IPC initializationis always fatal.

CID 739624 (#1 of 1): Unchecked return value (CHECKED_RETURN)
At (34): Calling function "ipc_init()" without checking return value. It wraps a library function that may fail and return an error code.

Fix command line parsing macro

CID 739642: Logically dead code (DEADCODE)
Execution cannot reach this statement "fprintf(stderr, "Error pars...".

httpinfo: add a comment about why the client socket is not closed

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

Revert "httpdinfo: fix resource leak"

This reverts commit 808928830ed65a3d8de80876d00ad78a8cfe540b.

It breaks the httpinfo page serving

cookie: add an assert

Coverity doesn't understand the olsr_exit

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

cookie: an array already is a pointer

Coverity:
CID 739655 (#1 of 1): Dereference after null check (FORWARD_NULL)
At (8): Passing null pointer "&branding->cmb_sig" to function
        "memcpy(void * restrict, void const * restrict, size_t)", which
        dereferences it.

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

Merge branch 'lua-official-502'

* lua-official-502:
  tas: lua: run gnu-indent.sh script
  use the POSIX standard '__linux__' macro instead of 'linux'
  Patch ALIGN_NOWARN_ARM macro for better priority handling.
  ARM alignment warnings fixed
  Fix sytax errs and line break quirks for gnu-indent
  * added -Wcast-qual: this killed lots of unnecessary type-casts
  * added -Wbad-function-cast
  cleanup, 1. step (to get some local changes away):
  x86_64 patch from Bernd
  tas: lua: official source code for 5.0.2

tas: lua: sync up to official source code with fixes

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

tas: lua: run gnu-indent.sh script

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

use the POSIX standard '__linux__' macro instead of 'linux'

"Use __linux__ for this purpose, not linux. Although the latter is defined, it is not POSIX compliant."
    http://www.faqs.org/docs/Linux-HOWTO/GCC-HOWTO.html#INDEX.25

from: 485c4d5ca8ad403b27afd39d3016f9741f28867a

Patch ALIGN_NOWARN_ARM macro for better priority handling.

from: 7fce805df297d920e5c65e537f610fa7da75bab5

ARM alignment warnings fixed

from: d1656d8112d5605cd709c211960ab71b34a49bed

Fix sytax errs and line break quirks for gnu-indent

from: 0e3c43820a333bea7863e08208de22d77dc828aa

corrected gnu-indent quirks

from: 29d2799ba478fc0354596ff132128e873454bb20

* added -Wcast-qual: this killed lots of unnecessary type-casts

from: 17271941ffaa174ee45959effee041831b489d00

* added -Wbad-function-cast

from: ee7eeb6d7f7b5a23830402c6b346d8db93995d41

cleanup, 1. step (to get some local changes away):

from: 6ff6c1cc6464a9ca6ac1d3961fc044960a402e5b

tas: lua: run gnu-indent.sh script

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

x86_64 patch from Bernd

from: eda03ea1f7554d8bc24c6443b7a405d0ed84e9a0

tas: lua: official source code for 5.0.2

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

defs: fix strscpy null pointer dereference

Coverity:
CID 739649 (#1 of 1): Dereference after null check (FORWARD_NULL)
At (5): Passing null pointer "dest" to function
        "strncpy(char * restrict, char const * restrict, size_t)", which
        dereferences it.

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

linux: sanitise ACPI battery data

Coverity:
CID 739695 (#1 of 1): Use of untrusted scalar value (TAINTED_SCALAR)
At (23): Using tainted variable "bat_max" as the divisor in
         "bat_val * 100 / bat_max".

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

gateway: walk the gateway tree to properly cleanup

No need to explicitly remove the tunnels, since these are
removed by the olsr_delete_gateway_entry function.

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

gateway: split olsr_delete_gateway_entry

The olsr_delete_gateway_entry function now is a wrapper around
the olsr_delete_gateway_tree_entry function. The latter function
will be used by the cleanup function later on, to prevent needless
iterations on the gateway tree.

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

gateway: allow immediate removal of a gateway entry

This is going to be used by the cleanup function later on.

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

gateway: minor simplification in olsr_delete_gateway_entry

Do an early return if the gw is NULL

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

gateway: ensure cleanup uses the reverse order of init

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

gateway: cleanup: free the allocated memory cookie

This was forgotten

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

gateway: initialise the handler before setting up the tunnel

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

pud: fix alignment warning

src/netTools.c: In function 'getIPv4Address':
src/netTools.c:102:11: warning: cast increases required alignment of target type [-Wcast-align]

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

android: fix type punning warning

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

lq: make sure last_ip != NULL

Coverity:
CID 739653 (#1 of 1): Explicit null dereferenced (FORWARD_NULL)
At (11): Passing null pointer "last_ip" to function
         "calculate_border_flag(void *, void *)", which dereferences it.

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

lq: make sure that info_head != NULL

Coverity:
CID 739652 (#1 of 1): Explicit null dereferenced (FORWARD_NULL)
At (88): Dereferencing null pointer "info_head".

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

olsr_switch: fix a memory leak

Coverity:
CID 739687 (#1 of 1): Resource leak (RESOURCE_LEAK)
At (16): Variable "oc" going out of scope leaks the storage it points to.

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

linux: apm: fix retry opening of apm proc file

Coverity:
CID 739684 (#1 of 1): Resource leak (RESOURCE_LEAK)
At (6): Variable "apm_procfile" going out of scope leaks the storage it
        points to.

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

cfgparser: fix dereference before assert

Coverity:
CID 739688 (#1 of 1): Dereference before null check (REVERSE_INULL)
Null-checking "in->cnf" suggests that it may be null, but it has already
been dereferenced on all paths leading to the check.

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

cfgparser: fix a copy/paste error and resulting memory leak

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

cfgparser: fix some memory leaks

Coverity:
CID 739682 (#1 of 2): Resource leak (RESOURCE_LEAK)
At (84): Variable "in" going out of scope leaks the storage it points to.
CID 739682 (#2 of 2): Resource leak (RESOURCE_LEAK)
At (89): Variable "in" going out of scope leaks the storage it points to.

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

main: fix an Out-of-bounds access

Coverity:
CID 739670 (#1 of 1): Out-of-bounds access (OVERRUN)
At (46): Overrunning struct type in_addr of 4 bytes by passing it to a
         function which accesses it at byte offset 15 using argument
         "16UL".

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

txtinfo: remove some dead code

Coverity:
CID 739639 (#1 of 1): Logically dead code (DEADCODE)
  Execution cannot reach this expression "97" inside statement
  "v4 = (autoV4 ? 97 : 115);".
CID 739640 (#1 of 1): Logically dead code (DEADCODE)
  Execution cannot reach this expression "97" inside statement
  "v6 = (autoV6 ? 97 : 115);".

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

tas: fix a memory leak

Coverity:
CID 739681 (#1 of 1): Resource leak (RESOURCE_LEAK)
At (21): Variable "walker" going out of scope leaks the storage it points to.
At (22): Variable "mess" going out of scope leaks the storage it points to.

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

quagga: explicitly ignore return value of fcntl

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

pud: remove some dead code

Coverity:
CID 739635 (#1 of 1): Logically dead code (DEADCODE)
  Execution cannot reach this expression ""disable"" inside statement
  "pudError(true, "Could not %...".

CID 739636: Logically dead code (DEADCODE)
  Execution cannot reach this expression ""enable"" inside statement
  "pudError(true, "Could not %...".

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

bmf: do not close an fd that is not open

CID 739656 (#1 of 4): Argument cannot be negative (NEGATIVE_RETURNS)
At (7): "encapsulatingSkfd" is passed to a parameter that cannot be
        negative.
CID 739656 (#2 of 4): Argument cannot be negative (NEGATIVE_RETURNS)
At (9): "encapsulatingSkfd" is passed to a parameter that cannot be
        negative.
CID 739656 (#3 of 4): Argument cannot be negative (NEGATIVE_RETURNS)
At (12): "capturingSkfd" is passed to a parameter that cannot be
         negative.
CID 739656 (#4 of 4): Argument cannot be negative (NEGATIVE_RETURNS)
At (11): "encapsulatingSkfd" is passed to a parameter that cannot be
         negative.

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

p2pd: remove some dead code

Coverity:
CID 739634 (#1 of 1): Dead default in switch (DEADCODE)
  Execution cannot reach this statement "default:".

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