wifi: Handle D-Bus method call failure

If -ECONNABORTED is received from the gsupplicant network disconnect
callback, it's an indication that the interface has already been
removed from wpa_supplicant side. This can happen due to a race condition
between rtnl and wpa_supplicant interface removal when a USB WiFi dongle
is pulled out.

The pointers given to the callback can no longer be trusted, don't touch
them in the callback. The have been removed by other code at this point.

supplicant: Handle race condition on WiFi dongle removal

When a WiFi USB dongle is removed, both rtnl and wpa_supplicant will
be telling that the device is gone. While disconnecting and removing
an interface via gsupplicant, check that D-Bus doesn't give us an
error indicating the interface is already unregistered from D-Bus.

The gdb output looks like:
 (gdb) where
 #0  0x0042247c in network_connect (network=0x2) at plugins/wifi.c:1197
 #1  0x004228f0 in disconnect_callback (result=<value optimized out>,
 interface=0x0, user_data=0x99) at plugins/wifi.c:1254
 #2  0x0042b910 in network_remove_result (error=0x4d5458
 "org.freedesktop.DBus.Error.UnknownMethod", iter=<value optimized
 out>,
     user_data=<value optimized out>) at gsupplicant/supplicant.c:3593
 #3  0x0042ede0 in method_call_reply (call=0x4c6fd0,
 user_data=0x4d3be8) at gsupplicant/dbus.c:386
 #4  0x2abfddb4 in complete_pending_call_and_unlock () from
 /usr/lib/libdbus-1.so.3

Also don't call the same callback twice.

bridge: Close socket on error

gdhcp: Fix resource leak

The socket was not closed when the sendto failed.

gweb: Check null before reference

inet: Fix resource leak

The socket was not closed when err < 0 is true.

wifi: Skip service if there is error while loading its settings

If service settings cannot be loaded, then skip that service.
Thanks for Felipe Tonello for reporting the issue.

README: Add information about network interface clearing at startup

session: Register session after policy plugin returned config

Move the configuration part of __connman_session_create() into
session_create_cb(). With this change the policy plugin is able
to do async work to retrieve a configuration.

session: Factor out user settings in __connman_session_create()

In order to be able to pass the user configuration provided through from
the D-Bus Manager.SessionCreate() call to the callback we need to store
the configuration into a local data data structure. This data structure
can then be passed into the callback introduced later on.

session: Remove struct connman_session_bearer

Instead start using enum connman_service_type directly.

session: Handle no match case in parse_bearers()

session: parse_bearers() should check for invalid types

session: Rename session_parse_allowed_bearers() to parse_bearers()

This function name was a source of confusion because in a later patch
we introduce connman_session_parse_allowed_bearers() which will
call parse_bearers(). With this change it should be more readable.

session: Remove match_all from connman_session_bearer

The match_all will be expressed through CONNMAN_SESSION_TYPE_UNKNOWN.
The 'no match' case happens when allowed_bearers is NULL.

session: Remove name variable from connman_session_bearer

The string is only used when appending the bearer to the D-Bus
message in append_allowed_bearers(). Let's use
__connman_session_type2string() in append_allowed_bearers(). This
saves a bit of memory.

session: Add callback to policy create()

Instead returning directly a config when create() is called
in policy plugin, use a callback function for handing over a valid
configuration from the plugin to the session core. This prepares
support for asynchronous create call.

session: Policy register should check callback existence

Let's ensure that the policy plugin has all necessary callbacks
installed when connman_session_policy_register() is called. The rest
of the code expects that the create() and destroy() callbacks exist
whenever a plugin is used.

session: Pick directly first policy in list

There is no point in iterating over the list when we always
pick the first element in the list.

session: Factor out memory release part of cleanup_session

We want to reuse this code snippet for the error case in
__connman_session_create() too.

manager: Allow async CreateSession method call

The CreateSession D-Bus call should be marked as async call in order
to allow the session core to defer the response.

storage: Add void if function does not have any arguments

dhcp: Clear ipaddresses from ipconfig before starting DHCP

