ipconfig: Catch interface name changes

Update the ipdevice interface name if it is changed by udev.

agent: Fixes refcounting issue

Since service is referenced already in the message queue, there is no need
to reference it also in the callback data: this might lead to an over refcount
issue if the queue message is canceled before receiving any answer to it.
(callback's data refcount is still there).

proxy: Set lookup to NULL if the tokens did not match

wispr: Add backpointer to wispr portal structure

When calling free_connman_wispr_portal_context(), the context is
freed. Add a backpointer to the wispr portal context in order to
clear the context from the wispr_portal struct.

gweb: Don't use debug functionality after possible free

When the callback has been called, the whole structure might be
freed. Thus don't call the debug function tied to the structure.

tethering: Fix error path in __connman_tethering_set_enabled()

In some cases we did not destroy the bridge and ip pool.

tethering: Re-add error message about missing bridge functionality

Commit 4fe07df removed an error message about missing bridge functionality.
Return it back.

tethering: Fix bridge module loading problem

Connman tries to check if bridging is supported by the kernel by checking
whether "/proc/sys/net/bridge" is present. If the bridge is a kernel
module which is not already loaded, then tethering cannot be enabled.

Instead of checking for the file we invoke the "get bridge version" socket
ioctl and the kernel will handle module loading issues - if the bridge
module is not loaded, it will first load it, and then check the version.

technology: Fix memory leak as the tethering strings were not freed

config: Call g_clear_error to prevent memory leak

test: set-timeservers was missing from makefile

agent: Always clean up and unref on agent shutdown

agent: Cancel agent requests

Clean up any queued Agent API messages when the agent disappears or
the service is disconnected. Send cancel to the agent on D-Bus
timeout.

agent: Implement Agent API Cancel() method call

Clean up the queue and cancel the current message at the agent, if
any. Split out the actual cancel message sending as it will be used
separately from the rest of the code.

agent: Use the Agent queueing functions when sending requests

agent: Implement Agent API message queueing

Add functions for queuing Agent API messages and sending the
first one when there is no message processed by the agent.

Handle D-Bus pending call replies and simplify the D-Bus reply
processing by handing the message to the callback function.

Use g_slist_prepend() where appropriate

Convert usage of g_slist_append() to g_slist_prepend() where
appropriate. gdbus, dnsproxy, resolver, rtnl, session and session
unit test have ordering requirements and thus not touched.

session: Do not track disconnect reason

All session stay in free ride mode all the time. We still
have to know if we have called connect on a service. Therefore
we need to distiguish between reason CONNECT and FREE_RIDE.

session: Reorder D-Bus documentation

Reorder and group the settings, first the read only values
and then the ones which can be changed.

session: Move RoamingPolicy to configuration plugin

The RoamingPolicy setting which should be set by the platform integrator
and not by the application developper is moved the configuration plugin.

session: Remove RoamingPolicy documetation

Remove this setting from the D-Bus API and move to the configuration
plugin.

session: Move EmergencyCall to configuration plugin

Instead fallback using the configuration plugin for selecting
the emergency application.

Now we don't have to update all other sessions which makes
the whole code logic more straight forward. If a session is
marked as an emergency call and this application calls
Connect()/Disconnect() it will succedd and block all other
sessions when they try to call Connect() or Disconnect().

session: Remove EmergencyCall documentation

The EmergencyCall setting which should be set by the platform integrator
and not by the application developer is moved the configuration plugin.

session: Remove StayConnected implementation

session: Remove StayConnected documentation

This setting is implicitly already available through AutoConnect.
Having yet another way to express this doesn't make it work any better.

session: Move Priority to configuration plugin

The Priority setting which should be set by the platform integrator
and not by the application developer is moved the configuration plugin.

session: Remove Priority documentation

Remove this setting from the D-Bus API and move it to the
configuration plugin.

session: Add configuration plugin

session: Remove SessionMarker implementation

session: Remove SessionMarker documentation

SO_MARK can only be set if the application has the CAP_NET_ADMIN
capability. We don't want to give that to all application for
good reasons and thus this approach won't work. Also touching all
applications to get this feature working is not a good idea. We
are going to use a cgroup controller to set SO_MARK and therefore
we don't need it in the Session API.

session: Remove PeriodicConnect implementation

session: Remove PeriodicConnect documentation

The idea was to let ConnMan know and optimize the connection
scheduling. Doing this per Session is a bit over-engineered.
The only use cases that have come up so far was for things like an
email client checking for new message.

Past experience showed that this kind of interface lead easily to
live locks. If there is need for such a feature, doing it on a per
technology would make more sense. For the time beeing it will be
removed. Later it could still be added to the Technology API.

If an application still wants to have a connection on regular
basis it can call the Session Connnect() method. ConnMan is still
able to defer any connection request.

session: Update ConnectionType documentation

The ConnectionType configuration option will be moved to
the config plugin when we know how the application can be
uniquely identified in a persistent way.

session: Remove IdleTimeout implementation

session: Move IdleTimeout to Technology API

Having an IdleTimeout per Session is not really reasonable. It is
hard to define the right behavoir if the timeouts differ per Session.

Doing this per technology is easier to understand and also makes
sense when no Session is used.

IdleTimeout is only documented here, it's not yet implemented.

session: Update overview doc

Update to the new D-Bus API and the simplified connection
algorithms (state).

service: Do not report error twice

In request_input_cb(), __connman_service_indicate_error() unconditionally
reports an error via agent after determing the service state. The error
reporting and user connectable state will be properly handled by
service_indicate_state().

inet: Properly remove IPv4 address and set interface down

The interface was not properly set down and thus the services
were not removed.

Fixes BMC#25659

gdbus: Fix compilation error if macro "error" is defined

The variable "signature" used in error is not defined and "args" is now
a struct instead of a string.

pptp: Fix PPTP option typo

Fixes BMC#25660

pptp: Get VPN gateway resolved IP address

We should not use the Host variable from provider as it can
contain a non-resolved address.

Fixes BMC#25658

l2tp: Get VPN gateway resolved IP address

We should not use the Host variable from provider as it can
contain a non-resolved address.

provider: Return VPN gateway IP address

If 'provider->host_ip[0]' is set, it contains the
resolved IP address. If not set, the 'Host' property
already contains an IP address.

vpn: Accept also "true" or "1" as a boolean value

Currently user is able to set the boolean VPN config value
only by setting the variable to "yes". This is not very
intuitive so accept also the value of "true" or "1".

device: Refactor device scan function

In wifi plugin this means that we combine three wifi scan
functions (normal, fast, hidden) into one scan function.

Release 1.6

doc: Manual page for connman configuration file

doc: Manual page for connman

git: Do not ignore connman manual pages

service: Always zero the timeout id when timer triggers

The timer id was not cleared in case the signal creation failed.

service: Send updated properties after ServicesChanged

Updated service properties are not sent before the new service
has been announced in a 'ServicesChanged' signal.

Fixes BMC#21285

wifi: Ignore duplicate hidden APs

We must not add duplicate entries to list of hidden AP.
There can be more that one service file for each AP if
we have multiple wifi cards in the system.

Fixes BMC#24779

wifi: Always Scan hidden APs

We should scan hidden AP even if the AP is not marked
as autoconnectable so that the user is able to connect
to it manually.

wispr: Fix refcounting issue

service: Restore error clearing on Agent reply

If the invalid key error is not cleared on Agent reply, the
service cannot be reconnected.

service: Allow connection if there are interfaces available

Allow user to initiate connection if there are multiple devices
of the same technology type available (like having multiple wifi
cards).

inet: Clear IP addresses when interface is set down

Clear interface IPv4 address by setting it to 0.0.0.0. IPv6
addresses are cleared automatically when the IFF_DYNAMIC flag is
set at the time the interface is brought down.

By removing the IP address and netmask, netlink properly reports
new addresses assigned by DHCPv4.

service: Clear service error on successfull connection attempts

Move clearing of the error to __connman_service_connect() so it is
cleared for all service connect code paths.

ipconfig: Use correct ipconfig ipaddress for FIXED method

Use ipconfig->address when reporting IP addresses for services
with FIXED method. AUTO method is used only by IPv6, don't report
any addresses should it erroneously appear for IPv4.

Fixes BMC#25084

technology: Move SetProperty to async method call

Handle properly -EINPROGRESS error when enabling/disabling technology.
This fixes the existing timeout-based implementation.

technology: Refactor enable/disable technology

- Cleanup the code (remove useless vars, remove duplicate code)
- Separate dbus logic to feature logic
- Fixes issue:
"dbus.exceptions.DBusException: net.connman.Error.Failed: Success"

dundee: Fix error code handling in add_device()

If no error happens then we should just return and not run
into the cleanup code. And when we cleanup we should not double
free pointers.

bluetooth: Add the device into hash table when relevant

Fixes BMC#25215

timezone: Add comment

timezone: If there is no d_type support use fstatat()

This is useful for filesystems where d_type is always DT_UNKNOWN,
for example reiserfs, jffs2 and ubifs. In this case use the
fstatat() function.

Release 1.5

provider: Return EINPROGRESS to the caller when connect is pending

service: Remember provider pending dbus message

The provider connect dbus reply message is stored in service
until provider is connected.

service: Allow setting path when sending pending dbus reply

connection: Disconnect VPN when underlaying service disconnects

Fixes BMC#25128

inet: Add function to return route to a given host

The netlink message handling is modified from iproute.c in iproute2 package.

wifi: Try only twice when connecting to a non-favorite service

service: Add getter for favorite flag

iptables: Load table at the right places

Using -j/-m options without -t one, will segfault due to table not loaded
before hand.

iptables: Refactor default return code usage

tools: Refactor where table should be loaded

Fixes segfault issues when using a non-builtin target without providing table name.
like: -A OUTPUT -j IDLETIMER (...)
will crash when: -t filter -A OUTPUT -j IDLETIMER (...) will success

service: Disconnect provider after connection timeout

The provider was not disconnected after a connection timeout.
This lead to weird issues in VPN code; like following provider
connect request would disconnect the earlier VPN connection and
report EALREADY to the user.

doc: Improve description of service state property

Add missing "disconnect" state and point to doc/overview-api.txt for
more information about state transitions.

dundee Create ipconfig before setting index

Since connman_network_set_index() does not create magically the
ipconfig for us anymore, we need to create it inside set_connected().

bluetooth: Defer add_networks() if adapter is not powered yet

In the case we add a new adapter and the adapter is powered off
we would ignore the reported devices. Later when the adapter was
powered on it would just not add those networks.

Instead silently ignoring the devices, store the D-Bus paths in
a hash table and process them as soon the adapter is powered.

Fixes BMC#25322

bluetooth: Remove unused argument in check_networks()

bluetooth: Remove unused argument in add_network()

ofono: Set IPv6 nameservers

network: Enable IPv6 on network interface

This patch enables IPv6 usage for network interfaces
with static/manual configuration.

ipconfig: Allow NULL gateway for IPv6

This patch makes possible configuring a GPRS connection with
IPV6 settings and without gateway.

ofono: Create ipconfig before setting index

Since connman_network_set_index() does not create magically the
ipconfig for us anymore, we need to create it inside set_connected().

network: Only set index in connman_network_set_index()

Service ipconfig structures are no longer created in this
function.

service: Allow plugins to lookup services by networks

service: Allow plugins access to connman_service_create_ip4/6config

technology: Save tethering identifier and passphrase

technology: Rename save_state function by technology_save

dundee: Clean up device in case of setup failure

If the creation and registration of Bluetooth DUN device fails we need to
clean up before return, otherwise we leak memory.

bluetooth: Add device to hash before registration

During the connman_device_register() procedure a lookup to the
bluetooth_devices hash table happens, however the device is not on the
hash at this point and the look out fails.

If the registration fails, technology_disable() returns the Failed
message on D-Bus with the error status zero. That happens because we
don't have any device registered.

This patch moves the insertion of the device to before the device
registration.

dhcp: Apply received hostname option according to conf file setting

We do not use the received DHCP hostname option (12) if system
config file setting says so.

Fixes BMC#25597

main: Introduce conf file option to disable hostname updates

The AllowHostnameUpdates in system config file can be used
to ignore the hostname update request that for example
can be received via DHCP hostname option.

provider: Discard routes to VPN server via VPN tunnel

These obsolete routes must be discarded as we cannot
contact VPN server via VPN tunnel. If the VPN server
address is not yet resolved, we do that while starting
the VPN client process.

Example of non working routes:
Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
0.0.0.0         0.0.0.0         0.0.0.0         UG    0      0        0 vpn0
10.10.15.0      0.0.0.0         255.255.255.0   U     0      0        0 wlan0
10.10.15.1      0.0.0.0         255.255.255.255 UH    0      0        0 wlan0
10.252.112.0    0.0.0.0         255.255.240.0   U     0      0        0 vpn0
193.229.0.40    10.10.15.1      255.255.255.255 UGH   0      0        0 wlan0
193.229.0.42    10.10.15.1      255.255.255.255 UGH   0      0        0 wlan0
113.108.211.91  0.0.0.0         255.255.255.255 UH    0      0        0 vpn0
113.108.211.91  10.10.15.1      255.255.255.255 UGH   0      0        0 wlan0

The correct routes should be like this:
Kernel IP routing table
Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
0.0.0.0         0.0.0.0         0.0.0.0         U     0      0        0 vpn0
10.10.15.0      0.0.0.0         255.255.255.0   U     0      0        0 wlan0
10.10.15.1      0.0.0.0         255.255.255.255 UH    0      0        0 wlan0
10.252.112.0    0.0.0.0         255.255.240.0   U     0      0        0 vpn0
193.229.0.40    10.10.15.1      255.255.255.255 UGH   0      0        0 wlan0
193.229.0.42    10.10.15.1      255.255.255.255 UGH   0      0        0 wlan0
113.108.211.91  10.10.15.1      255.255.255.255 UGH   0      0        0 wlan0

provider: Allow VPN connection if not in connected state

Allow user to connect to VPN via dbus and supplying all
VPN parameters if the provider already exists but the VPN
is not in connected state.

This means that the user can call the Manager.ConnectProvider interface
multiple times to connect to VPN service. This patch is needed so that
OpenConnect can work correctly. Without this patch the OpenConnect cookie
cannot be passed to openconnect client program.

connection: Set VPN default route to the VPN interface

Do not bother setting individual routes for default gateway so
for VPN we set the whole interface as a default gateway route.

connection: Add more debugging when setting VPN routes

service: Signal error property changes

In order to allow clients to update the property in a manner similar to
other service properties, signal changes in the service "Error" property
key/value pair.

The value "" is used when no error is asserted because D-Bus property
changed convention has no semantics for indicating a key/value pair was
removed.