Imported Upstream version 1.24

client: Fix passphrase handling

An empty passphrase means that WPS is to be tried next. If WPS is not
supported by the service, passphrase can not be left empty.

client: Implement VPN Agent API RequestInput method call

Instead of having a global agent_input struct, make the structure a
member of the agent_data structure that gets passed around when
handling the RequestInput method call.

With this modification agent_data is self-contained with all needed
information and the RequestInput handling code can be shared between
ConnMan and VPN agent implementations.

client: Add vpnagent command to register and unregister VPN Agent

client: Implement support for queueing Agent messages

ConnMan and ConnMan VPN daemon both send only one Agent message at
a time. Since they are two different daemons, they can be sending
messages at the same time independent of each other.

Implement a check for an agent request from the other daemon being
already processed and if so store the message for later. When the
current agent request has been processed, check if there are
pending requests from the other daemon and re-run the pending
function.

In order to support arbitrary queueing, make all method call
implementations asynchronous.

client: Add VPN Agent support for Release, Cancel and ReportError

Create a new agent data structure for the VPN Agent and add the needed
minor modifications to Release, Cancel and ReportError method calls in
order to get them working both for Agent and VPN Agent.

Create new register and unregister functions for the VPN Agent code.

client: Create agent data structure and pass it in function callbacks

Collect the agent variables into a data structure and update the agent
functionality to pass this data structure around as function callback
user data.

Update the agent mode input functionality to store both the callback
function and user data pointers. Notice that only only one input
callback can be handled at any one time due to the input handling itself
and the simple storing of callback and user data pointers.

client: Don't handle Agent messages in monitor code

Agent messages need to be handled in the agent code, ignore them when
they get captured in the monitor code.

client: Fix DBUS_TYPE_UINT16 pretty-printing

The previous pretty-printing fix causes another change to be
made for DBUS_TYPE_UINT16.

dhcpv6: Send domain list option to server

OPTION_DOMAIN_LIST(24) was not sent to server.

unit: Multiple collisions with different pools

Existing tests did not test this scenario:
* new address is registered to 192.168.0.2
* new address is registered to 192.168.1.2
* ippool is created to 192.168.2.1 (no collision)
* ippool is created to 192.168.3.1 (no collision)
* new address is registered to 192.168.3.1 (collision detected)

unit: Test ippool collision differently

Existing tests did not test this scenario:
* new address is registered to 192.168.1.2
* ippool is created to 192.168.0.1 (no collision)
* ippool is created to 192.168.2.1 (there is a collision and the
  code should skip the 192.168.1.1 range)

ippool: Clear allocated block pointer at cleanup

Without this the unit tests might core dump as the allocated_block
pointer would point to already freed memory.

ippool: Private address check was incorrect

Fixes BMC#25993

service: Send proxy properties when connected or disconnected

Fixes BMC#26019

technology: Offlinemode could not be disabled if no technology was powered

If all the technologies were powered off, then offline mode could
not be disabled.

Fixes BMC#26018

TODO: Mark GSequence removal done

service: Use GList instead of GSequence for service list

The hash table is now the owner of the service entry instead of the
service list.

session: Use GList instead of GSequence for service list

The hash table is now the owner of the service entry instead of
the service list.

session: Use service iterator instead of match/creation callbacks

When a new session is created we need to get the initial list
of services which match the session AllowedBearers criteria. As soon
the session is running add or removed services are tracked via the
notifier interface.

We concluded that exposing all the services is not really necessary
and therefore we are using currently two callbacks to get the initial
list. That exposes implementation details from session to service, that
is GSequence. We also need to expose the type 'struct service_entry'.

Instead doing so we are using an iterator and keep all the
implementation details in session.c.

service: Initialize tech_data

tech_data is allocated on the stack and is therefore not initilized
with NULL.

client: Avoid printing extra prompt on RequestInput

client: Read D-Bus string type only if argument is 'Type'

A basic type was always read independent of the argument type which
on some systems lead to a crash and on others only to a warning
printout.

client: Pretty-print D-Bus signed integers

