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.

service: Initiate autoconnect after changing ipconfig method

service: Do not set the service state when changing IPv4 method

The service state will be set to proper state if we are
actually doing the connection so do not set the state
when we are just setting the IPv4 method.

service: Do not try to connect to service that cannot create connection

Check if the service IPv4/6 ipconfig is turned OFF, in this
case do not try to connect to that service because it will
not succeed.

Fixes BMC#25587

ipconfig: Add function to check if the ipconfig is OFF

main: Add support for specifying a non-default configuration file.

This patch adds support for specifying a main configuration file
different from the default value of /etc/connman/main.conf. This
is valuable during debug to specify alternate debug-only
configurations.

main: Remove unused compat argument

storage.c: Search for settings file even if d_type is DT_UNKNOWN.

This is useful for filesystems where d_type is always DT_UNKNOWN,
for example in reiserfs.

TODO: Add WiMAX removal task

device: Remove unused phyindex

Release 1.4

wifi: Return -EBUSY when tethering

Fixes BMC#25561

wispr: Add a reference counter mechanism on contexts

Basically, wispr provides a pointer (a wispr/portal context) to third parties
(agent, gweb...) which are answering asynchronously. However, if the technology
related to the service owning such context is disabled: this context is not
valid anymore, leading to a crash when third part answers after such action.

In order to avoid such issue, wispr implement an internal refcount system.

Fixes BMC#25479

AUTHORS: Mention Ross' contributions

doc: Add coding-style.txt

This is a copy of the oFono coding-style document, with oFono instances
replaced with ConnMan ones.
It's also in sync with the neard one.

storage: check that the string isn't empty before splitting

If the string was non-NULL but empty (str="\0"), the following \0 assignment
would write to str[-1] and thus cause memory corruption.

On PPC and MIPS, this was causing crashes in glibc.

device: Do not set regdom when device is not powered

Fixes crash bug:
connmand[6761]: plugins/ofono.c:get_properties_reply() /huawei_0 path /huawei_0 org.ofono.NetworkRegistration
connmand[6761]: plugins/ofono.c:netreg_properties_reply() /huawei_0
connmand[6761]: plugins/ofono.c:netreg_update_regdom() /huawei_0 MobileContryCode 460
connmand[6761]: src/device.c:connman_device_ref_debug() 0x8686ac0 ref 3 by plugins/wifi.c:991:wifi_set_regdom()
connmand[6761]: Aborting (signal 11) [src/connmand]
connmand[6761]: ++++++++ backtrace ++++++++
connmand[6761]: #0  0xb7749400 in
connmand[6761]: #1  0x806aa0b in g_supplicant_interface_set_country() at nat.c:0
connmand[6761]: #2  0x8062297 in wifi_set_regdom() at wifi.c:0
connmand[6761]: #3  0x80a607d in connman_technology_set_regdom() at ??:0
connmand[6761]: #4  0x806fefb in netreg_update_regdom() at ofono.c:0
connmand[6761]: #5  0x8071f3b in netreg_properties_reply() at ofono.c:0
connmand[6761]: #6  0x806f747 in get_properties_reply() at ofono.c:0
connmand[6761]: #7  0x4a374b62 in /usr/lib/libdbus-1.so.3
connmand[6761]: #8  0x4a3602b7 in /usr/lib/libdbus-1.so.3
connmand[6761]: #9  0x4a3639f5 in /usr/lib/libdbus-1.so.3
connmand[6761]: #10 0x8054250 in message_dispatch() at mainloop.c:0
connmand[6761]: #11 0x49f3e9d7 in /lib/libglib-2.0.so.0
connmand[6761]: #12 0x49f3de38 in /lib/libglib-2.0.so.0
connmand[6761]: #13 0x49f3e13e in /lib/libglib-2.0.so.0
connmand[6761]: #14 0x49f3e5a6 in /lib/libglib-2.0.so.0
connmand[6761]: #15 0x8053ab6 in main() at nat.c:0
connmand[6761]: #16 0x49cfc4c4 in /lib/libc.so.6
connmand[6761]: +++++++++++++++++++++++++++

