vpn-provider: Setting VPN properties will send PropertyChanged signal

doc: Add description of GetProperties method

test: Script for getting, setting and clearing VPN properties

vpn-provider: Set the state to FAILURE after auth error

vpn-provider: Go internally into IDLE after FAILURE

Do not stay in FAILURE state as clients like connmand can
get confused about our current state. Symptom for the problem
is that after a failed connect attempt, a new connect attempt
by the client would cause immediately a failure return code
because vpnd is still in FAILURE state. The actual connect
attempt might still succeed but then the client would not
know about the success status.
By setting the state to IDLE we avoid this confusion.

vpn: Ignore VPN UserRoutes and ServerRoutes properties

These properties are not needed here so ignore them.

error: Handle ECONNREFUSED gracefully in dbus error handler

pptp: Set the username/password before starting daemon

This is required in order to avoid free memory access that
is happening if we call vpn_provider_set_string() with same
string that is already in the settings db.

l2tp: Set the username/password before starting daemon

This is required in order to avoid free memory access that
is happening if we call vpn_provider_set_string() with same
string that is already in the settings db.

vpn-provider: Make state debug print more useful

Print the state value as a string as it is useful information.

vpn-provider: Avoid printing password to log files

Add a variant to vpn_provider_set_string() that does not print
sensitive data like password in clear text to log files.

l2tp: Clear password if authentication fails

This allows plugin to query agent so that user can give
correct password.

pptp: Clear password if authentication fails

This allows plugin to query agent so that user can give
new password.

Fixes BMC#25963

main: Create VPN_STORAGEDIR when starting up

Eventually all VPN directories from STORAGEDIR is to be
migrated into VPN_STORAGEDIR

TODO: Add note about removal of L2TP and PPTP prefix for PPP options

vpn-provider: Add extra whitespace

vpn-provider: Add route support in vpn config file

vpn-provider: Remove unprovisioned providers at startup

Check if there are any providers that were provisioned
but their .config file is removed. If such providers are found,
then remove the provider files from file system.

vpn-provider: Type string in provider needs to be in lower case

vpn-config: Provision providers from .config file

vpn-provider: Initial .config file support

storage: Add function to load provider configuration file

doc: VPN config file specification

pptp: Use PPPD prefix for pppd specific options

For backward compatibility purposes, support also the PPTP prefix
for PPPD options.

l2tp: Use PPPD prefix for pppd specific options

For backward compatibility purposes, support also the L2TP prefix
for PPPD options.

l2tp: Add rx and tx bps pppd options

l2tp: Fixed the nodeflate pppd option name

config: Read only wifi config

config: Remove obsolete definition

iptables: Cannot flush all rules without API to set them

Currently there exists no API where iptables rules can be set. The flush
code does not touch the default chain policy at the moment. Any pre-
existing iptables rules setting default policy to reject and relying on
individual iptables rules allowing packets going through will prevent
all IP communication. Removing all iptables rules on startup can expose
the device to unwanted traffic as well.

For the time being disable iptables flush on init. Please be careful
with iptables rules and the masquerading ones ConnMan sets when
tethering.

gresolv: Optimize the response parser

It will check first if the response belongs to a query,
before interpreting any of its content (rcode, count...).

gresolv: Do not remove a query on failure if other results are pending

Fixes BMC#25973

In the case one of the resolving failed, the query is removed and
destroyed from the queue. So the responses of the requests sent to
the other namerservers - which might be successful - will thus be
lost since they cannot be matched anymore to their initial request.

gsupplicant: Return zero for max scan SSID parameter

A driver can return a valid max scan SSID value of zero. Thus no fast
scans can be done, so the code falls back to a simple scan instead.

A value of zero is properly handled in plugins/wifi.c. An active scan
for a hidden SSID adds only the SSID parameter to the wpa_supplicant
D-Bus method call, which wpa_supplicant then handles properly.

Some drivers also report a max scan SSID value of one. In some of the
cases that value is bogus, the driver will not be able to do a fast
scan anyway. In addition, it is questionable why only one SSID can be
fast scanned as the feature would not differ much from an active scan
for a hidden network. Thus we set the limit to two, i.e. zero or one
is treated as zero, two or more is reported as is.

Thanks to Grant Erickson and Tomasz Bursztyka for finding and
pinpointing this issue.

Fixes BMC#25971

gdbus: Add g_dbus_proxy_set_removed_watch

unit: Fix wrong format identifier for size_t

unit: Add test cases for SHA-1 based PBKDF2 and PRF

shared: Add helpers for SHA-1 based HMAC, PBKDF2 and PRF

shared: Add generic debug and hexdump helpers

gdbus: Fix missing PropertiesChanged signal

If D-Bus ObjectManager is not supported, InterfacesAdded signal
checking needs to be ignored otherwise PropertiesChanged signal
will never be sent.