This is needed if manually configured addresses were used and later
DHCP was taken into use. If the manually configured IP information
(address, netmask and gateway) and the information given by DHCP is the
same, DHCP will not set the IP address to the interface.

ipconfig: Add function to clear ipaddress information

openvpn: Support '--tls-auth <file>' with optional '<direction>' argument

Connman              OpenVPN

OpenVPN.TLSAuth      --tls-auth <file> [<direction>]
OpenVPN.TLSAuthDir   optional <direction> argument to '--tls-auth'

Since 'OpenVPN.TLSAuthDir' contains the optional <direction> argument for
'OpenVPN.TLSAuth', handle them separately. The special handling is
identified by setting openvpn command line argument to NULL.

openvpn: Add more options

Connman              OpenVPN

OpenVPN.NSCertType   --ns-cert-type <client|server>
OpenVPN.AskPass      --askpass <file>
OpenVPN.AuthNoCache  --auth-nocache

Release 1.9

technology: Do not proceed with tethering if technology is not enabled

Fixes BMC#25846

systemd: prevent duplicate logging messages in journal

By default, both stdout and syslog messages go to the systemd journal,
which results in duplicate messages being logged.

Thanks to Vinicius Costa Gomes for pointing out this problem.

technology: Do not presume a technology being hardblocked when created

This fixes an issue with the TechnologyAdded signal when the first
un-hardblock event occurs:
- when a technology was created, D-Bus registration was done and hardblock
  was set to TRUE even if there was no evidence that the technology was
  rfkill driven
- when the technology was updated to be rfkill driven, hardblock was already
  set to TRUE and thus the technology was not unregistered
- when an rfkill event un-hardblocks the technology, the TechnologyAdded
  signal was not sent since the technology was already registered to D-Bus

dnsproxy: Add reply host part length checking

Check that the first part of the name is not of zero length before
attempting to calculate the length of the domain part. Also ensure
the domain lenght checking does not run outside of the receive
buffer.

Also add debug messages for ids and lengths in order to pinpoint
any possible problems.

wispr: Free wispr portal context in the relevant place

This fixes 2 issues:
- calling __connman_service_ipconfig_indicate_state() might lead to
  restart wispr_portal check, so context should not be freed afterward
  but beforehand.
- freeing the context in wispr_manage_message() should not happen since
  wispr_manage_message() will return back to wispr_portal_web_result()
  where we can still use the context.

Thanks to Julien Massot <jmassot@aldebaran-robotics.com> who reported
the issue and provided logs.

device: Do not mark connecting networks unavailable

When autoscan fallback code is started, it sets scanning to true which
in turn marks all networks unavailable except for the ones that are
already connected. When connecting during an ongoing autoscan, the
connection attempt stops autoscan and all unavailable networks are
removed, also the one to be connected.

The fix is to ignore both connected and connecting networks when
marking networks unavailable.

network: Fix typo in debug print

web: Make debug func print more useful information

The file and function name are printed in debug prints.

gresolv: Remove query from queue before destroying the query

The order of actions is important here.

gresolv: Make sure we will not receive DNS data after closing

We must close the channel when freeing the resolver object,
otherwise we might still receive data when the resolver has been
freed already.

Fixes BMC#25757

gresolv: Add more debug prints

gresolv: Make debug func print more useful information

The file and function names are printed in debug prints.

service: Prefer user connected services with SingleConnectedTechnology

Don't override user connected services with the ones selected by the
preferred technology list when SingleConnectedTechnology is enabled.
Do this by checking each connected service sorted in the beginning of
the service list for the userconnect flag.

service: Clear user connected flag on disconnect

Remember whether the service was connected by the user via D-Bus
until the service gets disconnected.

service: Keep only a single connected technology if configured

If SingleConnectedTechnology is enabled in main.conf, disconnect any
previously connected services when the new service enters ready state.

main: Add SingleConnectedTechnology main.conf variable

main.conf: Add SingleConnectedTechnology description

service: Fix default service switching and setting gateway