gdhcp: Set client state to INIT_SELECTING on restart

With the previous behavior, we would end up sending a discover message
and then ignoring the reply due to the client state mismatch, so a
restart would never result in successful acquisition of a lease.

RFC 2131 specifies that the client should return to INIT state in
various restart scenarios.  However, also note that it would not make
sense in any case to send a discover message and ignore the reply.

rtnl: Support PPP connections

The ARP header type for PPP connections is ARPHRD_PPP (512).

gresolv: Use predefined mnemonics rather than magic numbers

Use predefined mnemonics from arpa/nameserv.h for the return value from
ns_msg_getflag rather than magic numbers.

gresolv: Do not update successful status with unsuccessful one

When performing a resolver lookup from timeserver or wpad, both perform
queries with an unspecified address family. This means that both A and
AAAA record queries are issued. In cases where a valid, successful A
response comes back but where the AAAA query results in a timeout, do
not smash the successful A status with the time out AAAA status;
otherwise, the timeserver or wpad will appear to fail to them when, in
fact, the A query was successful and more than satisfies its unspecified
address family requirement.

Partial fix for BMC#25486.

gresolve: Fix a typo in conditional check for returning results

In both parse_response and query_timeout there exists logic that checks
to ensure that both an A and AAAA lookups have either been responded to
or timed out before processing and returning results to the caller. In
query_timeout, there was a typo in the condition check such that it did
not match those conditions tested in parse_response.

ntp: Add debug message when starting NTP against resolved server

This patch adds an analagous debug message when starting NTP against a
resolved server as when starting against a numbered server.

ntp: Fixed a debug message typo

ntp: Retry a given server address multiple times before falling back

This patch causes the internal NTP client to retry a given server
address up to three times (four total attempts) at a two second interval
before falling back to another server name or server address.

ntp: Use mnemonic when specifying the request precision

This patch uses a mnemonic to specify the request precision and also
adds a comment how a future implementation might pass along the actual
system wall clock precision.

ntp: Add mnemonics for common clock precisions

ntp: Use mnemonics when checking reply flags

Use mnemonics when checking reply flags and print out information in
debug mode about why the reply flags were rejected.

ntp: Specify leap-not-in-sync in flags

This patch sets the non-in-sync flag of the leap subfield of the packet
flags field. This reflects behavior implemented by ntpdate or ntpd in
one-shot mode.

ntp: Send a transmit time value that reflects current system time

This sets the ntp request packet transmit time to the current system
time to reflect behavior implemented by ntpdate and ntpd in one-shot
mode.

ntp: Use mnemonics rather than magic numbers

Convert a magic number into equivalent mnemonics for the flags (leap,
version, mode) packet field.

ntp: Add mnemonics for the flags (leap, version, mode) packet field

timeserver: Try all resolved timeserver IP addresses

If we get more than one IP address, then add those
server addresses to the time server list so that they
can be tried if the first entry fails.

ntp: If the server cannot be contacted, try next one

This could happen if the network routes are not yet setup
so the sendto() could fail. In this case try next address and
hope for the best. This is a rare issue but I have seen it
couple of times.

Revert "build: Use -Wshadow compiler option"

build: Use -Wshadow compiler option

dnsproxy: Avoid multiple shadowing variables

tethering: Avoid shadowing connection variable

session: Avoid shadowing hash table iterator

session: Avoid shadowing session info variable

config: Avoid shadowing config structure variable

counter: Avoid shadowing connection variable

notifier: Avoid shadowing connected variable

agent: Avoid shadowing connection variable

service: Avoid shadowing string variable

task: Avoid shadowing connection variable

plugins: Avoid shadowing connection variable

gsupplicant: Avoid shadowing key variable

unit: Avoid shadowing session info variable

tools: Avoid shadowing buffer variable