gdbus: Use destroy callback for service watch

Even though service watches accepted a "destroy" callback, they were
being ignored. This fix properly pass them along so they are called when
the watch is removed.

gdbus: Fix white space coding style issue

- corrected the space before '{'

connection: Add host route for VPN when setting default gateway

This is needed so that VPN gets default route when moving
services. That can happen if VPN did not had default route
before.

connection: Remove wrong VPN route

Before this patch the system created following routes for VPN

Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
0.0.0.0         192.168.10.9    0.0.0.0         UG    0      0        0 vpn0
192.168.2.0     0.0.0.0         255.255.255.0   U     0      0        0 wlan0
192.168.2.1     0.0.0.0         255.255.255.255 UH    0      0        0 vpn0
192.168.2.1     0.0.0.0         255.255.255.255 UH    0      0        0 wlan0
192.168.10.1    192.168.10.9    255.255.255.255 UGH   0      0        0 vpn0
192.168.10.9    0.0.0.0         255.255.255.255 UH    0      0        0 vpn0

Here the route to gateway in wlan0 192.168.2.1 via vpn0 is not
correct and it will prevent connections to 192.168.2.1

The correct routes should be:

Destination     Gateway         Genmask         Flags Metric Ref    Use Iface
0.0.0.0         192.168.10.9    0.0.0.0         UG    0      0        0 vpn0
192.168.2.0     0.0.0.0         255.255.255.0   U     0      0        0 wlan0
192.168.2.1     0.0.0.0         255.255.255.255 UH    0      0        0 wlan0
192.168.10.1    192.168.10.9    255.255.255.255 UGH   0      0        0 vpn0
192.168.10.9    0.0.0.0         255.255.255.255 UH    0      0        0 vpn0

Fixes MBC#25035

inet: Build failure fix

"The format modifier for size_t is %zd." (Marcel, 20/12/2011)

6to4: Connectivity check was done too early

The 6to4 code checks that we can connect to ipv6.connman.net
host via tunnel. The check can be done only after we have the
tunnel up and running. The patch makes sure that we do the
checks only after we have got newlink message from rtnl.

gweb: Add more debug prints when error happens

These prints are useful when checking why network
connection failed.

6to4: Add web debug prints

In order to see what is the problem with web connection,
add debug print function.

inet: Refactor rtnl functions in 6to4.c

The rtnl support functions are now in inet.c which is a
more logical place for them and now other files can also
use them.

connection: Set additional point-to-point route

This is useful if P-t-P link does not have a default route.

Fixes BMC#25027

service: Nameservers behind P-t-P link need special handling

Setting routes to nameservers need special setup if the
nameservers are behind point-to-point link, the gateway
must not be set in this case.

net: Add functions to get dest address of P-t-P link

These functions are needed when we need to setup a route
to point-to-point link destination address. The route is
needed if P-t-P link does not have a default route.

todo: Mark split tunneling/routing done

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