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.

client: Enable method call return to exit or continue running

Enable a client method call return function to exit or continue running when
in non-interactive mode. When a connmanctl_dbus_method_return_func_t returns
-EINPROGRESS in non-interactive mode, the command line client is kept running
even after the method call has been processed.

The command line client needs to keep running after a successful Agent
registration in order to process Agent method calls. Also enabling tethering
in non-interactive mode needs the client executing until SSID, passphrase and
tethering mode have been set.

client: Make dbus helper function variable const char *

dhcpv6: Add elapsed time option to sent messages

A mandatory elapsed time option was missing from the sent
DHCPv6 messages. See RFC 3315 chapter 22.9 for details.

technology: Disable tethering when device disappears

If tethering is active when device is removed, then shutdown
tethering cleanly so that tether interface and bridge are
properly removed.

dnsproxy: Do not unref g_io_channel if we know it is null

Currently tethering does not support IPv6 so its listener is
not created and is null. This will cause following output

connmand[18363]: src/dnsproxy.c:destroy_tcp_listener() index 31
(connmand:18363): GLib-CRITICAL **: g_io_channel_unref: assertion `channel != NULL' failed
connmand[18363]: src/dnsproxy.c:destroy_udp_listener() index 31
(connmand:18363): GLib-CRITICAL **: g_io_channel_unref: assertion `channel != NULL' failed

client: Fix up wrongly included typedef

client: Remove unneeded members from structs

Remove now obsolete memebers from the option struct and instead add a description
field. Also remove the now unnecessary include.

client: Move commands around to more logical places in the table

core: Print timezone information if available from server

dnsproxy: Conditional jump depends on uninitialized value

This came from Daniel when running ConnMan using valgrind

==20635== Conditional jump or move depends on uninitialised value(s)
==20635==    at 0x49ECF4: get_name (dnsproxy.c:867)
==20635==    by 0x49F3F7: parse_response (dnsproxy.c:1096)
==20635==    by 0x49FD40: cache_update (dnsproxy.c:1410)
==20635==    by 0x4A0FF3: forward_dns_reply (dnsproxy.c:1803)
==20635==    by 0x4A15D9: udp_server_event (dnsproxy.c:1912)
==20635==    by 0x3331E47A54: g_main_context_dispatch (gmain.c:2715)
==20635==    by 0x3331E47D87: g_main_context_iterate.isra.24 (gmain.c:3290)
==20635==    by 0x3331E48181: g_main_loop_run (gmain.c:3484)
==20635==    by 0x44B600: main (main.c:670)

build: Install connmand and connman.conf manual pages

man: Update connman.conf.5 manual page

Updated PreferredTechnologies description.
Added SingleConnectedTechnology, AllowedTetheringTechnologies and
PersistentTetheringMode descriptions.

make: Add main.conf example file into dist tar ball

dhcpv6: Fix memory leak

The received IP address was not freed.

gdhcp: The last option in a message was not parsed correctly

If the desired option was last one in the message, then it was
lost because the remaining data calculation was wrong.

dhcpv6: Retransmit a message with same transaction id

Make sure we do not create a new transaction id for a message
when it needs to be retransmitted.

gdhcp: Do not change the transaction id if retransmitting

As requested in RFC 3315 chapter 15.1, a client MUST leave
the transaction id unchanged in retransmissions of a message.

gdhcp: Add helper functions for message retransmission

gdhcp: Add debug print for transaction id

Print transaction id of sent and received messages so that it is
easier to match the packets.

TODO: Update type definitions cleanup task

include: Remove definition for connman_uint16_t

network: Replace connman_uint16_t with uint16_t

include: Remove definition for connman_uint8_t

network: Replace connman_uint8_t with uint8_t

service: Replace connman_uint8_t with uint8_t

tethering: Use fallback nameservers if dnsproxy is not in use

If user has disabled dnsproxy with -r option, then fallback to
user specified nameservers instead of using the Google one.

dnsproxy: Return proper return code if listener_table is missing

The listener_table can be NULL if we have started connman with -r
option. Because we return < 0 value to caller in this case,
the caller, e.g. the tethering code, has a possibility to change
its functionality, e.g. the tethering code can fall back to some
other DNS servers.

technology: Auto start tethering if enabled and configured

If the user has enabled persistent tethering mode in main.conf, then
try to activate tethering when technology is re-enabled, or we are
returning from offline mode, or after the device has rebooted.

main: Add support for PersistentTetheringMode configuration variable

conf: Add description of PersistentTetheringMode

ethernet: Add tethering support

rtnl: Interface was added to technology too early

The __connman_technology_add_interface() needs to be called
after we have created the technology which is done by rtnl->newlink()
callback. If this is done the old way, the ethernet devices
will not get their interfaces set properly.

device: Add support functions needed by ethernet tethering

main: Add support for AllowedTetheringTechnologies configuration variable

conf: Add description of AllowedTetheringTechnologies

client: Trim off spaces from user input

After splitting the input string into components, create a new array holding
the non-zero substrings as input for the commands. Thus the input can start
and end as well as divide its separate parts by any number spaces.

client: Add command completion

Add completion support for connmanctl commands.

client: Follow coding style

Follow coding style and use __connmanctl prefix for the commands() function.

client: Add support for tethering

Add tethering support. For WiFi also set the SSID and passphrase if they
have been specified on the command line. Check that setting both SSID and
passphrase succeed before trying to enable tethering.

After successfully setting SSID and passphrase for WiFi, use the boolean
value parsing helper function return value to decide whether WiFi tethering
is to be enabled, disabled or not modified.