systemd: Force daemon restart on failure

Important if you have a headless system.

iptables: Update the hook entries correctly in iptables_delete_rule()

The builtin value is only valid in the chain head entry and not
in any other entry. That means we need to lookup the head entry
and use that builtin value (== hook id) and then update all
references which follow that chain.

iptables: Test if match and target arguments are also the same

We need to verify that also the arguments are the same, e.g.
if we have two rules like

-t filter -A INPUT -m mark --mark 1 -j LOG
-t filter -A INPUT -m mark --mark 2 -j LOG

then the matcher and the target would be the same without looking
at '1' or '2'.

When deleting a rule, we would always remove the first
rule which matches the 'match' type and target type, so let's have a
look also on the arguments. iptables does it the same way.

test-iptables: Add chain add remove tests

iptables: Fix fallthrough rules

test-iptables: Add fallthrough unit test

A fallthrough rule is a one which does not have a verdict, e.g.
ACCEPT, DROP etc.

iptables: Remove unused __connman_iptables_command()

nat: Use new iptables API

test-nat: Move tests to test-iptables

These are iptables related tests. So let's move them over.

test-iptables: Add unit test for the new API

iptables: Add split out iptables commands

Instead of having a pure string based API, we add two new
main functions, __connman_iptables_append() and
__connman_iptables_remove(). The missing commands will be added later.

To simplify the whole code, the __connman_iptables_command() code
is refactored into smaller pieces: parse_rule_spec() calls a few
functions such as clear_tables_flags() and parse_xt_modules()
which should make the reading of the main parser loop simpler.

Also added a few comments on the parser which is really tricky.

nat: No need to 'nat' table anymore

This is done in iptables.c directly.

iptables: Flush 'filter' 'mangle' and 'nat' table

The implementation is ugly but there is not much we can do about it, the
iptables API is being just plain stupid here.

iptables: Add CONNMAN_IPTABLES_DEBUG environment variable

If CONNMAN_IPTABLES_DEBUG is set, then print the table when it is loaded
and print it the table which will be written.

Also use DBG() instead of connman_info().

iptables: Remove dependency on table in iterator_entries_cb_t

With removing the table argument the callback can now either
iterate over buffer we get from the kernel or the one we write
to the kernel.

iptables: Do not pass table into dump_match() and dump_target()

iptables: Get rid of the iterator macro

Instead implement the iterator loop directly. Since both dump_entry()
and add_entry() have calculated 'builtin' and 'offset' let's pass in
them in as well.

In the next step we are able to remove also the table argument
which will allow us to unify the parsing the table we get
from IPT_SO_GET_ENTRIES and the table we will pass in to the kernel
via IPT_SO_SET_REPLACE.

iptables: Clear global parser flags

That allows the parser to be reentrantable.

iptables: Make string arguments const

iptables: Add some documentation

These are some random notes but should give the next person to debug
iptables some introduction.

test-iptables: Add unit test for iptables

gdbus: Don't call property changed callback during client init

When the client uses ObjectManager to init properties, do not call
property changed callbacks. They should only be called once the proxy
added has been successfully signaled since the proxy itself provides
a full copy of available properties.

gdhcp: DHCP server IP address is stored in host order

This change affects unicasted DHCP renew and release messages.
A DHCP server receiving packets via a raw socket will get all
packets destined to port 67. If the DHCP server checks the
intended server IP address, the request will fail. If the server
does not care about the IP address being used, it will send a
reply. The reply is not recognized by ConnMan since it is coming
from another IP address than what it was sent to. ConnMan will
retry quite a few times, but eventually settle down and use the
address even without a proper response from the server.

ipconfig: Use proper address structure when appending IPv4 info

service: Validate the IP addresses before use

When setting IP address information via D-Bus, first create a new ipconfig
structure to hold the values. If the values were validated correctly, update
the service ipconfig structure with the new values. In order to achieve
this, refactor the existing code.

Fixes BMC#25930

service: Factor out ipconfig creation

The caller has to set the created ipconfig into its data structures.

ipconfig: Add checks for invalid IPv4/IPv6 addresses

Also fine tune returned errors.

test: Fix simple-agent so it can run even if service files are missing

Fixes BMC#25964

ipconfig: Do not overwrite gateway address

If we have already received gateway address via newroute rtnl
message, then do not overwrite it when receiving newaddr message.

Fixes BMC#25931

gdbus: Fix unpublished interface signal emission

If we haven't published an interface yet (i.e. it's in the data->added
list), we should just ignore any property changed indications as the
values for the properties will anyway be part of the InterfacesAdded
signal.

Release 1.11

dundee: Set network available is needed only when scanning

ofono: Set network available is needed only when scanning

ethernet: Set network available is needed only when scanning

