ofono: Improve coding style

Makefile: Remove test-profile from test_scripts

service: Fix calling Agent API request input function

Requesting passphrases from the user has been done only when the
ConnectService D-Bus method has been called. As user input may be
needed every time a service is connected, input requesting moved to
__connman_service_connect() instead. Also to note is that the same
service can be retried after a failed connection attempt, for those
cases an already associated service->network needs to be disconnected
before an new connection is attempted.

service: When indicating error, also indicate IPv6 state as failed

Set IPv6 ipconfig state to failure in __connman_service_indicate_error().
The return value of the function isn't used for any practical purposes,
therefore always return 0.

service: Correctly set 8021X, WEP and PSK passphrases

When returning a passphrase from the Agent API RequestInput method, check
the service security type and set the passphrase accordingly. For WEP and
PSK use __connman_service_set_passphrase() and for 8021X use
__connman_service_set_agent_passphrase(). Print a debug message if the
service security type is none of the three above.

TODO: EAP-GTC support done, EAP-FAST not making progress

test: Remove test-profile

The profile API being gone for a long time now, test-profile is
irrelevant.

wifi: network_remove: Clear network pointer in wifi

When a network is destroyed we also have to remove the pointer
from the wifi data structure.

connmand[25303]: plugins/wifi.c:network_remove() network 0x69c350
connmand[25303]: src/network.c:connman_network_unref() network 0x69c350 name bmwnet02 refcount 1
connmand[25303]: src/network.c:connman_network_unref() network 0x69c350 name bmwnet02 refcount 0
connmand[25303]: src/network.c:network_destruct() network 0x69c350 name XXXXX

[...]

connmand[25303]: plugins/wifi.c:interface_state() wifi 0x699d60 interface state 1
connmand[25303]: plugins/wifi.c:is_idle() state 9
connmand[25303]: src/network.c:connman_network_set_associating() network 0x69c350 associating 0
connmand[25303]: src/network.c:connman_network_set_connected() network 0x69c350 connected 0

Fixes BMC#22307

service: Remove __connman_service_reset_from_networks()

This is not needed because there is only 1 network per
service.

Fixes BMC#22078

ofono: Destroy network_hash when disconnecting

ofono: Duplicate network path value

Path value will be freed after the function call, this may lead to
can't connect/reconnect to a 3G service

device: Free network after resetting service

After g_hash_table_remove(device->networks, identifier); the previously
cached service pointer can be invalid.

Bug reported by: Daniel Wagner <daniel.wagner@bmw-carit.de>

build: Remove g_dhcp_* from list of exported symbols

wifi: Use agent provided credentials as a fallback

service: Store agent provided credentials separately

Add new fields for agent provided identity/passphrase in service struct
(agent_identity/agent_passphrase) and network ("WiFi.AgentIdentity" and
"WiFi.AgentPassphrase").

Preparing these values in prepare_8021x() is left out because the same
thing is done in request_input_cb(), which is where the values always
come from.

network: Remove connman_network_set_pac()

This routine is not used.

service: Propagate changes of proxy auto-config url over DBus

service: Enable clearing of proxy auto-config url

This could introduce a need to null-check values where this function
is used (src/dhcp.c, src/network.c, src/provider.c). However, src/dhcp.c
requires that URL can be cleared.

AUTHORS: Mention Bertrand's contributions

ofono: Network name is empty when operator is not available

When the operator is not available and the network name is left to NULL,
the service is built with the hidden flag set.
The operator name will be updated whenever oFono passes it to ConnMan.

network: Set driver pointer to NULL back if not ready

The network_probe() function will be called several times until it
creates successfuly the service object.

In the case of the oFono plugin, the connman_device_add_network() is
called after the driver has been registered to the network. Therefore
the network->driver has been set but no device yet.
__connman_service_create_from_network() returns -EINVAL because
the device is not yet set.

The next call on network_probe() will bail out on 'if (network->driver
!= NULL) return -EALREADY;' so we have to reset it in the first round.

Fixes BMC #21705

service : Fix put data on null pointer

when using ConnectService of Manager interface a segfault arises due
to a null pointer.

in __connman_service_create_and_connect:

[...]
service = lookup_by_identifier(name);

if (service != NULL)
goto done;

network = create_hidden_wifi(device, ssid, mode, security);
if (network != NULL) {
connman_network_set_group(network, group);
service->network_created = TRUE;
}
[...]
in this part "service" is null

network: Fix double free in network_destruct()

ofono: Fix network owner ship

network: Do not unref network

wifi: Fix network owner ship

bluetooth: Fix remove device/network

device: Reorder cleanup

If free_network is called before the driver->disable() callback
the device has been removed from the networking structure.

bluetooth: Unref network objects

The bluetooth plugin creates the network object therefore
it should also unref (destroy) it.

device: Change signature of connman_device_remove_network()

ethernet: Take ownership of network objects

Therefore destroy network objects when the carrier goes offline.

device: Remove needless call to __connman_network_set_device()

