ntp: Add debug message when starting NTP against resolved server

This patch adds an analagous debug message when starting NTP against a
resolved server as when starting against a numbered server.

ntp: Fixed a debug message typo

ntp: Retry a given server address multiple times before falling back

This patch causes the internal NTP client to retry a given server
address up to three times (four total attempts) at a two second interval
before falling back to another server name or server address.

ntp: Use mnemonic when specifying the request precision

This patch uses a mnemonic to specify the request precision and also
adds a comment how a future implementation might pass along the actual
system wall clock precision.

ntp: Add mnemonics for common clock precisions

ntp: Use mnemonics when checking reply flags

Use mnemonics when checking reply flags and print out information in
debug mode about why the reply flags were rejected.

ntp: Specify leap-not-in-sync in flags

This patch sets the non-in-sync flag of the leap subfield of the packet
flags field. This reflects behavior implemented by ntpdate or ntpd in
one-shot mode.

ntp: Send a transmit time value that reflects current system time

This sets the ntp request packet transmit time to the current system
time to reflect behavior implemented by ntpdate and ntpd in one-shot
mode.

ntp: Use mnemonics rather than magic numbers

Convert a magic number into equivalent mnemonics for the flags (leap,
version, mode) packet field.

ntp: Add mnemonics for the flags (leap, version, mode) packet field

timeserver: Try all resolved timeserver IP addresses

If we get more than one IP address, then add those
server addresses to the time server list so that they
can be tried if the first entry fails.

ntp: If the server cannot be contacted, try next one

This could happen if the network routes are not yet setup
so the sendto() could fail. In this case try next address and
hope for the best. This is a rare issue but I have seen it
couple of times.

Revert "build: Use -Wshadow compiler option"

build: Use -Wshadow compiler option

dnsproxy: Avoid multiple shadowing variables

tethering: Avoid shadowing connection variable

session: Avoid shadowing hash table iterator

session: Avoid shadowing session info variable

config: Avoid shadowing config structure variable

counter: Avoid shadowing connection variable

notifier: Avoid shadowing connected variable

agent: Avoid shadowing connection variable

service: Avoid shadowing string variable

task: Avoid shadowing connection variable

plugins: Avoid shadowing connection variable

gsupplicant: Avoid shadowing key variable

unit: Avoid shadowing session info variable

tools: Avoid shadowing buffer variable

gdhcp: Avoid shadowing address string variable

service: Gateway IP address property is not sent

The address of the IP gateway is not sent when the IP
property changes. This was because of timing issues i.e.,
the gateway was not yet set when the other IP properties
were sent.

Fixes BMC#25471

wifi: Debug print fixed

Convert a warn message to debug one because it is printed
too often to be usefull as a warning message.

wifi: Scan all hidden and provisioned AP

config: Get configurations that are provisioned

We need the list of provisioned services so that
all the hidden ones can be scanned.

config: Read hidden AP information from config file

doc: Hidden AP provisioning information added

TODO: Remove task about wpspin as PreviousPassphrase type

doc: Add necessary documentation about wpspin as a PreviousPassphrase type

agent: Handle wpspin type as a PreviousPassphrase

agent: Rewrite how PreviousPassphrase field is handled

This is a necessary rewrite to make it clearer and simpler, before implementing
wpspin type support.

device: Turn off all running interfaces at startup

Turning off running interfaces that have IP address,
causes all routes related to those interfaces to be
removed by kernel. This way connman will get a fresh
start without any extra and obsolete routes around.

inet: Return all running interfaces

Add a function that returns all (except loopback)
interfaces that are up and running (have IP address).
This information is used in following patch to clean
up the routes that are left hanging around.

service: Make sure the pending reply is not lost

The user data (pending reply) can be null if agent returns
an error. In this case do not overwrite the real pending reply.

dhcp: warn on invalid netmask

main: Dynamically allocate default blacklist array, free on exit

main: Move default string arrays top of the file

technology: check validity of given properties for tethering

A WPA2 passphrase is between 8 and 63 characters,
a SSID name is between 1 and 32 characters.

technology: return already enabled when tethering is enabled

TODO: Remove BackgroundScanning task

wifi: Add support for autoscan request

gsupplicant: Add a function to handle autoscan method

TODO: Add what wifi legacy support should be removed about set_regdom

wifi: Add capability to set regulatory domain through device's interface

Fixes BMC#25244

gsupplicant: Add a method to set interface's country property

gsupplicant: Adapt set country callback to return a result

This is necessary for the next coming patches, to fix country setting by interface

technology: Set regulatory domain by device or fallback to technology based

device: Add capability to notify on regulatory domain setting

technology: Notify the regulatory domain setting by device

device: Add driver capability to set regulatory domain on device basis

doc: Fix small typo

Release 1.3

rtnl: Fixed log for the type RTM_GETADDR

The connman prints an unknown type when sending a request RTM_GETADDR

tethering: Fix incorrect index value for private network

In __connman_private_network_request, pn->fd is used as the index value
and passed to __connman_ippool_create, which is not correct.
It result in the private network is block used externally.

