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

stktest: Add PlayTone test sequence 1.1.1-1.1.8

unit: Move PlayTone test data for test sequence 1.1

stk: Tweak the concept of continuous tones

The spec explicitly mentions continuous or repeatable tones.  02.40 only
mentions the RP-ACK tone as a single tone, all other tones seem to be
repeatable

stktest: Add stk_add_timed_test

stktest: Add PlayTone and LoopTone methods

stktest: Add More Time test sequence 1.1

unit: Move More Time test data

doc: Update hardware support information

systemd: prevent duplicate logging messages in journal

By default, both stderr 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.

stktest: Fix unused variable warnings

stktest: Add GetInput test sequence 12.1-12.2

unit: Move GetInput test data 12.1-12.2

stktest: Add GetInput test sequence 11.1-11.2

unit: Move GetInput 11.1-11.2 test data

stktest: Add GetInput test sequence 10.1-10.2

unit: Move GetInput 10.1-10.2 test data

unit: Add GetInput test sequence 9.1-9.2

unit: Move GetInput 9.1-9.2 test data