network: Remove connman_network_register/unregister()

Since the service object doesn't take a reference the
intermediate step of having a connman_network_register/unregister
is not needed. This function are only there to have an additional
step where probe()/remove() will be called. This is the job of
connman_device_add_network()/connman_device_remove_network().

service: Don't ref/unref network objects

which were created by the plugins. The hidden wifi
networks will be created by service and therefore
service is the owner of the network objects and
does the unref when the service is destroyed.

service: Update network pointer in service

device: Fix a inbalance of ref/unref of network objects

The destroy callback of the hash table (device->networks)
will unref and unregister the network object.

service: Fix memory leak

valgrind reports that __connman_service_nameservers_remove leaks
memory.

g_strv_length already returns the right size of the new array.

dhcp: Fix memory leak

Reported by valgrind.

dhcp: Fix cleanup

__connman_dhcp_cleanup() is called early in the shutdown process
(before device or network).

network: Stop DHCP

If DHCP has been started for the IPv4 IP configuration we should also
stop it later.

The only place where it has been done so far was in
__connman_network_clear_ipconfig() which is called from
service.c:set_ipconfig(). But this will only be called if the user
changes service settings.

gweb: Fix gnutls_priority_set_direct() for various GnuTLS versions

dnsproxy: Add error print when DNS response send failed.

gsupplicant: set the cipher even if security is WEP

This patch fix an issue with WEP40, i tested it with WEP40 WEP104 and open AP.

It set the group cipher even if security is WEP otherwise connman won't connect to WEP40 service.

iptables: Work around API breakage introduced in iptables 1.4.11

The xtables_merge_options() paramter list got extended without version bump.
These clumsy #ifdef's make it possible to work with newer and older iptables
libraries. It got inspired from an iproute2 patch:
https://bugs.gentoo.org/attachment.cgi?id=276401

Once all distributions start to carry 1.4.11 it can be considered to bump the
minimum version for connman and remove these ifdef's.

Release 0.76

build: Use color output for unit tests

build: Add forgotten binaries to .gitignore

build: Remove wifi-scan testing utility

service: wpad only run on ipv4

service: clear ipconfig data and operations on free

Ensure ipconfig data and operations are cleared out when a service is
freed.

This is at least a partial fix and maybe a full fix for issues 21000
<http://bugs.meego.com/show_bug.cgi?id=21000> and 21285
<http://bugs.meego.com/show_bug.cgi?id=21285>.

build: Remove no longer needed build magic

build: Use BUILT_SOURCES for generating header files

build: Check for signalfd support

loopback: Recover Loopback on Setup when Up but No Address

It is possible to end up in situations in which the loopback interface
is up but has no valid address. In that case, we expect SIOCGIFADDR
will return EADDRNOTAVAIL and valid_loopback should return FALSE such
that connman can add a loopback address back to the interface.

This is a partial fix for bug 21001
<https://bugs.meego.com/show_bug.cgi?id=21001>.

loopback: Improve Error Reporting when Recovering Loopback

When fixing up the loopback address or netmask, log what is being
fixed up.

loopback: Use Existing Loopback Globals Rather than Temporaries

Use preassigned globals for the loopback address and netmask rather
than creating and assigning additional temporaries.

loopback: Return a More Descriptive Error Code

Return a more descriptive -errno rather than -1 when we fail to create
a socket.

service: Do not iterate over NULL service list

network: Remove unused function

service: Fix whitespace damage

provider: Fix ident memory leak if provider cannot be created

service: Cleanup fix

When the system shuts downn, we destroy the service_list.  The destroy
callback on the service_list sequence is service_free() which will
call the provider code. Provider wants to iterater over the
service_list we are currently freeing. This results into a crash.

#0  0x0000003edd680991 in __strlen_sse2 () from /lib64/libc.so.6
#1  0x0000003ee0a280fb in ?? () from /lib64/libdbus-1.so.3
#2  0x0000003ee0a15f19 in ?? () from /lib64/libdbus-1.so.3
#3  0x0000003ee0a19c0a in dbus_message_iter_append_basic () from /lib64/libdbus-1.so.3
#4  0x000000000043985b in append_path (value=0x6a7800, user_data=0x7fffffffe350)
    at src/service.c:121
#5  0x0000003edf65af07 in g_sequence_foreach_range () from /lib64/libglib-2.0.so.0
#6  0x0000000000439888 in __connman_service_list (iter=0x7fffffffe350, user_data=0x0)
    at src/service.c:127
#7  0x000000000045971e in connman_dbus_property_append_array (iter=0x7fffffffe440, key=
    0x472cd8 "Services", type=111, function=0x439860 <__connman_service_list>, user_data=0x0)
    at src/dbus.c:204
#8  0x000000000045990a in connman_dbus_property_changed_array (path=0x472ca0 "/", interface=
    0x472c8c "net.connman.Manager", key=0x472cd8 "Services", type=111, function=
    0x439860 <__connman_service_list>, user_data=0x0) at src/dbus.c:271
