connection: Remove original default route only when needed

If the new VPN gateway has split routing set, then do not
clear original default route because VPN will not set the
default route. Without this check we would not get any
proper default route set.

service: Return service split routing status

connection: Update service list order

This is done as we might have changed the order of the services
so force update of the service list.

service: Add function to update the service list order

service: Save/load split routing information for VPN

service: Allow user to do split routing for VPN services

The split routing is done when user moves the services and it is only
done if there are routes defined for that specific VPN provider.

provider: Check if there are any routes for the provider

provider: Allow user to set routes when creating provider

provider: Save and load user specified routes

provider: Add support for user defined routes

Allow user to add routes when setting up VPN. This is useful if
the VPN cannot be configured to setup additional routes itself.

update Gustavo's email address

The old one is not valid anymore.

ofono: Fix NULL pointer access in set_disconnect()

oFono sends several ConnectionContext.PropertyChanged("Active", False)
after it was told to disconnect.

After studying the oFono part, it looks like that serveral CREGS updates
could trigger oFono to send us the same flag over and over again
(see ofono_gprs_status_notify()).

Even though oFono might missbehave, ConnMan should not crash so
easily. Luckely, the fix is very simple. We should just check if we still
have a valid network pointer before accessing it.

Fixes BMC#25026

dnsproxy: Refactor request destroy code

Move common code into a function for destroying the request data.

provider: Free provider when destructing it

Valgrind reports memory leak here.

vpn: Unref provider properly

The provider can be unreffed in vpn_died() or in vpn_disconnect().
We must do the unref only once.

dnsproxy: Remove pending requests before destroying the channel

This fixes a segfault in request_timeout when disabling tethering.

gweb: workaround for setsockopt failure

Setting socket option BINDTODEVICE requires CAP_NET_RAW capability.

TODO: Create tasks for main.conf variables

set-ipv6-method: The arguments have to be wrapped into a variant

Fixes BMC#24986

set-ipv4-method: The arguments have to be wrapped into a variant

Fixes BMC#24986

list-services: Print only identifier instead of path

test-connman services also reports the identifier only. This makes
copy&paste with the mouse a bit simpler.

service: Fix refcount leak on ipconfigs

In __connman_service_connect() we call __connman_ipconfig_enable()
which takes a ref on the ipconfig object. Later when the service
state machine is updated service takes another ref.

When the device disapears during operation (the user didn't call
Service.Disconnect()), we do not call __connman_ipconfig_disable()
anymore which would drop one ref.

ipconfig: Disable IPv6 for new interfaces

We don't have yet an ipconfig for this interface, that means
the service.c does not yet know this interface. In order to
be able to get into sync we disable IPv6 auto connection at this point.

If the service state machine decides to connect, it will enable it
again.

ipconfig: Track ipconfigs enabled/disabled state

ipconfig: Do not enable IPv6 ipconfigs when loading the properties

When enabling IPv6 when loading the Service properties, the Service
state machine gets out of sync when AutoConnect is disabled.

At this point we should only load the properties and let the
Service state machine decide when to enable the ipconfigs.
And the good news is that the core handles this already.

service: Enable AutoConnect for new ethernet services

We need also to set the Favorite flag to true in order to let
the services to be auto connected.

service: Load/store AutoConnect and Favorite properties for ethernet

ethernet: Do not connect network on LOWER_UP

Ethernet networks should honor the AutoConnect feature as the rest
of the network types.

Just register the network object at the core when we get LOWER_UP.
Then the core can decide when to connect to the network.

There is no behavior change when AutoConnect is enabled, So if the
user plugs the cable the ethernet network will automatically connect
as it was before.

If AutoConnect is disabled, the core will not connect to the network
automatically.

vpn: Remove wrong include of <glib/ghash.h>

wispr: do not take care about http_proxy

wispr: do not lookup for proxy if service does not provide any

connection: Try to avoid having two default gateways

It is possible that because the rtnl messages arrive
asynchronously, we might end up having two default gateways
that are both active at the same time. If that happens, we try
to downgrade the one that is not yet active.

connection: Remove obsolete routes for VPN

connection: Check all active gateways in update

There can be multiple active gateways in the system at the same
time so we must check them all when updating gateway info.

connection: Get correct VPN phy link data

VPN could be using wrong phy link data if there are multiple
active links.

connection: Do not set host routes in VPN case

These routes are not needed.

connection: Avoid stale memory access

Get the active gateway pointer only after the gateway hash
has been manipulated by add_gateway(). It is possible that
we are accessing stale pointer otherwise.

service: Check redundant default changed notify

Notify default service changed only if we actually changed
the default service.

service: Allow removing only certain type nameservers

When nameserver are removed by __connman_connection_gateway_remove()
then remove only certain type nameserver (IPv4 or IPv6). This is
needed so that we do not loose IPv4 routes if only IPv6 nameservers
are to be removed. This is needed when there are multiple connected
services.

connection: Remove nameserver routes only once

connection: Trigger service updates only after setting gateways

This is needed so that gateways are set properly when service
triggers online checks.

connection: Fix debugging prints

inet: Fix debugging prints

Release 0.79

test: test-session does not exit any more when catching dbus exceptions

Fixes BMC#24964

test: Support the possibility to input multiple parameters into simple-agent

Fixes BMC#24965

test: Add support of hidden service input into simple-agent

ofono: Remove network whenever context is removed

Fixes BMC#24942

ofono: fix modem lookup in cm_context_added()

ipconfig: Fix broken attempt to handle {IPv4,IPv6}.Configuration

service: Fix two small style issues

core: Add extra checks for set property methods

