ofono: Do not call set_connected() from the activation reply

As the IP setting may not be set when getting the activation D-Bus reply,
set_connected should not be called at that point but rather from
the context_changed routine.

Fixes BMC#24284

tethering: Avoid removing dhcp server twice.

The tethering_dhcp_server pointer needs to be cleared as
otherwise the __connman_tethering_cleanup() will call it again.

wifi: The remove interface callback used stale pointer.

When a wlan interface is removed we call proper method in
supplicant dbus interface. As the return message from supplicant
might come very late, we must not use the wifi pointer as
user data in interface remove callback. This is because the wifi
pointer might be removed before the answer comes from supplicant and
the callback user data pointer would not be valid at that point.

wifi: Add debugging in order to find memory problems.

ipconfig: Remove ipconfig from ipconfig_list

It is possible that ipdevice does not contain ipconfig for
some reason. If that happens, then ipconfig is not removed
from ipconfig_list which will then escalate into freed memory
access when interface is taken down.
The problem was seen with valgrind when tethering was enabled
and then disabled for wifi.

6to4: Use the right interface index with gweb

wispr: Use the right interface index with gweb

gweb: Bind socket to the right interface

gresolv: Bind socket to the right interface

technology: Save the pending reply only when the status is inprogress

For ethernet device, there is no need to save a pending reply as
the technology enablement is synchronous.

Fixes BMC#24249

service: Support automatically added nameservers

IPv6 autoconfigured nameservers can be added to resolver
via netlink messages in rtnl.c. Because of this they are
not seen in service object so we need to get those auto
added nameserver to be added in service.c so that
service can show them to user if necessary.

Fixes BMC#24196

storage: Add default values to settings file.

On a clean start (without default.profile) create the settings file
with default values. Here is a list of default values:

Offlinemode = off.
Wifi = disabled.
bluetooth = disabled.
ethernet = disabled.
3G = disabled.
WiMAX = disabled.

The reason for offlinemode to be off here is that any changes to
the technology will not be recorded if we start in offlinemode.

Fixes BMC#24285

dnsproxy: Do not copy two extra bytes if using TCP

dnsproxy: Get rid of extra strlen() calls

dnsproxy: One extra garbage byte was sent at the end of the message

Fixes BMC#24253

service: Fix memory leak in service_load()

service: Set network_created flag only if create_hidden_wifi succeeds

ofono: Don't set network index when device is removed

A NULL interface means the device is removed so there is no need for
setting the network index as it will eventually be removed.
Setting the network index to -1 prevents wpad or wispr to unreference
the service.

Fixes BMC#23979

service: network_created is set only for hidden services

Fixes BMC#23976

plugins: Don't attempt to save strings with value NULL

If a string with value NULL is set with g_key_file_set_string(),
a critical glib error will be printed.

network: connman_network_set_nameservers() handle NULL argument correctly

Trying to access an array pointer which points to NULL is generally a
bad idea. Instead just return early.

notifier: Remove service from hash in error case

CONNMAN_SERVICE_STATE_FAILURE should be treated in the same
way as CONNMAN_SERVICE_STATE_DISCONNECT or CONNMAN_SERVICE_STATE_IDLE
in  __connman_notifier_service_state_changed().

This should help to reduce the "Service state machine inconsistency detected."
error message.

network: Call indicate error function in error case

Setting the error only on one ipconfig type e.g. IPv4 is not enough.
service.c:combine_state() will report to stay in associating state:

src/service.c:service_indicate_state() service 0x6b1a60 old association - new failure/association => association

After applying the patch it loosk like this:

src/service.c:service_indicate_state() service 0x6b1f70 old association - new failure/association => association
src/service.c:service_indicate_state() service 0x6b1f70 old association - new failure/failure => failure

l2tp: Config save support added

pptp: Config save support added

todo: Mark l2tp and pptp as done

task: Make sure the process is eventually killed

If a task refuses to kill itself, then wait two secs before
trying again and if that does not help then forcefully kill it.

readme: Add l2tp and pptp information

pptp: Do not put password in command line options

pptp: Return authentication errors properly

l2tp: Check authentication error properly

scripts: Report authentication error to connman

The ppp plugin now checks ppp authentication error and report it
to correct connman plugin.

scripts: Compile libppp-plugin.so properly

pptp: Add pptp makefile and configure file

l2tp: Add l2tp makefile and configure file

test: Add test script support for pptp/l2tp

Modify connect-vpn script to handle pptp/l2tp connection.

l2tp: Add l2tp vpn support

Initial revision.

pptp: Add ppp plugin library

pppd will use this library to talk with connman vpn plugin.

Original patch comes from Mohamed Abbas. Prepared for inclusion
by Jukka Rissanen, also changed the pptp function prefix to ppp as
the plugin is used by both pptp and l2tp plugins.

pptp: Add pptp vpn support

Initial revision.

task: Allow vpn plugins to send reply

Change task notify to allow client to send dbus reply. This
will allow vpn plugin to send requested user/password info.

vpn: Add null checks

vpn: Add support to allow ppp tunnelling

pptp and l2tp does not use tun/tab approach so change the vpn.c to
allow vpn daemon to create the interface. Now vpn plugin tell
vpn to create the device or leave to vpn client.
On connection vpn will get the interface name set the index
for the provider.

This patch is prepared by Jukka Rissanen and it contains
Mohamed Abbas original patch + fixes suggested by Daniel Wagner.
Jukka fixed memory leak in vpn_set_ifname().

tethering: Use already existing 'tether' bridge interface

When creating bridge interface 'tether', don't report error if
it already exists. Check also -EALREADY given by connman_inet_ifup()
via enable_bridge() if the bridge interface was already up.

