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

stktest: Add GetInput test sequence 8.1-8.10

unit: Move GetInput test data 8.1-8.10

stktest: Add GetInput test sequence 6.1-6.4

unit: Move GetInput 6.1-6.4 test data

stktest: Add GetInput test sequence 5.1-5.2

unit: Move GetInput 5.1-5.2 test data

stktest: Add GetInput test sequence 4.1-4.2

unit: Move GetInput 4.1-4.2 test data

stktest: Add GetInput test sequence 3.1-3.2

unit: Move GetInput 3.1-3.2 test data

stktest: Add GetInput test sequence 2.1

unit: Move GetInput 2.1 test data

stktest: Add GetInput test sequence 1.10

unit: Move GetInput test data 1.10

stktest: Add GetInput test sequence 1.9

unit: Tweak unit test to match test data

unit: Move GetInput 1.9 test data

stktest: Add GetInput test sequence 1.8

unit: Move GetInput 1.8 test data

stktest: Add GetInput test sequence 1.7

unit: Move GetInput 1.7 test data

stktest: Add GetInput test sequence 1.6

unit: Move GetInput 1.6 test data

stktest: Add GetInput test sequence 1.6

unit: Move GetInput 1.5 test data

stktest: Add GetInput test sequence 1.4

unit: Move GetInput 1.4 test data

stktest: Add GetInput test sequence 1.3

unit: Move GetInput 1.3 test data

stktest: Add GetInput test sequence 1.2

unit: Move GetInput 1.2 test data

stktest: Add GetInput test sequence 1.1

unit: Move GetInput 1.1 test data

stktest: Be more explicit with errors

stktest: Add RequestInput and RequestDigits

udevng: Fix MBM detection

stktest: Fix unused variable warnings