#9  0x00000000004391cd in services_changed (user_data=0x0) at src/profile.c:120
#10 0x000000000043924c in __connman_profile_changed (delayed=0) at src/profile.c:143
#11 0x000000000043f467 in service_free (user_data=0x6aab40) at src/service.c:2951
#12 0x0000003edf65aac8 in ?? () from /lib64/libglib-2.0.so.0
#13 0x0000003edf65aaa2 in ?? () from /lib64/libglib-2.0.so.0
#14 0x0000003edf65ae39 in g_sequence_free () from /lib64/libglib-2.0.so.0
#15 0x000000000044459f in __connman_service_cleanup () at src/service.c:5432
#16 0x000000000042ef5f in main (argc=1, argv=0x7fffffffe6b8) at src/main.c:412

provider: Untangle create and connect code block

provider: Remove pointless jump to error label

This fixes a compiler warning for an uninitialized variable. And this
warning is a real error.

  CC     src/provider.o
src/provider.c: In function ‘__connman_provider_create_and_connect’:
src/provider.c:476: warning: ‘provider’ may be used uninitialized in this function

network: Set network driver before creating the service

The autoconnect machine can potentially start from
__connman_service_create_from_network() so it needs a network driver.

service: disconnect state has priority over online one.

service: preventing to call ready state more than once.

timezone: Remove /etc/localtime if it's a symbolic link

And copy the timezone file over.

Fixes BMC #18887

device: Ignore dummy networking devices

When a kernel comes with dummy network devices support enabled, the
dummy interface will take the default route whenever a real interface
disconnects.
This is not what we want as the default route won't be released then.

network: Fix temp pointer memory leak

network: Let the gateway code handle the READY transition

For IPv4 (DHCP, fixed or manual), and for manual IPv6, the gateway handling
code (connection.c) is the one responsible for moving to the READY state.

ipconfig: Don't call IP bound ops when type does not match

network: Autoconf IPv6 is ready when address is set

It's not ready when the technology is connected. The ASSOC->READY transition
will happen through the IP bound ops.

service: Separate ipconfig state and service state handling

service: Use __connman_service_ipconfig_indicate_state()

service: Add __connman_service_ipconfig_indicate_state()

monitor-connman: Parse Notification messages

device: Get service type from __connman_device_find_device

The device search is based on the service type, not on the device type.

Fixes BMC#18747

dnsproxy: Fix indentation.

ofono: Set IPv4 address if IPv6 method is not supported

Also, fall back to IPv6 when the IPv4 method is not supported.

gweb: Apply address family restiction to resolver

gresolv: Queries address family scope restriction

Adding the capability to restrict DNS queries to a specific address family
through gresolv_set_address_family(). By default, AF_UNSPEC is applied.

session: Service type to bearer string conversion support

session: Correct GSequence sorting to use service_entry structs

The items on session->service_list GSequence are of type
service_entry. Fix sorting function to cast the items correctly.

provider: Get 'Host' and 'VPN.Domain' from provider struct

provider: Properly set host and domain provider members

Detect 'Host' and 'VPN.Domain' strings being set and insert them
properly into the provider structure. Also ensure that the
strings corresponding to provider structure members are not
duplicated into the provider string settings hash.

provider: Fix memory leak; free provider->host in provider destruct

main: Move signalfd setup in its own function

main: Use signalfd instead of plain signals

It's unsafe to call syslog in the terminat signal handler
because syslog takes an lock. So when the signal handler
kicks in and we were already in syslog, we have a nice
deadlock.

gweb: Use GnuTLS priority string to provide wide compatibility

This priority string will only enable SSL 3.0 and TLS 1.0 as protocols
and will disable, via the %COMPAT keyword, several TLS protocol options
that are known to cause compatibility problems.

Reference: http://www.gnu.org/software/gnutls/manual/html_node/Interoperability.html#Interoperability

gweb: Remove call to deprecated GnuTLS function

If we use gnutls 2.12.0 or later it's necessary to call
gnutls_transport_set_lowat() to disable the lowat functionality. Since
this function is deprecated in new versions of gnutls, call it only if
using an older version.

tethering: Update APIs to be able to use multiple private networks

doc: Update private network method prototypes

TODO: IPv6 gateway handling done.

network: Clear IPv6 autoconfigured routes when disconnected.

ipconfig: Added function to enable IPv6.

network: Clear interface addresses after removing gateway address.

The reason for this patch is that service disconnect code path
is different from service change code path. In disconnect,
the __connman_ipconfig_address_remove() is called which clears the
addresses from interface. When changing from one service to another,
the function __connman_service_disconnect() is not called and thus
the interface address is not removed.
We cannot call __connman_ipconfig_address_remove() yet at this point
because it would clear addresses from internal data structures so we
use __connman_ipconfig_address_unset() instead.

ipconfig: Add function to remove address from interface.

The new function removes the address from interface but does not
touch internal data structures.

network: Set manual IPv6 state and gateway.

connection: Separate IPv4 and IPv6 gateway and routing handling.