diff options
| author | Aleksander Morgado <aleksander@aleksander.es> | 2021-03-04 18:32:56 +0100 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2021-03-04 20:45:00 +0000 |
| commit | 55d60f3c1201a4eb80f82da9d2abcf503a7de3f0 (patch) | |
| tree | 4ab2b8dd7a231981cb77460d64d0f31662825998 /src/mm-base-sms.c | |
| parent | 5d176a1e61df603b74434dce9050f7a9d3139f13 (diff) | |
sms: common timeout of 180s for the send operation
On low signal quality conditions, the process of sending the SMS to
the network may take a really long time, way more than the 30s used as
default in some implementations.
We now define a common timeout value of 180s for this operation in all
protocols.
Fixes https://gitlab.freedesktop.org/mobile-broadband/ModemManager/-/issues/328
Diffstat (limited to 'src/mm-base-sms.c')
| -rw-r--r-- | src/mm-base-sms.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/src/mm-base-sms.c b/src/mm-base-sms.c index ad6bba26..1c291daf 100644 --- a/src/mm-base-sms.c +++ b/src/mm-base-sms.c @@ -1145,7 +1145,7 @@ send_generic_ready (MMBaseModem *modem, * sent right away (not queued after other AT commands). */ mm_base_modem_at_command_raw (ctx->modem, ctx->msg_data, - 60, + MM_BASE_SMS_DEFAULT_SEND_TIMEOUT, FALSE, (GAsyncReadyCallback)send_generic_msg_data_ready, task); @@ -1219,7 +1219,7 @@ sms_send_next_part (GTask *task) mm_sms_part_get_index ((MMSmsPart *)ctx->current->data)); mm_base_modem_at_command (ctx->modem, cmd, - 60, + MM_BASE_SMS_DEFAULT_SEND_TIMEOUT, FALSE, (GAsyncReadyCallback)send_from_storage_ready, task); @@ -1245,9 +1245,11 @@ sms_send_next_part (GTask *task) g_assert (cmd != NULL); g_assert (ctx->msg_data != NULL); + + /* no network involved in this initial AT command, so lower timeout */ mm_base_modem_at_command (ctx->modem, cmd, - 60, + 10, FALSE, (GAsyncReadyCallback)send_generic_ready, task); |