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.

ntp: Calculate transmit time from receive time and monotonic delta

Calculate transmit time used in NTP as the reception time minus
the delta of the monotonic receive and transmit times. When
calculated this way, it does not matter if the time happens to
be set to something else between the sending and receiving of the
NTP packet.

On sending the added monotonic time and the previous transmit time
are saved at the same point in time. On reception the monotonic time
is evaluated after the packet has been received by ConnMan. This is
in contrast to the actual reception wall clock time which is added
by the kernel. The difference between the reception times on a normal
system is about 100µs, which is neglible.

test: Check test-connman arguments

Check test-connman offlinemode arguments and the number of arguments
in general.

Fixes BMC#25723

technology: Update tethering properties when relevant

wispr: Properly handle Agent error

gdbus: Refactor filter_data_find()

Now this function is only used for searching the listeners of a
connection and the other parameters are not needed anymore.

gdbus: Fix wrong signal handler match

When we add a signal handler with g_dbus_add_signal_watch(), this
function tries to multiplex the matches added in libdbus by checking
if there's a previous filter_data with the same fields. However, if the
field is NULL it accepts as being the same. The result is that the
following watches will use the same filter data:

watch1 = g_dbus_add_signal_watch(conn, BUS_NAME, NULL, iface, member,
cb1, data1, NULL);
watch2 = g_dbus_add_signal_watch(conn, BUS_NAME, "/path2", iface, member,
cb2, data2, NULL);
watch3 = g_dbus_add_signal_watch(conn, BUS_NAME, "/path3", iface, member,
cb3, data3, NULL);

The result is that when a signal arrives with path == "/path2", all 3
callbacks above will be called, with the same signal delivered to all of
them.

Another problem is that, if we invert the calls like below, only signals
to cb1 will never be trigerred, nonetheless it used path == NULL.

watch2 = g_dbus_add_signal_watch(conn, BUS_NAME, "/path2", iface, member,
cb2, data2, NULL);
watch1 = g_dbus_add_signal_watch(conn, BUS_NAME, NULL, iface, member,
cb1, data1, NULL);
watch3 = g_dbus_add_signal_watch(conn, BUS_NAME, "/path3", iface, member,
cb3, data3, NULL);

This is fixed by not multiplexing the matchs with filter data if any of
the fields are different, including being NULL. When a signal arrives,
if a field is NULL we accept it as a match, but not when adding the
signal handler.

gdbus: Fix crash when getting disconnected from the bus

When getting disconnected from the bus sometimes (maybe always?)
dbus_watch_handle() can cause the "info" context to be free'd meaning
that we should not try to access it after the call. The only member we
need access to is the connection pointer and as the code already has a
ref() call for it it's only natural to solve the issue by adding a local
variable not dependent on "info".

The backtrace of the crash fixed looks as follows:

 Invalid read of size 8
   at 0x121085: watch_func (mainloop.c:105)
   by 0x4C72694: g_main_context_dispatch (gmain.c:2539)
   by 0x4C729C7: g_main_context_iterate.isra.23 (gmain.c:3146)
   by 0x4C72DC1: g_main_loop_run (gmain.c:3340)
   by 0x120541: main (main.c:551)
 Address 0x5bbcd90 is 16 bytes inside a block of size 24 free'd
   at 0x4A079AE: free (vg_replace_malloc.c:427)
   by 0x4C7837E: g_free (gmem.c:252)
   by 0x4F708BF: dbus_watch_set_data (dbus-watch.c:614)
   by 0x4F70938: _dbus_watch_unref (dbus-watch.c:132)
   by 0x4F6E9A7: _dbus_transport_handle_watch (dbus-transport.c:884)
   by 0x4F59AFB: _dbus_connection_handle_watch (dbus-connection.c:1497)
   by 0x4F70AF9: dbus_watch_handle (dbus-watch.c:683)
   by 0x121084: watch_func (mainloop.c:103)
   by 0x4C72694: g_main_context_dispatch (gmain.c:2539)
   by 0x4C729C7: g_main_context_iterate.isra.23 (gmain.c:3146)
   by 0x4C72DC1: g_main_loop_run (gmain.c:3340)
   by 0x120541: main (main.c:551)

wispr: Service does not need to be reference counted

wispr: Free wispr portal context struct instead of unreferencing it

The structure can now be directly freed, since the reference count
is always one. As a result, remove wispr_portal_context_unref()
function.

