gdbus: Add g_dbus_get_properties function

This function can be used to construct custom D-Bus messages containing
the properties for a specific interface on a given path.

gdbus: Add support for invalidated properties

If there's a pending property but its exists() callback returns false
the property should be considered invalidated and included in the
relevant list of the PropertiesChanged signal.

gdbus: Fix processing pending properties in remove_interface()

gdbus: Fix up Properties.Set() code path

Minor fixes to make setter actually work:

- Add propdata in pending_property_set
- Break loop when we are removing propdata from list and we
  found it
- in_args and out_args were swapped
- interface and method name arguments were swapped

gdbus: Fix invalid memory access while unregistering

If an interface is added and removed on the same mailoop iteration,
ObjectManager would try to send InterfacesAdded signal while running the
idler because the interface was added to data->added list.

This is easily reproduced by forcing an error path in a plugin
registration, like on sap_server_register(), resulting in the following
error:

==11795== Invalid read of size 4
==11795==    at 0x496F592: dbus_message_iter_append_basic (dbus-message.c:2598)
==11795==    by 0x117B39: append_interface (object.c:554)
==11795==    by 0x48955E7: g_slist_foreach (gslist.c:840)
==11795==    by 0x11923B: process_changes (object.c:592)
==11795==    by 0x11956D: generic_unregister (object.c:980)
==11795==    by 0x4973BAC: _dbus_object_tree_unregister_and_unlock (dbus-object-tree.c:516)
==11795==    by 0x4965240: dbus_connection_unregister_object_path (dbus-connection.c:5776)
==11795==    by 0x1178A5: object_path_unref (object.c:1219)
==11795==    by 0x118517: g_dbus_unregister_interface (object.c:1344)
==11795==    by 0x19AF5B: sap_exit (sap.c:385)
==11795==    by 0x13E9E2: sap_server_register (server.c:1428)
==11795==    by 0x13C092: sap_server_probe (manager.c:44)

With this patch we don't send the InterfacesAdded signal, removing it
from data->added while unregistering.

gdbus: Implement PropertiesChanged signal

gdbus: Simplify code for appending properties

This reuse append_properties for GetAll and GetManagedObjects

gdbus: Integrates ObjectManager with Properties interface

This appends the properties and its values when using ObjectManager.

gdbus: Only export ObjectManager interface on root path

ObjectManager should be exported only in the root path and list all
the children paths.

gdbus: Group interface changes to reduce the amount of signals emitted

InterfacesAdded and InterfacesRemoved can group all the interfaces
changes together in one message.

gdbus: Add support for org.freedesktop.DBus.ObjectManager interface

This implements initial support for ObjectManager, it automatically adds
objects to its parents so no action is needed by daemons to get their
objects managed by this interface.

ObjectManager is part of D-Bus spec since revision 0.17:
http://dbus.freedesktop.org/doc/dbus-specification.html#standard-interfaces-objectmanager

gdbus: Add properties into Introspectable interface

gdbus: Implement DBus.Properties.Set method

Contrary to Get() and GetAll(), Set() is asynchronous so we pass an id
to the setter so later it can declare the Set() as successful or
otherwise.

gdbus: Implement DBus.Properties.GetAll method

gdbus: Implement DBus.Properties.Get method

gdbus: Add skeleton of DBus.Properties interface

This interface is responsible for handling properties of all objects in
a given path. Right now it only registers itself, doing nothing useful.
A conversion to this new layout will be done by subsequent patches.

org.freedesktop.org.DBus.Properties spec can be found at
http://dbus.freedesktop.org/doc/dbus-specification.html#standard-interfaces-properties

gdbus: Use macros to add annotations

Besides being more readable this way it avoids going over 80 chars.

gdbus: Move typedefs up

Move the typedefs up so they can be used by functions and callbacks.

unit: Fix SMS unit tests

atmodem: Always quote data in CRSM write operations

atmodem: Support CRSM with paths

For GET STATUS and READ BINARY variations, support appending of the
optional path in the CRSM command

sim: Read icons with a full SIM/USIM path

simfs: Handle optional EF paths

include: Tweak ofono_sim_read_bytes API

Reading of icons requires us to specify the path

qmimodem: Add support for UIM service handling

udev: Let Huawei detection also check for cdc_wdm drivers

udev: Use Qualcomm Gobi driver if QMI based modem

udev: Handle USB descriptors from Vodafone K5005 modem

gobi: Use UIM service is available

sim: Assume CPHS files are in ADFusim

test: Fix modem path handling for icon testing