Calling switch_default_service() didn't change the service order since
the services were already sorted that way. Also update the gateway
immediately.

main.conf: Update preferred technology description

service: A preferred service in state ready is good enough

Simplify the preferred service selection such that a connected
service is good enough, especially since a connecting service
will also terminate the search for the current preferred one.

service: Don't trigger autoconnect when neither service is preferred

Don't trigger a new autoconnect when neither the default nor the new
service is preferred. Rely on the fact that normal autoconnect selection
mechanism has done the work for us already.

client: Add Remove() method support for service

client: Fix various memory leaks in monitor and services parts

client: Fix some memory leaks in data_manager part

client: Fix some memory leaks in commands part

gresolv: Remove the lookup for real

The call to g_resolv_cancel_lookup() will do nothing
because we just removed the lookup from the queue.
The fix is to remove the lookup directly and not call
the cancel function.

technology: Fix hardblocked state on non hw rfkillable switch added

If hardblock is on and a new device is inserted and detected as not
hardblocked, then it will be possible to enable/disable it (soft rfkill)
independently to the main hw rfkill switch.

technology: Recompute hardblocked state on rfkill remove event

This fixes the case of cascading rfkill switches: if enabled, hard rfkilling
such technology might generate contradictory events.

1 - first all switches are hardblocked
2 - then one of these switch (usually: device's switch) gets fully unblocked
3 - then this same switch gets removed

Step 2 is in contradiction with step 1, so we need to care about such switch
getting removed by recomputing the hardblocked state.

technology: Properly handle rfkill driven state

device: Add a getter for powered property

technology: Use the right method to enable or disable

If rfkill driven, use rfkill soft block/unblock. If not, request the
device to be enabled or disabled.

technology: Change enabled to a boolean and refactor accordingly

technology: Refactor and split functions setting enabled state

technology: Refactor rfkill event handling according to soft/hard block

No need to proceed with softblocked if technology is already hardblocked.
Apply offlinemode and persistant state according to softblocked state.
(saner logic and helped to cleanup code from style point of view too)

technology: Add and handle a marker for softblock status

technology: Add a marker to know if a technology is rfkill driven

Useful for coming patches: enabling/disabling technologies will be done
differently whether technology is rfkill driven or not:
- if rfkill driven -> enabled will rely on rfkill states
- if not -> enabled will rely on driver/devices states

technology: Refactor how a technology is enabled or disabled

Refactor how a device list is enabled/disabled: this will be useful
for coming patches. Simplify also the code, and remove useless gotos.

wispr: Keep track of proxy callback timeout

Keep track of the zero-second no proxy callback timeout and remove
it when freeing up the WISPr context.

wifi: Do not start autoscan if interface was not ready

gsupplicant: Send -ENOLINK error on scan reply if interface was not ready

gsupplicant: Add a new state according to 'interface_disabled'

When soft rfkill is on, wpa_supplicant sets the interface disabled and
sends a state named 'interface_disabled'. Taking this information into
account fixes the following issue:
- disable wifi (user setting) and hard rfkill it
- then un-hard rfkill it, whereafter rfkill states (soft/hard) will go
  like this:
    * from 1/1 to 0/0
    * from 0/0 to 1/0
    when 0/0 occurs, connman will request to enable wifi

The problem with this is that enabling wifi takes quite some time and
in between ConnMan will soft block wifi to disable it (according to
previous user setting). Thus it will request to disable wifi but since
enabling is still going on, this request won't do anything. Meanwhile
wpa_supplicant will also catch the soft rfkill event and wpa_supplicant
will set the state to 'interface_disabled', but since it's not handled
properly by ConnMan, the wifi_enable() callback will be called and the
function will assume wifi got enabled.

rfkill: Return the error when write() fails

gresolv: Remove all pending lookups when resolver is removed

Remove all lookups found in queue when GResolv object is removed.

gresolv: Avoid accessing already freed memory

We must remove the lookup from lookup queue and query from query queue
before calling user callback. The callback might unref the GResolv which in
turn would remove the lookup/query what we are trying to access after
the callback is returned.

So it is enough to remove the lookup or query entry from queue before
cb is called and then manually remove it after the callback has returned.

iptables: Ignore module loading error

In iptables 1.4.9 module loading gives an error even if the module
is built in. Ignore the loading errors because the missing iptables
support is noticed when trying to get the iptables socket options.

gdhcp: Returned IP address is already in host byte order

gresolv: Remove lookup from correct queue

The lookup must be removed from lookup queue and not from query
queue when cancelling the lookup. Otherwise it is possible that
we might access an already removed lookup that is still found in
lookup queue.

Fixes BMC#25728

nat: Memory leak when cleaning up

The nat struct was not freed when it is was removed from the hash.

gdhcp: Use data size specific option setting function

Instead of using dhcp_add_simple_option() the gdhcp now uses
three functions for setting uint8, uint16 and uint32 values.

gdhcp: Use host byte order internally

Convert to network byte order just before sending the packet.

gdhcp: Add unalignment macros

The alignment macros are copied from Bluez.

gdhcp: Remove obsolete load lease function

Release 1.8

doc: State is a string type

unit: Wrong enum type used in session code

gdbus: Fix not freeing list node by using g_slist_delete_link

g_slist_remove_link does not free the node which can cause leaks so
replace that with g_slist_delete_link which does free memory properly.

dnsproxy: Do not overwrite protocol and channel in server struct

We copied too much data into addrinfo struct which corrupted
the protocol and channel fields.

Fixes BMC#25726

technology: Do not expose a technology which is hard rfkilled

If hard rfkilled, a technology will not be exposed through DBus via
GetTechnologies. If hard rfkill status changes, TechnologyAdded and
TechnologyRemoved signals will be sent accordingly.

technology: Add helpers for (un)registering a technology in D-Bus

technology: Do not apply Powered setting change if hardblock is on

technology: Simplify set_powered function

technology: Link hard rfkill with actual device's state

When hardblocking a technology, it should disable the devices which belongs
to that technology. When un-hardblocking it should do the same but taking
care about user setting (it will enable the devices if only enable_persistent
is on).

technology: Handle harblock if all of them have identical type

On some hardware, there exist two rfkill entities for the same type
with a cascading issue: if one is soft blocked, the other one is
hardblocked. But if the hardblock switch is set, all are hardblocked.
This patch figures out that a technology is hardblocked only if all
related rkill events get the same hardblock value.

technology: Handle rfkill hardblock

technology: Handle rfkill hash table in a saner way

conf: Allow Counter and Notification interfaces in connman-polkit.conf

build: Don't use deprecated INCLUDES variable

Makefile.am:127: warning: 'INCLUDES' is the old name for 'AM_CPPFLAGS' (or '*_CPPFLAGS')

build: Don't use deprecated AM_PROG_MKDIR_P

AM_PROG_MKDIR_P is deprecated since:

configure.ac:26: warning: The 'AM_PROG_MKDIR_P' macro is deprecated, and will soon be removed.
configure.ac:26: You should use the Autoconf-provided 'AC_PROG_MKDIR_P' macro instead,
configure.ac:26: and use '$(MKDIR_P)' instead of '$(mkdir_p)'in your Makefile.am files.

We are already using $(MKDIR_P) so we just need to call the right macro.

build: Fix missing -lrt for clock_gettime()

librt is needed for using clock_gettime():

/usr/bin/ld: src/ntp.o: undefined reference to symbol 'clock_gettime@@GLIBC_2.2.5'
/usr/bin/ld: note: 'clock_gettime@@GLIBC_2.2.5' is defined in DSO /usr/lib/librt.so.1 so try adding it to the linker command line
/usr/lib/librt.so.1: could not read symbols: Invalid operation
collect2: error: ld returned 1 exit status

session: If no policy plugin is selected use default config

session: Move the default config create part back to core

error: EINPROGRESS error was not handled

inet: Cleanup rtnl callback data properly

The inet_rtnl_cb_data struct allocated in __connman_inet_rtnl_talk()
was not deallocated if we expected data from kernel.