wispr: Remove service usage reset function

Remove reset_service_usage() as it is now obsolete with the
refcount always being equal to one.

wispr: Agent calls need not be referenced

There is no need to refcount while doing Agent API calls, since
outstanding Agent API calls are removed when the service is
disconnected.

Also remove wispr_portal_context_ref() as it is no longer used.

wispr: Clean up reference counting

connman_wispr_portal_context is initialized with refcount set to one.
Don't reference it an additional times while doing asynchronous calls
if the calls can be cancelled. Unreference the structure when done.

TODO: Add --nobacktrace removal task

log: Disable stack trace from command line

Add a '--nobacktrace' command line switch to let the system
handle stack traces. The default is to let ConnMan handle them
as before

test: Fix PrefixLength setting in set-ipv6-method

ipconfig: Fix IPv6.Configuration parsing

PrefixLength is exposed as a byte in the API, but code was
waiting for a string when setting a new value.

unit: Do not append empty AllowedBearers dictionary

An empty AllowedBearers means do not match anything.

session: Apply policy on AllowedBearers

session: Update AllowedBearers documentation

unit: Support ConnectionType unknown

session: Apply policy on ConnnectionType

session: Enforce correct ConnectionType settings

Up to now an empty or wrong ConnectionType was interpreted as 'any'.
If the ConnectionType is either wrong ignore it. If no ConnectionType
setting is passed in during creation of the session choose 'any'.

session: Update ConnnectionType documentation

session: Support no match in AllowedBearers

Up to now an empty AllowedBearers was interpreted as match any. Let's
make the match all explicit by expecting '*'. If an empty
AllowedBearers is provided, we interpret this as no match. In this
case the application will never get an online notifcation.

session_policy: Use bearer free function

session: Use g_slist_free_full to free bearer list

session: Add bearer list cleanup function

session: Remove unused functions

Basic types are not used for the configuration anymore.

session: Get config from policy plugin

Let's ask the plugin for the default configuration. AllowedBearers
and ConnectionType are not yet taken fromt the plugin. Instead
the user configuration will be taken.

session_policy: Implement create() and destroy()

session: Add callbacks for config creation and distruction

Whenever a new session is created, the core will ask the plugin
the create the config. This config can then be used by the core
to enforce the user settings.

The owner ship of this config stays in the plugin. Therefore, the plugin
will destroy is later.

session: Allow plugin access to connman_session_allowed_bearers_any

session: Move configuration bits to a public struct

Instead of passing basic datatypes between the session core and the
policy plugin, provide a complex data structure which holds
the session configuration.

session: Move bearer_info to session.h

Rename bearer_info to connman_session_bearer. Use also 'bearer' as local
variable instead of 'info' to avoid confusion with session_info.

session: Use session pointer instead of string id

Instead of passing in some string to identify we can use the
connman_session pointer. This allows us to keep the way sessions
are identfied away from the core, e.g. using the D-Bus owner id or
something else.

session: Move struct connman_session to session.h

session: Update session when policy plugin is added or removed

session: Assign config plugin to session

Let's pick the first plugin. If needed we can add some more smart code
later on.

session: Move get config calls after basic session initialization

The config call will need some context later on. Therefore, create
and initializate with the owner and add the watch on the ownership
before we ask the policy plugin for any configuration.

session: Maintain a sorted policy plugin list

session: Remove global configuration plugin

Implement support for per session configuration plugin. Obviously,
this patch itsel is not complete. The code for assigning a
configuration plugin to session->config follows in the next patches.

session: Add plugin priority

Support several session configuration plugins at runtime.
Set the default priority to low for the current policy plugin.

session: Move policy plugin code up

We don't want to have forwared decleration and the next patches
would make this necessary.

session: Rename session_config to session_policy

session: No need to free NULL list

session: Check D-Bus argumets for Changed method

unit: Update session unit test to API changes

session: Remove AvoidHandover implementation

session: Remove AvoidHandover documentation

The AvoidHandover boolean was there to fine tune the selection
algorithm in ConnMan. It was never implemented. Let's get rid
for the time beeing. If there is a real need for it we should
implemented this setting through the configuration plugin.

manager: Manage properly the pending message for session mode

README: Mention readline and command line client disable switch

build: Add manual pages for connman and config file

resolver: Make sure we do not use already removed timer