To keep it more logical, rename the unsigned variable in the code.

client: Add VPN monitoring support

When adding VPN monitoring capabilities, also clean up the monitor code.

client: Add support for displaying VPN connections

Show a list of VPN connections known to the VPN daemon.

Also allow D-Bus path names to start with numbers, as it is fine with
the D-Bus library to define such path names. Fix up help command
pretty-printing as the VPN command is slightly longer.

client: Add VPN connection helper function

Add helper function for printing VPN connections. Show also configuration
and failure states using 'C' and 'F' as configuration might take some
time and failure conveys practical information to the user.

client: Add parameter to specify D-Bus service for method call

client: Add missing header file boilerplate

dhcpv6: Make sure release message is sent

Register an empty Release callback so that the message will be sent.

build: Add session policy format document

doc: Document session policy file format

session-test: Add tests for session_policy_local

Use the UID as identification.

session_policy_local: Set session identification method

For SELinux we need to store the complete context for iptables
to work.

session: Add session identification method to config

The session core needs to the know which kind of session
identification method has been selected in the plugin in order
to use the right iptables rules etc.

session_policy_local: Retrieve UID/GID from session user

When the session core ask to create a configuration, then we
first ask the D-Bus server which UID/GID the session belongs to.
If possible we also ask for the SELinux context. Then we try
to figure out which file containts the configuration for
SElinux, UID or GID identification.

The order of matching preference is SELinux, UID and finally GID.

session_policy_local: Remove struct create_data

Use struct policy_data instead of struct create_data.

session_policy_local: Rework policy file handling

The old assumption was that a config file is associtated
with one session only. With introducing UID/GID support a policy
might be used for several sessions. Furthermore, it was assumed
that the file name is the key/ident to identify a session and
a file containts exactly one policy.

Here are the new rules for writing a policy file.

- A valid file name contains letters or numbers and must have a '.policy'
  suffix.
- The file name has not semantical meaning
- A policy file may contain contain more than 1 policy
- Each policy entry starts with 'policy_'
- Each policy entry shall have one and exactly one valid key (e.g.
  selinux)

The implementation has two main hash tables, file_hash and session_hash
which owns 'the file' respectively the session config. Additionally there
is a hash table which connects a policy with a session (selinux_hash).

session_policy_local: Add some more debug information

session_policy_local: Update session before unref policy

The update flag can be removed as it is no longer needed.

session_policy_local: Refactor SELinux context parser

g_strplit() will eventually strdup the tokens so no need to
strdup() 'context'. But we an ugly cast (from 'const unsigned char *'
to 'const char *') is needed for g_strsplit() to make the compiler happy.

session_policy_local: Do not handle small allocation

Let's remove the small allocations error path because glib will
abort on memory exhausting anyway. Basically we remove dead code.

session: Do not access stale entry pointers

When removing the entry clear all references towards it.

session: Add session argument to create_service_entry_cb

Remember which session this struct service_entry belongs to when
removing the entries again.

TODO: Support other time sources

timeserver: Periodically wake up to recheck timeservers

The current situation is that a failed timeserver name resolution
removes the timeserver immediately from the list of used timeservers
and failing to connect to the nameserver IP address removes the
timeserver after a 2 second delay. Currently no mechanism exists to
re-create the list of nameservers once the list has been exhausted or
the more preferred timeservers have been removed from it.

This implementation combats both problems where less used timeservers
further away end up being used despite a more optimal closer one
becoming available as well as the problem of exhausting the
nameserver list. This is done by periodially waking up and checking
the timeserver in use. If the timeserver in use is not the most
preferred one or the list of timeservers is empty, the timeserver
list is recreated and the timeserver polling procedure is restarted.

iptables: Fix memory leak when invoking xtables_find_match

xtables_find_match() returns two different kind of pointers.
The first type is pointing to the function pointer table loaded
via dlopen(). The second type is a copy (via plain malloc) of the
first type. xtables_find_match() marks the copies/clones with
m == m->next. So we need to free the struct xtables_match which
point back to themself.