Fixes BMC#25886

dbus: Add connman_dbus_get_connnection_unix_user()

The UID is fetched via org.freedesktop.DBus.GetConnectionUnixUser D-Bus
method call.

dbus: Refactor callback helper struct

The helper struct will be used by a new dbus function which uses
a different callback signature. Therefore, we need to use a
void pointer for storing the callback.

session: Do not fail when creating default policy configuration

Do not handle the small alloc failures because glib will abort
when memory is tight anyway.

session: Change to "func_t" postfix to callback typedef

Streamline the callback typedefs with the rest of the code.

session_policy_local: Use callback helpers

Use the generic callback helpers to encapsulate struct create_data.
In a later patch we will add some more data elemets to struct
create_data. That is the reason why it is not remove here.

session_policy_local: Load policy when a new file is added

When a file is added (not modified) we need to parse it as well

session_policy_local: Do not free policy on load error

We should not unref the policy on parsing errors becase the policy
lifetime is attached to the lifetime of the file. When the file
is removed we remove it from the hash table.

session_policy_local: Empty policy list indicated no match all

Match all is expressed via CONNMAN_SERVICE_TYPE_UNKNOWN. An empty
list means no match.

session_policy_local: Set default policy using common code

Use connman_sessoin_set_default_config() instead of open coded
version.

This prepars the next fix.

session: Empty policy list indicated no match all

Match all is expressed via CONNMAN_SERVICE_TYPE_UNKNOWN. An empty
list means no match.

session: Do not try to access already removed hash

Make sure we never try to cleanup when the hash table has
been destroyed. In this case all resources have been
freed already.

session: Reorder shutdown sequence

By calling __connman_session_cleanup() before __connman_plugin_cleanup()
we make sure all resources allocated can released in the correct order.
The code assumes after an successful allocation, free will always work
and therefore we double free allocated memory.

wifi: improve fix to avoid device double release on disable

Per MEEGO-25999 <https://bugs.meego.com/show_bug.cgi?id=25999>, there
exists a race between scan_callback and wifi_disable such that a care-
fully-timed deassertion of the technology Powered property between
throw_wifi_scan and scan_callback will lead to a device reference
count underflow. A second Powered property false -> true -> false
transition will then deallocate the device and Wi-Fi will no longer
function until connman or the system is restarted.

This patch improves upon and revises the prior patch to address this
issue by recognizing that hidden networks involve a slightly different
code path and timing and that both this and the prior case can be
addressed by tracking the device scanning state and only releasing the
device when the state is asserted.

tools: Rename dnsproxy-unit to dnsproxy-test

Rename the dnsproxy file as it is more of a test tool than a unit test.

tools: Make message buffers static

service: Set state properly after disconnecting the associating service

This is related to fix d7d8f8bdb7c1ddae098893e10608fcb42034a9ff
We did not set the state to idle after disconnecting the
associating service.

Fixes BMC#25981

wifi: avoid device double release on disable

Per MEEGO-25999 <https://bugs.meego.com/show_bug.cgi?id=25999>, there
exists a race between scan_callback and wifi_disable such that a care-
fully-timed deassertion of the technology Powered property between
throw_wifi_scan and scan_callback will lead to a device reference
count underflow. A second Powered property false -> true -> false
transition will then deallocate the device and Wi-Fi will no longer
function until connman or the system is restarted.

This patch addresses that race by tracking and observing the disabling
state such that scan_callback ad wifi_disable can effectively negotiate
which has performed the device release, thereby avoiding the double
release and resulting underflow.

Release 1.13

build: Add dnsproxy-unit to .gitignore

gdbus: Fix not calling property_changed callback

In case a property did not appear by the time proxy_added was called
property_changed has to be called if it appear latter otherwise the
application will be unaware of it.

gdbus: Use gcc builtin instead of g_atomic

g_atomic_* end up using G_STATIC_ASSERT, causing gcc 4.8 to yell due to
-Wunused-local-typedefs.

gdbus/client.c: In function ‘g_dbus_client_ref’:
/usr/include/glib-2.0/glib/gmacros.h:162:53: error: typedef ‘_GStaticAssertCompileTimeAssertion_2’ locally defined but not used [-Werror=unused-local-typedefs]
 #define G_STATIC_ASSERT(expr) typedef char G_PASTE (_GStaticAssertCompileTimeAssertion_, __COUNTER__)[(expr) ? 1 : -1]

vpn: Host IP address was not freed

The host ip address field is only used if user has given vpn
server as a host name instead of host ip address.

vpn: Free ip address before allocating a new one

Possible memory leak here as we might have ip address allocated
already.

dnsproxy: Fix cached message length correctly

We must copy the response buffer before fixing the packet length.
Otherwise the packet length will go wrong if the request contained
EDNS0 data.

client: Add --help, -h options

tools: Add unit test for dnsproxy

dnsproxy: Handle partial TCP messages from client

We were not handling client sent partial TCP messages correctly.
This meant that ConnMan would block if the client using TCP would
not send full DNS request.

When a new TCP client connects in tcp_listener_event(), a new
TCP channel is created to handle a partial client message if
all of the message is not already available. The partial TCP
message is patched together in tcp_client_event(). When all of
the TCP message has been received, the message is processed in
read_tcp_data(). The client has to send the DNS request within two
seconds before the connection is closed by ConnMan.

service: Drop unused argument in preferred_tech_list_get()

client: Fix help text, IPv4 setting uses netmask

dnsproxy: Create cache if it is missing when doing lookup

dnsproxy: Fix memory leak when request timeouts