Fixes BMC#23931

iptables: Initiate defined table before working on it

tools: Initiate defined table before working on it in iptables-test

build: Create src/builtin.h via BUILT_SOURCES

network: Fix failed to reconnect to 3G network when connect failed

When connect failed, set the associating value to false again, else
when we connect again the return err will be EALREADY.

bluetooth: Cleanup if technology driver register fails

rfkill.h: Remove unused header file

gdhcp: Use gcc atomics instead glib's ones

g_atomic_int_exchange_and_add() has been removed from glib 2.30
and g_atomic_int_add() should be used. Though there are still
quite a few distros out which do not ship a glib version with
g_atomic_int_add().

Instead of maintaing a compatiblilty glib layer we just use
the built-in functions for atomic memory access.

gweb: Use gcc atomics instead glib's ones

g_atomic_int_exchange_and_add() has been removed from glib 2.30
and g_atomic_int_add() should be used. Though there are still
quite a few distros out which do not ship a glib version with
g_atomic_int_add().

Instead of maintaing a compatiblilty glib layer we just use
the built-in functions for atomic memory access.

plugins: Use gcc atomics instead glib's ones

g_atomic_int_exchange_and_add() has been removed from glib 2.30
and g_atomic_int_add() should be used. Though there are still
quite a few distros out which do not ship a glib version with
g_atomic_int_add().

Instead of maintaing a compatiblilty glib layer we just use
the built-in functions for atomic memory access.

core: Use gcc atomics instead glib's ones

g_atomic_int_exchange_and_add() has been removed from glib 2.30
and g_atomic_int_add() should be used. Though there are still
quite a few distros out which do not ship a glib version with
g_atomic_int_add().

Instead of maintaing a compatiblilty glib layer we just use
the built-in functions for atomic memory access.

doc: Remove outdate SetupRequired property

The APN provisioning is fully handled by oFono.

test: Connecting a saved provider does not need all arguments

vpnc: Save support for vpnc

Vpnc saves all settings with prefix 'VPNC.'

openvpn: Save support for openvpn

Openvpn saves all settings with prefix 'OpenVPN.'

openconnect: Save support for openconnect VPN

Openconnect saves VPN MTU, Server and CA cert file paths.

vpn: Extend vpn driver struct with save support

Add provider save function and register it in the
vpn driver initialization.

provider: Helper function for fetching the keyfile group

provider: Define a save function for provider drivers

Define a save function for connman provider driver
specific settings. No corresponding load function
is defined since all saved settings are already loaded
by connman_provider_load().

provider: Load and save provider data

On connect, save 'Type', 'Name', 'Host' and 'VPN.Domain' provider
settings. On probe, load saved keys and their values from storage
and add the key/value pairs to the provider struct.

storage: Load and save functions for providers

storage: Trivial change to free pathname in common part of the code

wifi: Only consider identifiers starting with 'wifi_'

storage: Don't list provider settings directories

test: Tiny typo fix in test-session python script

session doc: Document PeriodicConnect and IdleTimeout

session doc: Document state transitions

session doc: Fix typo

wifi: Handle erroneous disconnection after 4way-handshake

And interpret it as a wrong psk after 4 consecutive failures.

gsupplicant: Add a function to enable or disable the selected network

network: Check device pointer when doing IPv6 autoconf

The device can disappear if network is disconnected while setting
autoconf parameters.

network: Disconnect network properly

When network is removed (for example when ethernet cable
is disconnected), then it needs to be disconnected also.
The disconnect was not done at all because of the reference
counting fixes in dhcp code.

inet: Cleanup the router solicitations handler properly

inet: Fix file descriptor leak when comparing subnets

device: Cancel pending trigger when device is removed

connmand[28326]: src/device.c:device_destruct() device 0x6b97c0 name candlejack.bmw-carit.intra-1
connmand[28326]: plugins/bluetooth.c:powered_reply()
connmand[28326]: Name "org.bluez" does not exist
connmand[28326]: src/device.c:device_pending_reset() device 0x6b97c0

test: Use session.Destroy()

Use session.Destroy() instead of the Manager API DestroySession()
method call.

doc: Document that DestroySession will not send Release to sessions

session: Implement Session API Destroy method call

Call session_disconnect() from destroy_session() thus adding
functionality to Session API Destroy method call.

iptables: Fix xtables API version issues

tools: Fix xtables API version issues with iptables-test

gdhcp: Possible file descriptor leak

session: Add a list of pending connect/disconnect actions

deselect_service() and select_offline_service() want to clear
up the session state machinery by setting up a zero second
timeout function thereby letting the main loop run once through
all waiting actions. Unfortunately, if a service is being
removed meantime, the service structure stored in the timeout
functions will become invalid.

This is fixed by adding a list of pending connect/disconnect
actions per struct service_entry and cancelling all of them
if the service is removed.

Fixes BMC#22879

iptables: Support builtin chain policies changes

iptables: Disable getopt unknown option display

iptables: Support for rule matches list

iptables: Finalize matches extensions setup

iptables: Add extensions options parsing for matches

iptables: Finalize target extensions setup

iptables: Add extensions options parsing for target

iptables: Add x6_options matches handling

iptables: Add x6_options target handling

iptables: Move target and matches preparation routines

iptables: Support matches rule list loading

iptables: Refactor matches preparation

iptables: Move target options merge to prepare_target

iptables: Code preparation for option handling

tools: Add iptables_test builtin chain policy changes

tools: Initialize the iptables_test invert flag properly

The invert flag is specific to each option so it needs to be initialized
after each succesfully handled option.