README: Add kernel options needed for tethering

TODO: Update TODO with Bluez 4.x removal task

bluetooth: Implement tethering for the Bluez 5 bluetooth plugin

technology: Support multiple technology drivers

As tethering is enabled on the technology level, both the bluetooth and
the bluetooth_legacy plugin need to register technology drivers for
CONNMAN_SERVICE_TYPE_BLUETOOTH. Modify the technology code to create
a list of registered technology drivers instead of a single technology
driver pointer.

bluetooth: Connect and disconnect a PAN NAP network

Implement connect and disconnect functionality for a ConnMan network.
On network creation check the connection status of the Bluez 5 Network1
object and set the network to connected when needed.

service: Must not return NULL with a synchronous GDBus method call

If NULL is returned, GDBus will call the registered function repeatedly.

bluetooth: Expose Bluez 5 org.bluez.Network1 objects as networks

Create ConnMan networks for those Bluez 5 org.bluez.Network1/Device1
objects that support PAN NAP. A network is created or removed in response
to the Network1/Device1 being created or when the Device1 object's UUID
changed.

The ConnMan network struct is added to and removed from the ConnMan
device when the device is created, enabled or disabled.

network: Allow more than one network driver register the same type

Allowing more than one network driver to register the same network type
requires the network driver to properly identify which networks belong
to it in the network driver probe function.

bluetooth_legacy: Check network in probe function

The network probed can belong to the new Bluez 5 bluetooth plugin or to
this plugin. Check if this plugin created the network and proceed
accordingly.

bluetooth: Keep track of Bluez 5 org.bluez.Network1 objects

Set up functions monitoring Bluez 5 org.bluez.Network1 objects being
added and removed. Add a GDBusProxy for the org.bluez.Device1 object
that corresponds to the Network1 object. Set up functions monitoring
Network1 'Connected' and Device1 'UUID' properties.

bluetooth: Expose Bluez 5 adapters as ConnMan devices

Set up functions monitoring Bluez 5 objects being added and removed using
GDBusProxy. When an adapter appears, create a new device for it and set up
a watch function for the adapter's 'Powered' property.

Keep the adapter 'Powered' property in sync with the device state and if
they differ reset the adapter to the device state. Also update the Bluez 5
adapter 'Powered' property when the device is enabled or disabled.

technology: Enable/disable individual devices also with rfkill

In order to keep ConnMan devices in sync with Bluz 5 adapters, the
individual devices need to be enabled/disabled also when
unblocking/blocking them with rfkill. Thus enable devices after
unblocking and disable devices before blocking with rfkill.

bluetooth: Register technology driver

bluetooth: Add new bluetooth plugin

bluetooth: Rename bluetooth plugin to bluetooth_legacy

Set the plugin priority to CONNMAN_PLUGIN_PRIORITY_LOW and the technology
priority to -10. Update strings to "bluetooth_legacy" and use ordinary
string instead of ident.

technology: Check technology driver probe function before calling

technology: Remove technology driver by pointer

Compare the technology driver pointer to the driver being unregistered
as the function is supposed to remove only the given driver.

Also check if the driver has a remove function before calling it.

technology: Remove unnecessary comparison and goto

ofono: Use ordinary string instead of ident

The string is only used for debugging purposes.

dundee: Use ordinary string instead of ident

The string is only used for debugging purposes.

network: Unset driver on failed match

gresolv: Destroy query at the relevant place when parsing the response

parse_response() will eventually call sort_and_return_results() which in turn
will call the result function. But the result function might cancel the gresolv.
At that point all queries belonging to this gresolv are destroyed. Returning back
to parse_response(), it calls again destroy_query() on an already destroyed one.
Thus leading to a crash.

Same issue with query_timeout()

Reported by Daniel Wagner

service: Notify leaving online state when downgrading it to ready

Fixes BMC#25862

ipconfig: IPv6 was enabled too early

When enabling IPv6 ipconfig, we enable kernel IPv6 support too early.
The ipconfig might get unreffed which will disable ipconfig and
thus disable kernel IPv6 support. By moving kernel IPv6 enabling
after the ipconfig unref call, we make sure that IPv6 kernel support
is properly enabled.
This is related to fix in commit d479904ecaa2bd9

gdbus: Check for valid path before handling properties

dnsproxy: DNS response answer count was incorrectly set

We set the answer count in host byte order instead of network
byte order when sending cached AAAA record. This problem is only
seen in special case when an AAAA record is generated by us when
there is IPv4 address in cache but no IPv6 address for the host.

network: Do not report any error when removing a connecting network

When connecting a network, if it's related technology is disabled, this network
will raise an error through the agent. Leading to a possible retry request from
the user, then to a crash if the user does so.
Thus, we prevent any error to be raised in such situation and silently disconnect
the connecting network instead.