unit: Watch for signals only on CONNMAN_SERVICE

ofono: Watch for signals only on OFONO_SERVICE

dundee: Watch for signals only on DUNDEE_SERVICE

bluetooth: Watch for signals only on BLUEZ_SERVICE

gdbus: Fix removal of filter after last filter_data

If there's a signal watch that's also watching for name
(data->name_watch) currently we are trying to remove the message_filter
twice since we may have the following call chain:

filter_data_remove_callback()
  filter_data_free()
    g_dbus_remove_watch()
      filter_data_remove_callback()
filter_data_free()
        dbus_connection_remove_filter()
  dbus_connection_remove_filter()

Because of this we can't currently watch for signals passing the bus
name. After this patch we don't have this issue anymore.

We fix it by removing the filter before calling filter_data_free() if we
are the last filter_data and thus avoid calling
dbus_connection_remove_filter() twice.

ofono: Fix potential crash

If a dummy context is created, it might happen
that its index is equal to -1. In this particular case
the IP address has not been copied to context->address
during ipv4 extraction.
If we force from oFono a context activation, it will
make ConnMan crashing on set_connected().

inet: Fix usage of getaddrinfo() return value

Fixes BMC#25304

wifi: Add wifi pointer NULL checks

Always check what connman_device_get_data() returns and act
accordingly.

main: static-qualify default config values

The default configuration arrays default_auto_connect and
default_blacklist must be static-qualified. Otherwise, their
stack-allocated storage will go out of scope after parse_config
is finished and empty or commented-out DefaultAutoConnectTechnologies
or NetworkInterfaceBlacklist config entries will refer to
random stack data.

TODO: Add WPS pin code task

TODO: Remove DUN task, update WiFi scan owner

Release 1.2

gweb: Use g_try_realloc instead of g_realloc

doc: Document 'informational' value of Requirement field

agent: Fix how to provide PreviousPassphrase through D-Bus

ipconfig: Save prefixlen only if not cleared

This commit fixes a bug when using manual ipv4 addressing.
The first time it works, but once the service is reconnected it uses
an invalid netmask: 255.255.255.255.

wifi: Check we have valid wifi pointer in autoscan

The wifi pointer in device might be NULL if wpa_supplicant dies
between autoscans.

technology: Return proper error code when already disabled

Return AlreadyDisabled to the caller instead of InProgress if we
are already disabled.

technology: Return proper error code when already enabled

We do not call the generic error message function because
it maps EALREADY to EINPROGRESS. So instead we call correct
error return function which returns AlreadyEnabled to the
caller.

Fixes BMC#25306

main.conf: Comment out example default values and add documentation

main: Use 'NetworkInterfaceBlacklist' string everywhere

rtnl: Use filtered devices to determine blacklisted devices

Instead of hard-coding some selected devices to be ignored, the
blacklist check now uses the connman_device_isfiltered list.

device: Consider blacklisted interfaces from main.conf when filtering

Interfaces blacklisted in main.conf are now also taken into account
by connman_device_isfiltered. This is done in addition to what gets
passed with -i or -I on the command line.

main: Support NetworkInterfaceBlacklist from main.conf

This adds a settings variable and a function for fetching that list

main.conf: Add NetworkInterfaceBlacklist

dnsproxy: Check cache for TCP request before connecting to server

It is not necessary to create TCP sockets to server if we have
the answer already in the cache.

dnsproxy: Use DNS header in checks

We should use the DNS header struct when checking DNS
protocol values instead of using pointer so that the
TCP offset is properly taken care of.

dnsproxy: Add more debugging prints

dnsproxy: Check cache properly for TCP packets

The DNS packet starts from offset 2 in TCP payload.

dnsproxy: Fix the negative caching of AAAA record

Handle the TCP offset properly.

dnsproxy: Save DNS packet in cache always in TCP format

The cached DNS packet will always contain two prefix bytes.
For UDP we do not send these two extra bytes. This way we do
not need to know if the cached message is from UDP or TCP query.

dnsproxy: Do not generate SIGPIPE

Instead of generating SIGPIPE, just return EPIPE error
if TCP stream is broken. This way we cannot get aborted
if we receive SIGPIPE (seen this happening).

ippool: Fix endless loop issue with 32 prefix length

While trying to establish a DUN connection with oFono,
an endless loop was found when requesting an IP block.
The problem was on data connection activation its address
with 32 length prefix was notified through ConnMann using
__connman_ippool_newaddr() but mask address shifting with
32 bits was obsolete. So IP pool was considerating block
0.0.0.0 to 255.255.255.255 was in use.

dnsproxy: Fix flushing pending requests

dnsproxy: Release all the fields of the request after forward dns reply

dnsproxy: Avoid duplication of domains in the TCP server domain list

The domain list is created in the tcp_server_event function
at connection time (G_IO_OUT event), so it is not necessary
to create it here too.

dnsproxy: If no request was sent then the TCP server is destroyed

dnsproxy: If the answer is in the cache, the TCP request is released