Also fix the issue that we didn't handle multple match instances
at the same time.

The memory leak is only visible when having more than one match
of a kind.

Release 1.15

config: Remove code implementing Protected config file setting

The code was relevant only for Manager.ProvisionService method call
which was removed a long time ago.

doc: Remove Protected description

The Protected setting was relevant only for Manager.ProvisionService
method call which was removed a long time ago.

vpn-config: Log a warning if the provider ident already exists

Also set the returned value appropriately when removing or adding
configuration files.

vpn-config: Remove obsolete protected code

For VPN connections created via configuration files the immutable flag
is set and it ensures no modifications are allowed. The protected
feature implementation was a left-over from service configuration code.

Removing the code fixes a bug where only VPN entries of different types
can be provisioned in the same provisioning file.

test-ippool: Shortcut exhausting allocation test

Instead really trying to allocate all possible 69888 blocks, let's
reserve the first two IP pools and then we only need to allocate
255 blocks. That should bring down the runtime considerable.

service: Update Domains property when Domains.Configuration changes

Domains property was not updated when Domains.Configuration was changed.
If Domains.Configuration is for example set to empty when using DHCP,
the Domains property is to be updated with the value configured via DHCP.

Fixes BMC#26010

service: Check service type before fetching interface index

For VPNs, the index is available from provider, for other services from
network.

Fixes BMC#26009

ipconfig: Use system set addresses for IPv4 property in 'manual' mode

Addresses and gateway for the service IPv4 property being used in the
system are available from ipconfig->system when the property is set to
'manual'.

client: Accept correctly spelled option name

Accept also correctly spelled version of 'preferred' option.

client: Fix minor issues in D-Bus path name checking

client: Fix index for IPv6 configuration option

client: Reserve space for trailing NULL in the array

session: Fix service2bearer to return a correct match all rule

doc: Remove documentation for Technology IdleTimeout property

IdleTimeout hasn't so far been implemented in any release and will be
handled by other means if needed.

service: Restart timeserver polling only for the default service

When timeservers are set, recreate timeserver polling list only if the
service is the the default one.

doc: Document the deprecated RemoveProvider method call

Fixes BMC#25958

test: Add test script for Clock API

clock: Implement load and save for Clock API properties

Load and save TimeUpdates and TimezoneUpdates Clock API properties.
Clock API Timeservers property is already handled by the timeserver
code.

Fixes BMC#25988

timeserver: Check the time update method

If the time update method is manual, no NTP time servers are returned.

clock: Export information about time update method

TODO: Add Clock API support for connmanctl

wifi: Scan all stored hidden networks

Previously the number of scanned hidden WiFi networks was
restricted by the fast scan limit of the WiFi driver. This
patch takes into account the driver fast scan limit and will
scan for all configured hidden WiFi networks.

Thanks to Jukka for testing this.

README: Update information about wpa_supplicant configuration

TODO: DHCPv6 DECLINE message support

gdhcp: Handle dhcp_client->type == G_DHCP_IPV4LL

Clean up several places where dhcp_client->type == G_DHCP_IPV4LL is unhandled.

shared: Add helper for looking up debugfs mount point

gdbus: Fix using NULL DBusPendingCall when disconnected from D-Bus

From D-Bus documentation for dbus_connection_send_with_reply():

"Warning: if the connection is disconnected or you try to send Unix file
descriptors on a connection that does not support them, the
DBusPendingCall will be set to NULL, so be careful with this."

Fix these errors when killing D-Bus daemon with the client still
running:

process 5712: arguments to dbus_pending_call_set_notify() were
incorrect, assertion "pending != NULL" failed in file
../../dbus/dbus-pending-call.c line 596.
This is normally a bug in some application using the D-Bus library.
process 5712: arguments to dbus_pending_call_unref() were incorrect,
assertion "pending != NULL" failed in file
../../dbus/dbus-pending-call.c line 572.
This is normally a bug in some application using the D-Bus library.

gdbus: Fix segfault when D-Bus daemon exits