test: Allow STK testing without voice call manager

qmimodem: Print error string instead of code if available

qmimodem: Pretty print service name in debug logs

qmimodem: Fix debug printing of messages

qmimodem: Add helper for error message strings

unit: Add 2G and 3G path lookup tests

unit: Fix Style

unit: Remove failing test

We removed the needed file id from the sim database.  It was not being
used.

simutil: Add 2G and 3G path lookup utilities

simfs: Update to the new SIM api

sim: Update to the new sim reading API

atmodem: Update sim driver to the new api

qmi: Update sim-legacy driver to the new API

isi: Update UICC driver to the new API

isimodem: Update sim driver to the new API

include: Add path and path_len arguments

The CRSM command has an additional parameter called pathlen which can be
used to specify the path of the elementary file outside the well-known
path range.

simutil: Introduce 2G and 3G parent paths

test: Allow DisplayText to be interrupted by incoming calls

voicecall: Handle RESETTING sim state

modem: Handle RESETTING sim state

test: Fix ascii conversion exception

simutil: Add file type information to sim_db

simutil: Rename structure variable

ifx: Move connman atom to post_sim state

ifx: Get rid of have_sim tracking

sim: Fix SIM re-init case of SIM Refresh

When the SIM is being refreshed, we try to access the SIM too fast after
the SIM REFRESH proactive command is received.  Instead set the sim atom
into the 'RESETTING' state and wait until the modem driver signals the
sim insertion again.

include: Add RESETTING state

stk: Fix crash in SIM refresh handling

stk: Always include IdleModeText in GetProperties

sim: destroy spn info as part of sim main state

stk: Use the shorter timeout for select item

stk: use longer timeout for ImmediateResponse

stk: Use short timeout for get_inkey

stk: Use a shorter timeout for DisplayText

For any commands that are tagged as wait-for-user to clear, we should
use the shorter timeout.  The test system expects 25 to 35 seconds.

ifx: Add comments about unhandled SIM states

atmodem: Monitor IFX technology and band changes

test: Handle DisplayText with async_callbacks option

atmodem: add PIN retry count for Alcatel modems

alcatel: create sim atom with vendor alcatel

AUTHORS: Mention Cedric's contributions

atmodem: Poll SIM state after entering PIN

Encountered a problem of CME ERROR 14: SIM busy on Alcatel and Huawei modem.
The Huawei modem has a ^SIMST unsollicited sim state indication, but not all
Huawei modems support this.
So poll the SIM state, as was already done for ZTE modems.

atmodem: Add ALCATEL vendor

mbm: Fix SIM not inserted detection

stktest: Add Poll Interval test sequence 1.1

unit: Move Poll Interval test data 1.1

stktest: Label stktest modem as type TEST

modem: Add support for modem type TEST

doc: Update modem API to reflect modem type test

include: Add TEST modem type

stktest: Add Play Tone test sequence 6.1

unit: Move Play Tone 6.1 test data

stktest: Add Play Tone test sequence 5.1

unit: Move Play Tone 5.1 test data

stktest: Add PlayTone test sequence 4.1-4.10

unit: Fixup unit test PlayTone 4.10

unit: Move Play Tone 4.1-4.10 test data

AUTHORS: Mention Holger's contributions

wavecom: Add peculiar CPIN as terminator behavior

sim: Fix the quirk handling of CPIN for Wavecom

The Wavecom Q2XXX support broke in commit 72ce19bf3e87b5e73c053f7ea1.
This is because at_cpin_cb called decode_at_error with final and not
with OK. This lead to an error being set in the error variable and the
new code returns early when an error is set.

The addition of the terminator in at_sim_probe for Wavecom broke in
git commit ac524be99f8c72a2593e4ffcecad8beea7679e55 because
terminators can not be added on cloned chats.

Move the addition of the terminator from the atmodem to the wavecom
plugin. Use the same terminator for Q2XXX and the normal Wavecom
class. The WAVECOM terminator has been tested on a Q2XXX modem.

Apply the CPIN quirk for both WAVECOM and WAVECOM_Q2XXX inside the
sim.c file. Introduce needs_wavecom_sim_quirk to handle it for
WAVECOM and WAVECOM_Q2XXX.

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.

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)

stktest: Add Play Tone test sequence 3.1-3.4

unit: Move Play Tone 3.1-3.4 test data

stktest: Add PlayTone test sequence 2.1

unit: Move PlayTone 2.1 test data

stktest: Use a define for Cyrillic test data

stktest: Add Play Tone test sequence 1.1.9-1.1.15