nameservice: remove some dead code

Coverity:
CID 739633 (#1 of 1): Logically dead code (DEADCODE)
  Execution cannot reach this expression "127" inside statement
  "tmp->len = ((len_of_name > ...".
  (The condition "len_of_name > 127U" cannot be true.)

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

nameservice: do not close an fd that is not open

Coverity:
CID 739644 (#1 of 1): Dereference after null check (FORWARD_NULL)At (11): Passing null pointer "add_hosts" to function "fclose(FILE *)", which dereferences it.

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

nameservice: check return value of fcntl

Coverity:
CID 739618 (#1 of 1): Unchecked return value (CHECKED_RETURN)
At (4): Calling function "fcntl(fd, 4, 1)" without checking return value.
        This library function may fail and return an error code.
At (5): No check of the return value of "fcntl(fd, 4, 1)".

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

nameservice: fix wrong size argument

Coverity:
CID 739689 (#1 of 1): Wrong sizeof argument (SIZEOF_MISMATCH)
  Passing argument "ip" of type "union olsr_ip_addr *" and argument
  "8UL /* sizeof (ip) */" to function "memset(void *, int, size_t)" is
  suspicious. Did you intend to use "sizeof(*ip)" instead of "sizeof (ip)"?

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

nameservice: fix a resource leak

Coverity:
CID 739678 (#1 of 1): Resource leak (RESOURCE_LEAK)
At (24): Handle variable "fd" going out of scope leaks the handle.

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

secure: initialise cmsg correctly

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

secure: decrease a (stack allocated) buffer size

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

secure: do not hard-code a sizeof()

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

secure: can only work for IPv4, so assert on it

Because of:
  memcpy(&cmsg.destination, new_host, olsr_cnf->ipsize);
with
  cmsg.destination being an uint32_t

secure: initialise crmsg correctly

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

secure: decrease a (stack allocated) buffer size

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

secure: use sizeof() of the fields being copied

secure: decrease a (stack allocated) buffer size

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

secure: do not hard-code a sizeof()

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

secure: can only work for IPv4, so assert on it

Because of:
  memcpy(&crmsg.destination, to, olsr_cnf->ipsize);
with
  crmsg.destination being an uint32_t

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

secure: initialise rrmsg correctly

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

secure: decrease a (stack allocated) buffer size

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

secure: use sizeof() of the fields being copied

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

secure: decrease a (stack allocated) buffer size

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

secure: do not hard-code a sizeof()

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

secure: can only work for IPv4, so assert on it

Because of:
  memcpy(&rrmsg.destination, to, olsr_cnf->ipsize);
with
  rrmsg.destination being an uint32_t

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

httpdinfo: fix resource leak

Coverity:
CID 739676 (#1 of 1): Resource leak (RESOURCE_LEAK)
At (18): Handle variable "client_socket" going out of scope leaks the
handle.

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

httpinfo: do not close an 'error' socket return value

Coverity:
CID 739658 (#1 of 1): Argument cannot be negative (NEGATIVE_RETURNS)
At (5): "client_socket" is passed to a parameter that cannot be negative.

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

bmf: do not dereference NULL pointer

Coverity:
CID 739667 (#1 of 1): Dereference null return value (NULL_RETURNS)
At (31): Dereferencing a null pointer "bestIntf".

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

bmf: do not close an 'error' socket return value

Coverity:
CID 739657 (#1 of 1): Argument cannot be negative (NEGATIVE_RETURNS)
At (12): "listeningSkfd" is passed to a parameter that cannot be negative.

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

bmf: explicitly cast fgetc result into a char

Coverity:
CID 739615 (#1 of 1): Truncated stdio return value (CHAR_IO)
Assigning the return value of "fgetc(FILE *)" to char "EthTapSpoofState"
truncates its value.

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

Fix scanf() format vulnerabilities

Close file descriptor if error happens

    Coverity:

    CID 739686 (#1 of 3): Resource leak (RESOURCE_LEAK)
    At (8): Handle variable "fd" going out of scope leaks the handle.

    CID 739686: Resource leak (RESOURCE_LEAK)
    At (12): Handle variable "fd" going out of scope leaks the handle.

    CID 739686: Resource leak (RESOURCE_LEAK)
    At (14): Handle variable "fd" going out of scope leaks the handle.

Signed-off-by: Henning Rogge <henning.rogge@fkie.fraunhofer.de>

Check inet_pton return value

    Coverity:

    CID 739622 (#1 of 1): Unchecked return value (CHECKED_RETURN)
    At (8): Calling function "inet_pton(int, char const * restrict, void * restrict)" without checking return value (as is done elsewhere 22 out of 25 times).

Signed-off-by: Henning Rogge <henning.rogge@fkie.fraunhofer.de>

Use strscpy instead of strncpy.

    Coverity:

    CID 739613 (#1 of 1): Buffer not null terminated (BUFFER_SIZE_WARNING)
    At (6): Calling strncpy with a maximum size argument of 16 bytes on destination array "t->if_name" of size 16 bytes might leave the destination string unterminated.

    CID 739614 (#1 of 3): Buffer not null terminated (BUFFER_SIZE_WARNING)
    At (4): Calling strncpy with a maximum size argument of 16 bytes on destination array "p4.name" of size 16 bytes might leave the destination string unterminated.

    CID 739614 (#2 of 3): Buffer not null terminated (BUFFER_SIZE_WARNING)
    At (4): Calling strncpy with a maximum size argument of 16 bytes on destination array "p6.name" of size 16 bytes might leave the destination string unterminated.

    CID 739614 (#3 of 3): Buffer not null terminated (BUFFER_SIZE_WARNING)
    At (5): Calling strncpy with a maximum size argument of 16 bytes on destination array "ifr.ifr_ifrn.ifrn_name" of size 16 bytes might leave the destination string unterminated.

Signed-off-by: Henning Rogge <henning.rogge@fkie.fraunhofer.de>

Merge branch 'master' of http://olsr.org/git/olsrd

sgwdynspeed: fix resource leak and file stat/use race

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

CID 739697 (#1 of 1): Time of check time of use (TOCTOU)
At (1): Calling function "stat(char const * restrict,
                               struct stat * restrict)"
        to perform check on "fileName".
At (4): Calling function "fopen(char const * restrict,
                                char const * restrict)"
        that uses "fileName" after a check function. This can cause a
        time-of-check, time-of-use race condition.

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

pud: fix checking of downlink message IP version

Coverity:
CID 739637 (#1 of 1): Logically dead code (DEADCODE)
Execution cannot reach this expression ""false"" inside statement
"pudError(false, "Received w...".

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

pud: fix closing fd

Coverity:
CID 739704 (#1 of 1): Argument cannot be negative (NEGATIVE_RETURNS)
At (7): "fd" is passed to a parameter that cannot be negative.

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

pud: nmealib: fix invalid format string conversion

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

pud: nmealib: fix invalid format string conversion

Coverity:
CID 739671 (#1 of 1): Printf format string issue (PW.BAD_PRINTF_FORMAT_STRING)
At (1): invalid format string conversion

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

add a COVERITY instructions text file

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

Close socket if bind() fails.

    Coverity:

    CID 739685 (#1 of 1): Resource leak (RESOURCE_LEAK)
    At (8): Handle variable "sock" going out of scope leaks the handle.

Signed-off-by: Henning Rogge <henning.rogge@fkie.fraunhofer.de>

Protect against stupid data from ACPI

    Coverity:

    CID 739643 (#1 of 1): Division or modulo by zero (DIVIDE_BY_ZERO)
    At (29): In expression "bat_val * 100 / bat_max", division by expression "bat_max" which may be zero has undefined behavior.

Signed-off-by: Henning Rogge <henning.rogge@fkie.fraunhofer.de>

Convert allocated memory into temporary variable

    Coverity:

    CID 739683 (#1 of 1): Resource leak (RESOURCE_LEAK)
    At (14): Variable "net_msg" going out of scope leaks the storage it points to.

Signed-off-by: Henning Rogge <henning.rogge@fkie.fraunhofer.de>

fix wrong sizeof() parameter

    Coverity:

    CID 739610 (#1 of 1): Incorrect sizeof expression (BAD_SIZEOF)
    Taking the size of pointer parameter "lower_border_ip" is suspicious.

    CID 739611 (#1 of 1): Incorrect sizeof expression (BAD_SIZEOF)
    Taking the size of pointer parameter "upper_border_ip" is suspicious.

    CID 739692 (#1 of 2): Wrong sizeof argument (SIZEOF_MISMATCH)
    Passing argument "lower_border_ip" of type "union olsr_ip_addr *" and argument "8UL /* sizeof (lower_border_ip) */" to function "memset(void *, int, size_t)" is suspicious. Did you intend to use "sizeof(*lower_border_ip)" instead of "sizeof (lower_border_ip)" ?

    CID 739692 (#2 of 2): Wrong sizeof argument (SIZEOF_MISMATCH)
    Passing argument "upper_border_ip" of type "union olsr_ip_addr *" and argument "8UL /* sizeof (upper_border_ip) */" to function "memset(void *, int, size_t)" is suspicious. Did you intend to use "sizeof(*upper_border_ip)" instead of "sizeof (upper_border_ip)" ?

Signed-off-by: Henning Rogge <henning.rogge@fkie.fraunhofer.de>

fix control flow if kill() command fails to end program

    Coverity:
    CID 739665 (#1 of 1): Argument cannot be negative (NEGATIVE_RETURNS)
    At (17): "ifp->olsr_socket" is passed to a parameter that cannot be negative.

Signed-off-by: Henning Rogge <henning.rogge@fkie.fraunhofer.de>

pud: nmealib: fix invalid format string conversion

Coverity:
CID 739674 (#1 of 1): Printf format string issue (PW.BAD_PRINTF_FORMAT_STRING)
At (1): invalid format string conversion

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

pud: nmealib: fix buffer overrun

Coverity:
CID 739669 (#1 of 1): Out-of-bounds write (OVERRUN)
At (11): Overrunning array of 64 bytes at byte offset 64 by dereferencing
         pointer "psSatInfo".

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

pud: nmealib: fix buffer overrun

Coverity:
CID 739668 (#1 of 1): Out-of-bounds write (OVERRUN)
At (10): Overrunning array of 64 bytes at byte offset 64 by dereferencing
         pointer "psSatPrn".

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

pud: nmealib: fix copy/paste error

Coverity:
CID 739627 (#1 of 1): Copy-paste error (COPY_PASTE_ERROR)
"lat" in "pos->lat" looks like a copy-paste error.
Should it say "lon" instead?

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

pud: fix getBestUplinkGateway

Coverity:
CID 739645 (#1 of 1): Explicit null dereferenced (FORWARD_NULL)
At (18): Passing null pointer "&gw_best->originator.v4" to function
         "ip4cmp(struct in_addr const *, struct in_addr const *)", which
         dereferences it.

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

pud: fix position file stat/use race

Coverity:
CID 739696 (#1 of 1): Time of check time of use (TOCTOU)
At (1): Calling function "stat(char const * restrict,
                               struct stat * restrict)"
        to perform check on "fileName".
At (4): Calling function "fopen(char const * restrict,
                                char const * restrict)"
        that uses "fileName" after a check function. This can cause a
        time-of-check, time-of-use race condition.

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

pud: fix socket error

Coverity:
CID 739663 (#2 of 2): Argument cannot be negative (NEGATIVE_RETURNS)
At (8): "fd" is passed to a parameter that cannot be negative.

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

pud: fix socket error

Coverity:
CID 739662 (#2 of 2): Argument cannot be negative (NEGATIVE_RETURNS)
At (9): "fd" is passed to a parameter that cannot be negative.

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

pud: fix resource leak

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

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

common: fix varargs cleanup

Coverity:
CID 739703 (#1 of 1): Missing varargs init or cleanup (VARARGS)
At (4): va_end was not called for "ap2".

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

improve VERBOSE handling

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

android: gcc 4.6 comes with NDK r8b and is not in NDK r8

android: update to use gcc 4.6, available starting with NDK r8

Moved regex library to a makefile of its own to solve the issues with compiler warnings.

bsd: fixup comment

to make it the same as in the release-0.6.4 branch

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

Fix slightly insulting error message.

chose the proper #define in order to take the right #include
dependencies

Add VERBOSE flag to makefile to display full gcc commands

Add -Werror=format-security
Add -O2 -D_FORTIFY_SOURCE=2 for non-debug builds (unless overwritten)

Fix buffer overflow in mdns plugin

add gcc hardening flags to linux-gtk GUI

mdns: fix a compilation warning / copy-paste error

src/RouterElection.c: In function ‘electTimer’:
src/RouterElection.c:177:17: warning: ‘tmp’ may be used uninitialized in this function [-Wuninitialized]

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

mdns: fix compilation warnings

src/mdns.c: In function ‘isInFilteredList’:
src/mdns.c:406:7: warning: passing argument 2 of ‘olsr_ip_to_string’ from incompatible pointer type [enabled by default]
In file included from src/mdns.c:65:0:
../../src/ipcalc.h:146:1: note: expected ‘const union olsr_ip_addr *’ but argument is of type ‘struct in_addr *’
src/mdns.c:406:7: warning: passing argument 2 of ‘olsr_ip_to_string’ from incompatible pointer type [enabled by default]
In file included from src/mdns.c:65:0:
../../src/ipcalc.h:146:1: note: expected ‘const union olsr_ip_addr *’ but argument is of type ‘struct in_addr *’
src/mdns.c:411:7: warning: passing argument 2 of ‘olsr_ip_to_string’ from incompatible pointer type [enabled by default]
In file included from src/mdns.c:65:0:
../../src/ipcalc.h:146:1: note: expected ‘const union olsr_ip_addr *’ but argument is of type ‘struct in6_addr *’
src/mdns.c:411:7: warning: passing argument 2 of ‘olsr_ip_to_string’ from incompatible pointer type [enabled by default]
In file included from src/mdns.c:65:0:
../../src/ipcalc.h:146:1: note: expected ‘const union olsr_ip_addr *’ but argument is of type ‘struct in6_addr *’

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

gateway: merge the olsr_get_ipv{4,6}_inet_gateway functions

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

remove most '#if 0' blocks

Leave mpr related blocks alone.

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

Merge remote-tracking branch 'origin/mDNS-plugin-GSoC-2012'

fixed misspelling: usefull -> useful

fix misspelling: 'adress' --> 'address'

    - an 'address' represents a location, i.e. MAC address

    - 'adress' is a verb meaning "to speak to"

olsrd-adhoc-setup: configure NetworkManager to be nice when adhoc is manually setup

olsrd-adhoc-setup: use static BSSID by default to assoc. with the right mesh

Fix optional compilation of the nl80211 lq plugin

Signed-off-by: Frank de Brabander <brabander@fox-it.com>

Incorporate fix by Ferry Hubert for float to double promotion

Signed-off-by: Frank de Brabander <brabander@fox-it.com>

Merge branch with new lq plugin for NL80211 support

Signed-off-by: Frank de Brabander <brabander@fox-it.com>

gateway: rename 2 variables

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

olsrd_plugin: remove #if 1

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

update comments on all ifdefs

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

ifdef: fix some include ifdefs

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

use the Microsoft standard '_WIN32' macro instead of the self-defined 'WIN32'

http://msdn.microsoft.com/en-us/library/ff540443.aspx

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

http_info: pud satellites in use are now green, not in use are red

To improve readability

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

pud: include nmealib v1.0.3

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

pud: force pending buffer out if not enough space for our message

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

Pud: Do not use the variable name strlen.

gateway: merge 2 code blocks

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

gateway: prevent 2 NULL dereferences

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

gateway: the gateway to set must either be an IPv4 or IPv6 gateway or both

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

gateway: the gateway to set must be in the gateway tree

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

gateway: merge 2 code blocks

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

gateway: push an if condition into its nested if's

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

gateway: simplify a condition

Because of the preceding
  if (ipv4) {
    current_ipv4_gw = NULL;
  }
  if (ipv6) {
    current_ipv6_gw = NULL;
  }

The condition(s)
   if (ipvX && entry != current_ipvX_gw
Are effectively
   if (ipvX && entry != NULL
Also, the test
   entry != NULL
was already performed by the enclosing if statement

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

gateway: rename a function

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

gateway: inline olsr_trigger_inetgw_selection function

To simplify and reduce the number of functions

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

gateway: inline olsr_find_gateway_entry function

To simplify and reduce the number of functions

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

gateway: add comments to struct olsr_gw_handler function prototypes

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

gateway: rename hooks

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

gateway: add init and cleanup hooks for gateway plugins

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

gateway: clear gw_handler during cleanup

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

gateway: add asserts to validate gw_handler

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