Fix this crash if D-Bus exits while the client is still connected to it:

==5570== Invalid read of size 1
==5570==    at 0x402D28E: strcmp (in
/usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==5570==    by 0x4070E22: g_str_equal (ghash.c:1704)
==5570==    by 0x8055F61: message_filter (client.c:1123)
==5570==    by 0x4141500: dbus_connection_dispatch (in
/lib/i386-linux-gnu/libdbus-1.so.3.5.8)
==5570==    by 0x80506F7: message_dispatch (mainloop.c:76)
==5570==    by 0x4081A7E: g_timeout_dispatch (gmain.c:3882)
==5570==    by 0x4080D85: g_main_context_dispatch (gmain.c:2539)
==5570==    by 0x4081124: g_main_context_iterate.isra.21 (gmain.c:3146)
==5570==    by 0x408156A: g_main_loop_run (gmain.c:3340)
==5570==    by 0x41BF4D2: (below main) (libc-start.c:226)
==5570==  Address 0x0 is not stack'd, malloc'd or (recently) free'd
==5570==
==5570==

client: Do service name syntax check when a name is specified

Fix gcc compiler warning with -Wformat-security.

Cosmetic fix.

Release 1.14

TODO: Update connmanctl entry

Agent mode is mostly done, update the entry with Session mode and VPN
Agent support.

client: Check that strings passed to D-Bus to contain valid chars

D-Bus aborts if we feed invalid characters to it. So do some sanity
check before that.

client: Newline missing after printing scan error message

dhcpv6: Implement CONFIRM message support

See RFC 3315 Chapter 18.1.2. Creation and Transmission of Confirm
Messages for details

dhcpv6: Remember the last used IPv6 address

This is required so that we can decide whether to send CONFIRM
or SOLICITATION message to server.

ipconfig: Save last used DHCPv6 address

This is needed when implementing CONFIRM message decision logic.

dhcpv6: Added initial MRD support

MRD (Maximum Retransmission Duration) is the total duration of
of retransmission of the message. This will be used when sending
Confirm message.

gdhcp: Initialize message status code early

The message status code was initialized too late i.e., after
it was set so we lost the status code. This problem was seen
with wide-dhcpv6 server.

client: Fix for handling an unset callback

If there is no callback set, don't try to call readline function.

client: Fixes for handling Agent messages

Properly handle agent confirmation input, unset the prompt when RequestInput
processing ends and set the passphrase only if it contains some text.

man: Update connman.conf.5 manual page to use TetheringTechnologies

main: Rename AllowedTetheringTechnologies to TetheringTechnologies

gdbus: Fix calling GetManagedObjects twice in a row

Calling g_dbus_client_new followed by g_dbus_client_set_proxy_handlers
cause two calls to GetManagedObjects in a row as GetNameOwner reply is
asyncronously it triggers the second call because the handlers have
been set by g_dbus_client_set_proxy_handlers.

AUTHORS: Mention Yann's contributions

client: Initial RequestInput handling support

When receiving a RequestInput Agent API D-Bus call, ask the input in a somewhat
logical order from the user with user names first and passwords next. If a
passphrase is entered, ignore any WPS pin code requests. Prepare for passphrase
length checking by going through the RequestInput message to get the 'Type'
argument.

client: Add ReportError handling

Report error and wait for the user to reply with yes/no.

client: Add RequestBrowser handling

Print out the RequestBrowser URL and wait for user input to proced. Add a
helper function for getting a yes/no confirmation from the user and another
one for getting the last part of the service object path.

client: Implement Agent Release and Cancel method calls

Unref the saved Agent D-Bus message on Release and Cancel. In response to the
Release method call unnregister Agent interface and quit if in non-interactive
mode.

client: Add command mode and agent mode helper functions

In command mode remember history and do command completion, in agent mode
do neither. Enable saving and restoring of the readline prompt and add
the GIOChannel watch also for non-interactive mode.

client: Add agent command

client: Add support for registering and unregistering an agent

Add agent .h and .c files containing registering and unregistering functionality.