resolver: Do not touch IPv4 servers in redo case

The name server refresh should be only done for IPv6
nameservers.

dnsproxy: Remove AI_PASSIVE for connected socket

The AI_PASSIVE is useless here because the server address
is always known.

dnsproxy: Set recipient address when sending msg to DNS server

There has been cases where send() is returning EINVAL even though
there is a connected the socket. So use sendto() instead and supply
the destination address when sending.

ipconfig: Set default IPv6 method according to kernel IPv6 support

If kernel does not support IPv6, then turn default ipconfig
method OFF. If kernel supports IPv6, then the default is AUTO
meaning that we try to send router solicitation messages.

inet: Add function to check if IPv6 is supported by kernel

client: Show ready and online state correctly

AUTHORS: Mention Johannes' and Justin's contributions

client: Fix AutoConnect configuration

Changing the AutoConnect setting via connmanctl does not work, due to the
mistaken use of || instead of &&.

This patch fixes the issue, and tries to make things a little more readable.

rtnl: Ignore netlink messages generated by wext

ConnMan should just ignore netlink messages generated by wext
as described in commit 4997f6e38f34effc00179e849dee0dc2e2263d3a.

Handle wext message detection using the IFLA_WIRELESS type
because the ifinfomsg ifi_change field is reserved for future
use and code using it for wext detection has been removed.

wifi: Fix tethering with kernel 3.5

Wifi tethering works with kernel 3.4, but not with 3.5. Bisecting
showed that the following kernel patch causes the breakage:
    "3edaf3e mac80211: manage AP netdev carrier state".

Running connman with debugging enabled showed that in case of 3.4
we have the following sequence of RTM_NEWLINK events from the kernel:

1. IFF_UP not set, ifi_change=1
2. IFF_UP,IFF_LOWER_UP, ifi_change=1

which makes connman do the following:

connmand[210]: plugins/wifi.c:wifi_newlink() index 4 flags 4098 change 1
connmand[210]: plugins/wifi.c:wifi_newlink() interface down
connmand[210]: plugins/wifi.c:wifi_newlink() index 4 flags 69635 change 1
connmand[210]: plugins/wifi.c:wifi_newlink() interface up
connmand[210]: plugins/wifi.c:wifi_newlink() carrier on
connmand[210]: plugins/wifi.c:handle_tethering() index 4 bridge tether

However, in 3.5 we have the following sequents of events from the kernel:

1. IFF_UP, ifi_change=1
2. IFF_UP,IFF_LOWER_UP, ifi_change=0

which makes connman do the following:

connmand[493]: plugins/wifi.c:wifi_newlink() index 4 flags 4099 change 1
connmand[493]: plugins/wifi.c:wifi_newlink() interface up
connmand[493]: plugins/wifi.c:wifi_newlink() index 4 flags 69635 change 0

The root-cause for it is that connman handles the "ifi_change" flag
incorrectly. Connman interprets it as "if non-zero, there was some change",
which is wrong. According to RFC 3549, it is "reserved for future use.
Must be set to 0xFFFFFFFF". Thus, just remove that check, which makes
tethering work.

Release 1.7

TODO: Add entry for multiple agent support

doc: Remove copies of DNS and DHCP specifications

build: Include --enable-session-policy when building distribution

doc: Remove interactive command from client documentation

client: Enter interactive shell when no command is specified

build: Don't install connmanctl.1 manual page for now

TODO: Add two logging related tasks

client: Add printout about connmanctl experimental status

The arguments and output may see changes, warn users that the
current ones are experimental.

TODO: Update TODO with connmanctl agent mode entry

client: Change 'tech' to 'technologies'

Update argument and man page to use 'technologies'

AUTHORS: Add Tudor and Ceara to AUTHORS file

Update gitignore with new CLI name.

client: Update Makefile.am and add new command line client main file

The new main.c uses the previously added services, technology, manager,
interactive, and monitor files to create the connmanctl command line
interface. It is able to display properties of services and technologies,
and set/enable them.

configure.ac: Add readline check

doc: Add manpage for command line client

dbus.h: Update include/dbus.h with more container support

Add a couple more functions for opening D-Bus message containers
with different signatures currently used by the command line client.

client: Add interactive mode support

When the client is started with the 'interactive' option, it drops
into interactive shell mode. The interactive commands are handled
using readline. All of the commands (except interactive) can be
used in this mode.