clock: Revert change to accept variant of variant

AUTHORS: Metion Tim's contribution

AUTHORS: Mention Otavio's contribution

ipconfig: Accept variant types in __connman_ipconfig_set_config()

According the D-Bus API, the Service interface accepts
variant types.

clock: Accept variant types in SetProperty

According the D-Bus API, the Clock interface accepts
variant types.

service: Fix ServiceRemoved signals

Removed services hash table need not be consulted when sending
ServicesAdded signals as the service structures have already been
deleted. Also clean up both added and removed hash tables after the
signals have been sent.

inet: Ensure data->channel exist before accessing it

Log of segfault:
...
connmand[676]: src/network.c:check_dhcpv6() reply (nil)
connmand[676]: src/network.c:check_dhcpv6() re-send router solicitation 3
connmand[676]: src/inet.c:__connman_inet_ipv6_send_rs()
connmand[676]: Aborting (signal 11) [connmand]

technology: Enable tech if device is already powered

connman_technology_add_device() calls __connman_device_enable()
but since the device is already enabled, the calls does not propagate
through to __connman_technology_enabled via connman_device_set_powered.

core: Change coding style for passing NULL to time

Pass NULL rather than 0 when calling time(2) to follow prefered
stylistic convention.

dhcpv4-client: Change coding style for passing NULL to time

Pass NULL rather than 0 when calling time(2) to follow prefered
stylistic convention.

gdhcp: set secs in a RFC 951- and 2131-compliant manner

This patch sets the BOOTP secs field in a RFC 951- and 2131-compliant
manner for DHCPv4 DISCOVER and REQUEST/SELECT packets.

Certain DHCP servers, such as that implemented in Mac OS X (< 10.7)
for its "Internet Sharing" feature, refuse to issue a DHCP lease to
clients that have not set a non-zero value in their DISCOVER or
REQUEST packets. In fact, based on http://hints.macworld.com/article.php?
story=20071223001432304, it's not non-zero but a value greater than four (4)
seconds to allow another "authoritative" DHCP server on the subnet to reply
first.

Side-by-side packet analysis of Mac OS X, iOS, Android, ISC and
Windows clients show that these clients set the BOOTP 'secs' field and
are successfully issued a DHCP lease by Mac OS X. By contrast, a
connman-based client will issue 10 back-to-back DISCOVER packets and
will not be returned a DHCP OFFER from the server.

AUTHORS: Mention Mario's contributions

ippool: Fix NULL pointer access

Not all allocated blocks info data structure have a pool associated.

ippool: Fix typo in error message

ofono: Remove obsolete DUN comment

test-connman: Use GetServices() instead GetProperties()

GetProperties() does not return the services path anymore. Instead
the GetServices() method should be used.

list-services: Use GetServices() instead GetProperties()

GetProperties() does not return the services path anymore. Instead
the GetServices() method should be used.

test-connman: Use GetServices() instead GetProperties()

GetProperties() does not return the services path anymore. Instead
the GetServices() method should be used.

service: Keep ordering and send all services in ServicesAdded

Keep services ordering and send also already known services
in ServicesAdded signal. Properties are added for new services,
while the dictionary is left empty for already known ones.

When the service is removed, pack the value of "TRUE" to the
removed service hash in order to indicate a removed object path.
The service struct cannot be used, as it is freed immediately
after returning from service_schedule_removed().

unit: handling type of connection into session api

unit: switching from online to state property into test-session

test: modify test-session to handle a new property

test: fixing type conversion bug in test-session

doc: updated session api documentation

session: filtering session state update according to connection type

session: introducing a property to select a type on which the state gets up on

session: do not notify ipconfig content if it's not in a connected state

service: adding a function to know the connected state of an ipconfig

session: do not notify an ipconfig change if the session is not connected

session: refactor and optimize how notification are raised and when

session: changing online property to a state property

Instead of havine online as a boolean, session has now a state which can be
either disconnected, connected or online.

AUTHORS: Update email addresses

manager: Fix signature of ServicesRemoved signal

technology: Fix technology refcounting

__sync_fetch_and_sub() gives the value that had previously been in memory
which gives the older refount.

technology_find() does not refcount the technology.
__connman_technology_update_rfkill() was using technology_get() which
was refcounting the technology. Every technology_get() must be matched
with a corresponding technology_put() for accurate refcounting.

build: Remove find-service and connect-service from test_scripts

test: Remove find-service as manager LookupService does not exist

Fixes BMC#24901

test: Remove connect-service as manager ConnectService does not exist

Fixes BMC#24900

service: Remove 'Services' property from Manager D-Bus API

Remove support functions and 'Services' property from Manager
D-Bus API. Update gateway in all places service was changed.

service: Implement ServicesAdded and ServicesRemoved signals

When services are added or removed, collect them into separate
lists in order to send as many as possible in one go and with
then removed ones sent first. Add a wait of 100ms to collect
more services in one batch. When removing a service check
whether it was also on the added list; if so, remove it from
that list.

manager: Use helper function for adding arrays of objpath, dict

Use helper function for adding arrays of object path and
dictionary to D-Bus messages.

dbus: Function for adding array of object paths and dictionaries

Function for adding an array of object paths and dictionaries to
D-Bus messages.

tethering: Fix pool size request

Math is hard, I know.

Fixes BMC#24898

ippool: Add error and warning messages

wifi: Add support for default background scanning parameters

gsupplicant: Add support for background scanning parameters

manager: Fix TechnologyAdded signal signature

test: Fix scan method to work against technology api in test-connman

test: Update simple-agent to handle RequestBrowser method