Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 1 | /* SMS queue to continously attempt to deliver SMS */ |
| 2 | /* |
| 3 | * (C) 2010 by Holger Hans Peter Freyther <zecke@selfish.org> |
| 4 | * All Rights Reserved |
| 5 | * |
| 6 | * This program is free software; you can redistribute it and/or modify |
Harald Welte | 9af6ddf | 2011-01-01 15:25:50 +0100 | [diff] [blame] | 7 | * it under the terms of the GNU Affero General Public License as published by |
| 8 | * the Free Software Foundation; either version 3 of the License, or |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 9 | * (at your option) any later version. |
| 10 | * |
| 11 | * This program is distributed in the hope that it will be useful, |
| 12 | * but WITHOUT ANY WARRANTY; without even the implied warranty of |
| 13 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the |
Harald Welte | 9af6ddf | 2011-01-01 15:25:50 +0100 | [diff] [blame] | 14 | * GNU Affero General Public License for more details. |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 15 | * |
Harald Welte | 9af6ddf | 2011-01-01 15:25:50 +0100 | [diff] [blame] | 16 | * You should have received a copy of the GNU Affero General Public License |
| 17 | * along with this program. If not, see <http://www.gnu.org/licenses/>. |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 18 | * |
| 19 | */ |
| 20 | |
| 21 | /** |
| 22 | * The difficulty of such a queue is to send a lot of SMS without |
| 23 | * overloading the paging subsystem and the database and other users |
| 24 | * of the MSC. To make the best use we would need to know the number |
| 25 | * of pending paging requests, then throttle the number of SMS we |
| 26 | * want to send and such. |
| 27 | * We will start with a very simple SMS Queue and then try to speed |
| 28 | * things up by collecting data from other parts of the system. |
| 29 | */ |
| 30 | |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 31 | #include <limits.h> |
| 32 | |
Neels Hofmeyr | 9084396 | 2017-09-04 15:04:35 +0200 | [diff] [blame] | 33 | #include <osmocom/msc/sms_queue.h> |
Neels Hofmeyr | 9084396 | 2017-09-04 15:04:35 +0200 | [diff] [blame] | 34 | #include <osmocom/msc/db.h> |
| 35 | #include <osmocom/msc/debug.h> |
| 36 | #include <osmocom/msc/gsm_data.h> |
| 37 | #include <osmocom/msc/gsm_04_11.h> |
| 38 | #include <osmocom/msc/gsm_subscriber.h> |
| 39 | #include <osmocom/msc/signal.h> |
| 40 | #include <osmocom/msc/vlr.h> |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 41 | |
Pablo Neira Ayuso | 136f453 | 2011-03-22 16:47:59 +0100 | [diff] [blame] | 42 | #include <osmocom/core/talloc.h> |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 43 | |
Holger Hans Peter Freyther | 81c0e25 | 2010-12-25 14:08:00 +0100 | [diff] [blame] | 44 | #include <osmocom/vty/vty.h> |
| 45 | |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 46 | /* |
| 47 | * One pending SMS that we wait for. |
| 48 | */ |
| 49 | struct gsm_sms_pending { |
| 50 | struct llist_head entry; |
| 51 | |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 52 | struct vlr_subscr *vsub; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 53 | unsigned long long sms_id; |
| 54 | int failed_attempts; |
| 55 | int resend; |
| 56 | }; |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 57 | |
| 58 | struct gsm_sms_queue { |
Pablo Neira Ayuso | bf540cb | 2011-05-06 12:11:06 +0200 | [diff] [blame] | 59 | struct osmo_timer_list resend_pending; |
| 60 | struct osmo_timer_list push_queue; |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 61 | struct gsm_network *network; |
Holger Hans Peter Freyther | 7e59c83 | 2010-12-25 14:46:54 +0100 | [diff] [blame] | 62 | int max_fail; |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 63 | int max_pending; |
| 64 | int pending; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 65 | |
| 66 | struct llist_head pending_sms; |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 67 | |
Neels Hofmeyr | 7b61ffe | 2018-11-30 02:46:53 +0100 | [diff] [blame] | 68 | char last_msisdn[VLR_MSISDN_LENGTH+1]; |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 69 | }; |
| 70 | |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 71 | static int sms_subscr_cb(unsigned int, unsigned int, void *, void *); |
| 72 | static int sms_sms_cb(unsigned int, unsigned int, void *, void *); |
| 73 | |
| 74 | static struct gsm_sms_pending *sms_find_pending(struct gsm_sms_queue *smsq, |
Stefan Sperling | 87cba1f | 2018-01-22 17:05:37 +0100 | [diff] [blame] | 75 | unsigned long long sms_id) |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 76 | { |
| 77 | struct gsm_sms_pending *pending; |
| 78 | |
| 79 | llist_for_each_entry(pending, &smsq->pending_sms, entry) { |
Stefan Sperling | 87cba1f | 2018-01-22 17:05:37 +0100 | [diff] [blame] | 80 | if (pending->sms_id == sms_id) |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 81 | return pending; |
| 82 | } |
| 83 | |
| 84 | return NULL; |
| 85 | } |
| 86 | |
Stefan Sperling | 87cba1f | 2018-01-22 17:05:37 +0100 | [diff] [blame] | 87 | int sms_queue_sms_is_pending(struct gsm_sms_queue *smsq, unsigned long long sms_id) |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 88 | { |
Stefan Sperling | 87cba1f | 2018-01-22 17:05:37 +0100 | [diff] [blame] | 89 | return sms_find_pending(smsq, sms_id) != NULL; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 90 | } |
| 91 | |
Holger Hans Peter Freyther | 074b2b2 | 2011-07-25 00:13:06 +0200 | [diff] [blame] | 92 | static struct gsm_sms_pending *sms_subscriber_find_pending( |
| 93 | struct gsm_sms_queue *smsq, |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 94 | struct vlr_subscr *vsub) |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 95 | { |
| 96 | struct gsm_sms_pending *pending; |
| 97 | |
| 98 | llist_for_each_entry(pending, &smsq->pending_sms, entry) { |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 99 | if (pending->vsub == vsub) |
Holger Hans Peter Freyther | 074b2b2 | 2011-07-25 00:13:06 +0200 | [diff] [blame] | 100 | return pending; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 101 | } |
| 102 | |
Holger Hans Peter Freyther | 074b2b2 | 2011-07-25 00:13:06 +0200 | [diff] [blame] | 103 | return NULL; |
| 104 | } |
| 105 | |
| 106 | static int sms_subscriber_is_pending(struct gsm_sms_queue *smsq, |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 107 | struct vlr_subscr *vsub) |
Holger Hans Peter Freyther | 074b2b2 | 2011-07-25 00:13:06 +0200 | [diff] [blame] | 108 | { |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 109 | return sms_subscriber_find_pending(smsq, vsub) != NULL; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 110 | } |
| 111 | |
| 112 | static struct gsm_sms_pending *sms_pending_from(struct gsm_sms_queue *smsq, |
| 113 | struct gsm_sms *sms) |
| 114 | { |
| 115 | struct gsm_sms_pending *pending; |
| 116 | |
| 117 | pending = talloc_zero(smsq, struct gsm_sms_pending); |
| 118 | if (!pending) |
| 119 | return NULL; |
| 120 | |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 121 | pending->vsub = vlr_subscr_get(sms->receiver); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 122 | pending->sms_id = sms->id; |
| 123 | return pending; |
| 124 | } |
| 125 | |
| 126 | static void sms_pending_free(struct gsm_sms_pending *pending) |
| 127 | { |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 128 | vlr_subscr_put(pending->vsub); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 129 | llist_del(&pending->entry); |
| 130 | talloc_free(pending); |
| 131 | } |
| 132 | |
| 133 | static void sms_pending_resend(struct gsm_sms_pending *pending) |
| 134 | { |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 135 | struct gsm_network *net = pending->vsub->vlr->user_ctx; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 136 | struct gsm_sms_queue *smsq; |
Holger Hans Peter Freyther | eff40949 | 2012-11-10 19:46:58 +0100 | [diff] [blame] | 137 | LOGP(DLSMS, LOGL_DEBUG, |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 138 | "Scheduling resend of SMS %llu.\n", pending->sms_id); |
| 139 | |
| 140 | pending->resend = 1; |
| 141 | |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 142 | smsq = net->sms_queue; |
Pablo Neira Ayuso | bf540cb | 2011-05-06 12:11:06 +0200 | [diff] [blame] | 143 | if (osmo_timer_pending(&smsq->resend_pending)) |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 144 | return; |
| 145 | |
Pablo Neira Ayuso | bf540cb | 2011-05-06 12:11:06 +0200 | [diff] [blame] | 146 | osmo_timer_schedule(&smsq->resend_pending, 1, 0); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 147 | } |
| 148 | |
| 149 | static void sms_pending_failed(struct gsm_sms_pending *pending, int paging_error) |
| 150 | { |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 151 | struct gsm_network *net = pending->vsub->vlr->user_ctx; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 152 | struct gsm_sms_queue *smsq; |
| 153 | |
Neels Hofmeyr | 34a36da | 2019-03-06 16:00:41 +0100 | [diff] [blame] | 154 | pending->failed_attempts++; |
Holger Hans Peter Freyther | eff40949 | 2012-11-10 19:46:58 +0100 | [diff] [blame] | 155 | LOGP(DLSMS, LOGL_NOTICE, "Sending SMS %llu failed %d times.\n", |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 156 | pending->sms_id, pending->failed_attempts); |
| 157 | |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 158 | smsq = net->sms_queue; |
Neels Hofmeyr | 34a36da | 2019-03-06 16:00:41 +0100 | [diff] [blame] | 159 | if (pending->failed_attempts < smsq->max_fail) |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 160 | return sms_pending_resend(pending); |
| 161 | |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 162 | sms_pending_free(pending); |
| 163 | smsq->pending -= 1; |
| 164 | sms_queue_trigger(smsq); |
| 165 | } |
| 166 | |
| 167 | /* |
| 168 | * Resend all SMS that are scheduled for a resend. This is done to |
| 169 | * avoid an immediate failure. |
| 170 | */ |
| 171 | static void sms_resend_pending(void *_data) |
| 172 | { |
| 173 | struct gsm_sms_pending *pending, *tmp; |
| 174 | struct gsm_sms_queue *smsq = _data; |
| 175 | |
| 176 | llist_for_each_entry_safe(pending, tmp, &smsq->pending_sms, entry) { |
| 177 | struct gsm_sms *sms; |
| 178 | if (!pending->resend) |
| 179 | continue; |
| 180 | |
| 181 | sms = db_sms_get(smsq->network, pending->sms_id); |
| 182 | |
| 183 | /* the sms is gone? Move to the next */ |
| 184 | if (!sms) { |
| 185 | sms_pending_free(pending); |
| 186 | smsq->pending -= 1; |
| 187 | sms_queue_trigger(smsq); |
| 188 | } else { |
| 189 | pending->resend = 0; |
Vadim Yanitskiy | 24e025e | 2018-11-22 15:42:39 +0700 | [diff] [blame] | 190 | gsm411_send_sms(smsq->network, sms->receiver, sms); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 191 | } |
| 192 | } |
| 193 | } |
| 194 | |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 195 | /* Find the next pending SMS by cycling through the recipients. We could also |
| 196 | * cycle through the pending SMS, but that might cause us to keep trying to |
| 197 | * send SMS to the same few subscribers repeatedly while not servicing other |
| 198 | * subscribers for a long time. By walking the list of recipient MSISDNs, we |
| 199 | * ensure that all subscribers get their fair time to receive SMS. */ |
| 200 | struct gsm_sms *smsq_take_next_sms(struct gsm_network *net, |
| 201 | char *last_msisdn, |
| 202 | size_t last_msisdn_buflen) |
Holger Hans Peter Freyther | f7e2389 | 2010-12-25 17:45:23 +0100 | [diff] [blame] | 203 | { |
| 204 | struct gsm_sms *sms; |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 205 | int wrapped = 0; |
| 206 | int sanity = 100; |
| 207 | char started_with_msisdn[last_msisdn_buflen]; |
Holger Hans Peter Freyther | f7e2389 | 2010-12-25 17:45:23 +0100 | [diff] [blame] | 208 | |
Max | 98f7467 | 2018-02-05 12:57:06 +0100 | [diff] [blame] | 209 | OSMO_STRLCPY_ARRAY(started_with_msisdn, last_msisdn); |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 210 | |
| 211 | while (wrapped < 2 && (--sanity)) { |
| 212 | /* If we wrapped around and passed the first msisdn, we're |
| 213 | * through the entire SMS DB; end it. */ |
| 214 | if (wrapped && strcmp(last_msisdn, started_with_msisdn) >= 0) |
| 215 | break; |
| 216 | |
| 217 | sms = db_sms_get_next_unsent_rr_msisdn(net, last_msisdn, 9); |
| 218 | if (!sms) { |
| 219 | last_msisdn[0] = '\0'; |
Max | 5e2e9bd | 2018-02-06 19:31:08 +0100 | [diff] [blame] | 220 | wrapped++; |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 221 | continue; |
| 222 | } |
| 223 | |
| 224 | /* Whatever happens, next time around service another recipient |
| 225 | */ |
| 226 | osmo_strlcpy(last_msisdn, sms->dst.addr, last_msisdn_buflen); |
| 227 | |
| 228 | /* Is the subscriber attached? If not, go to next SMS */ |
Vadim Yanitskiy | 96262a7 | 2019-03-28 21:25:14 +0700 | [diff] [blame] | 229 | if (!sms->receiver || !sms->receiver->lu_complete) { |
| 230 | LOGP(DLSMS, LOGL_DEBUG, |
| 231 | "Subscriber %s is not attached, skipping SMS %llu\n", |
| 232 | vlr_subscr_msisdn_or_name(sms->receiver), sms->id); |
| 233 | sms_free(sms); |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 234 | continue; |
Vadim Yanitskiy | 96262a7 | 2019-03-28 21:25:14 +0700 | [diff] [blame] | 235 | } |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 236 | |
Holger Hans Peter Freyther | f7e2389 | 2010-12-25 17:45:23 +0100 | [diff] [blame] | 237 | return sms; |
| 238 | } |
| 239 | |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 240 | DEBUGP(DLSMS, "SMS queue: no SMS to be sent\n"); |
| 241 | return NULL; |
Holger Hans Peter Freyther | f7e2389 | 2010-12-25 17:45:23 +0100 | [diff] [blame] | 242 | } |
| 243 | |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 244 | /** |
| 245 | * I will submit up to max_pending - pending SMS to the |
| 246 | * subsystem. |
| 247 | */ |
| 248 | static void sms_submit_pending(void *_data) |
| 249 | { |
| 250 | struct gsm_sms_queue *smsq = _data; |
| 251 | int attempts = smsq->max_pending - smsq->pending; |
Holger Hans Peter Freyther | 2038457 | 2010-12-25 19:28:44 +0100 | [diff] [blame] | 252 | int initialized = 0; |
| 253 | unsigned long long first_sub = 0; |
Holger Hans Peter Freyther | 5479fc8 | 2010-12-25 19:32:12 +0100 | [diff] [blame] | 254 | int attempted = 0, rounds = 0; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 255 | |
Harald Welte | e5d2c60 | 2012-11-13 06:52:42 +0100 | [diff] [blame] | 256 | LOGP(DLSMS, LOGL_DEBUG, "Attempting to send %d SMS\n", attempts); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 257 | |
Holger Hans Peter Freyther | 2038457 | 2010-12-25 19:28:44 +0100 | [diff] [blame] | 258 | do { |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 259 | struct gsm_sms_pending *pending; |
| 260 | struct gsm_sms *sms; |
| 261 | |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 262 | |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 263 | sms = smsq_take_next_sms(smsq->network, smsq->last_msisdn, |
| 264 | sizeof(smsq->last_msisdn)); |
Neels Hofmeyr | 1e918c3 | 2016-05-09 21:48:53 +0200 | [diff] [blame] | 265 | if (!sms) { |
| 266 | LOGP(DLSMS, LOGL_DEBUG, "Sending SMS done (%d attempted)\n", |
| 267 | attempted); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 268 | break; |
Neels Hofmeyr | 1e918c3 | 2016-05-09 21:48:53 +0200 | [diff] [blame] | 269 | } |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 270 | |
Holger Hans Peter Freyther | 5479fc8 | 2010-12-25 19:32:12 +0100 | [diff] [blame] | 271 | rounds += 1; |
Neels Hofmeyr | 1e918c3 | 2016-05-09 21:48:53 +0200 | [diff] [blame] | 272 | LOGP(DLSMS, LOGL_DEBUG, "Sending SMS round %d\n", rounds); |
Holger Hans Peter Freyther | 5479fc8 | 2010-12-25 19:32:12 +0100 | [diff] [blame] | 273 | |
Holger Hans Peter Freyther | 2038457 | 2010-12-25 19:28:44 +0100 | [diff] [blame] | 274 | /* |
| 275 | * This code needs to detect a loop. It assumes that no SMS |
| 276 | * will vanish during the time this is executed. We will remember |
| 277 | * the id of the first GSM subscriber we see and then will |
| 278 | * compare this. The Database code should make sure that we will |
| 279 | * see all other subscribers first before seeing this one again. |
| 280 | * |
| 281 | * It is always scary to have an infinite loop like this. |
| 282 | */ |
| 283 | if (!initialized) { |
| 284 | first_sub = sms->receiver->id; |
| 285 | initialized = 1; |
| 286 | } else if (first_sub == sms->receiver->id) { |
Neels Hofmeyr | 1e918c3 | 2016-05-09 21:48:53 +0200 | [diff] [blame] | 287 | LOGP(DLSMS, LOGL_DEBUG, "Sending SMS done (loop) (%d attempted)\n", |
| 288 | attempted); |
Holger Hans Peter Freyther | 2038457 | 2010-12-25 19:28:44 +0100 | [diff] [blame] | 289 | sms_free(sms); |
| 290 | break; |
| 291 | } |
| 292 | |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 293 | /* no need to send a pending sms */ |
Stefan Sperling | 87cba1f | 2018-01-22 17:05:37 +0100 | [diff] [blame] | 294 | if (sms_queue_sms_is_pending(smsq, sms->id)) { |
Holger Hans Peter Freyther | eff40949 | 2012-11-10 19:46:58 +0100 | [diff] [blame] | 295 | LOGP(DLSMS, LOGL_DEBUG, |
Holger Hans Peter Freyther | dc53af6 | 2010-12-27 20:12:25 +0100 | [diff] [blame] | 296 | "SMSqueue with pending sms: %llu. Skipping\n", sms->id); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 297 | sms_free(sms); |
| 298 | continue; |
| 299 | } |
| 300 | |
| 301 | /* no need to send a SMS with the same receiver */ |
| 302 | if (sms_subscriber_is_pending(smsq, sms->receiver)) { |
Holger Hans Peter Freyther | eff40949 | 2012-11-10 19:46:58 +0100 | [diff] [blame] | 303 | LOGP(DLSMS, LOGL_DEBUG, |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 304 | "SMSqueue with pending sub: %llu. Skipping\n", sms->receiver->id); |
| 305 | sms_free(sms); |
| 306 | continue; |
| 307 | } |
| 308 | |
| 309 | pending = sms_pending_from(smsq, sms); |
| 310 | if (!pending) { |
Holger Hans Peter Freyther | eff40949 | 2012-11-10 19:46:58 +0100 | [diff] [blame] | 311 | LOGP(DLSMS, LOGL_ERROR, |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 312 | "Failed to create pending SMS entry.\n"); |
| 313 | sms_free(sms); |
| 314 | continue; |
| 315 | } |
| 316 | |
Holger Hans Peter Freyther | 2038457 | 2010-12-25 19:28:44 +0100 | [diff] [blame] | 317 | attempted += 1; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 318 | smsq->pending += 1; |
Holger Hans Peter Freyther | 701076e | 2010-12-28 14:25:25 +0100 | [diff] [blame] | 319 | llist_add_tail(&pending->entry, &smsq->pending_sms); |
Vadim Yanitskiy | 24e025e | 2018-11-22 15:42:39 +0700 | [diff] [blame] | 320 | gsm411_send_sms(smsq->network, sms->receiver, sms); |
Holger Hans Peter Freyther | 5479fc8 | 2010-12-25 19:32:12 +0100 | [diff] [blame] | 321 | } while (attempted < attempts && rounds < 1000); |
Holger Hans Peter Freyther | 2038457 | 2010-12-25 19:28:44 +0100 | [diff] [blame] | 322 | |
Holger Hans Peter Freyther | eff40949 | 2012-11-10 19:46:58 +0100 | [diff] [blame] | 323 | LOGP(DLSMS, LOGL_DEBUG, "SMSqueue added %d messages in %d rounds\n", attempted, rounds); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 324 | } |
| 325 | |
Holger Hans Peter Freyther | 93de8b2 | 2014-02-24 16:13:04 +0100 | [diff] [blame] | 326 | /** |
| 327 | * Send the next SMS or trigger the queue |
| 328 | */ |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 329 | static void sms_send_next(struct vlr_subscr *vsub) |
Holger Hans Peter Freyther | 93de8b2 | 2014-02-24 16:13:04 +0100 | [diff] [blame] | 330 | { |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 331 | struct gsm_network *net = vsub->vlr->user_ctx; |
| 332 | struct gsm_sms_queue *smsq = net->sms_queue; |
Holger Hans Peter Freyther | 93de8b2 | 2014-02-24 16:13:04 +0100 | [diff] [blame] | 333 | struct gsm_sms_pending *pending; |
| 334 | struct gsm_sms *sms; |
| 335 | |
| 336 | /* the subscriber should not be in the queue */ |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 337 | OSMO_ASSERT(!sms_subscriber_is_pending(smsq, vsub)); |
Holger Hans Peter Freyther | 93de8b2 | 2014-02-24 16:13:04 +0100 | [diff] [blame] | 338 | |
| 339 | /* check for more messages for this subscriber */ |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 340 | sms = db_sms_get_unsent_for_subscr(vsub, UINT_MAX); |
Holger Hans Peter Freyther | 93de8b2 | 2014-02-24 16:13:04 +0100 | [diff] [blame] | 341 | if (!sms) |
| 342 | goto no_pending_sms; |
| 343 | |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 344 | /* The sms should not be scheduled right now */ |
Stefan Sperling | 87cba1f | 2018-01-22 17:05:37 +0100 | [diff] [blame] | 345 | OSMO_ASSERT(!sms_queue_sms_is_pending(smsq, sms->id)); |
Holger Hans Peter Freyther | 93de8b2 | 2014-02-24 16:13:04 +0100 | [diff] [blame] | 346 | |
| 347 | /* Remember that we deliver this SMS and send it */ |
| 348 | pending = sms_pending_from(smsq, sms); |
| 349 | if (!pending) { |
| 350 | LOGP(DLSMS, LOGL_ERROR, |
| 351 | "Failed to create pending SMS entry.\n"); |
| 352 | sms_free(sms); |
| 353 | goto no_pending_sms; |
| 354 | } |
| 355 | |
| 356 | smsq->pending += 1; |
| 357 | llist_add_tail(&pending->entry, &smsq->pending_sms); |
Vadim Yanitskiy | 24e025e | 2018-11-22 15:42:39 +0700 | [diff] [blame] | 358 | gsm411_send_sms(smsq->network, sms->receiver, sms); |
Holger Hans Peter Freyther | 93de8b2 | 2014-02-24 16:13:04 +0100 | [diff] [blame] | 359 | return; |
| 360 | |
| 361 | no_pending_sms: |
| 362 | /* Try to send the SMS to avoid the queue being stuck */ |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 363 | sms_submit_pending(net->sms_queue); |
Holger Hans Peter Freyther | 93de8b2 | 2014-02-24 16:13:04 +0100 | [diff] [blame] | 364 | } |
| 365 | |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 366 | /* |
| 367 | * Kick off the queue again. |
| 368 | */ |
| 369 | int sms_queue_trigger(struct gsm_sms_queue *smsq) |
| 370 | { |
Neels Hofmeyr | 1e918c3 | 2016-05-09 21:48:53 +0200 | [diff] [blame] | 371 | LOGP(DLSMS, LOGL_DEBUG, "Triggering SMS queue\n"); |
Pablo Neira Ayuso | bf540cb | 2011-05-06 12:11:06 +0200 | [diff] [blame] | 372 | if (osmo_timer_pending(&smsq->push_queue)) |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 373 | return 0; |
| 374 | |
Pablo Neira Ayuso | bf540cb | 2011-05-06 12:11:06 +0200 | [diff] [blame] | 375 | osmo_timer_schedule(&smsq->push_queue, 1, 0); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 376 | return 0; |
| 377 | } |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 378 | |
| 379 | int sms_queue_start(struct gsm_network *network, int max_pending) |
| 380 | { |
| 381 | struct gsm_sms_queue *sms = talloc_zero(network, struct gsm_sms_queue); |
| 382 | if (!sms) { |
| 383 | LOGP(DMSC, LOGL_ERROR, "Failed to create the SMS queue.\n"); |
| 384 | return -1; |
| 385 | } |
| 386 | |
Pablo Neira Ayuso | bbc5b99 | 2011-05-06 12:12:31 +0200 | [diff] [blame] | 387 | osmo_signal_register_handler(SS_SUBSCR, sms_subscr_cb, network); |
| 388 | osmo_signal_register_handler(SS_SMS, sms_sms_cb, network); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 389 | |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 390 | network->sms_queue = sms; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 391 | INIT_LLIST_HEAD(&sms->pending_sms); |
Holger Hans Peter Freyther | a37e3bc | 2010-12-25 17:43:03 +0100 | [diff] [blame] | 392 | sms->max_fail = 1; |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 393 | sms->network = network; |
| 394 | sms->max_pending = max_pending; |
Pablo Neira Ayuso | 5121576 | 2017-05-08 20:57:52 +0200 | [diff] [blame] | 395 | osmo_timer_setup(&sms->push_queue, sms_submit_pending, sms); |
| 396 | osmo_timer_setup(&sms->resend_pending, sms_resend_pending, sms); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 397 | |
| 398 | sms_submit_pending(sms); |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 399 | |
| 400 | return 0; |
| 401 | } |
| 402 | |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 403 | static int sub_ready_for_sm(struct gsm_network *net, struct vlr_subscr *vsub) |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 404 | { |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 405 | struct gsm_sms *sms; |
Holger Hans Peter Freyther | 074b2b2 | 2011-07-25 00:13:06 +0200 | [diff] [blame] | 406 | struct gsm_sms_pending *pending; |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 407 | |
Holger Hans Peter Freyther | 074b2b2 | 2011-07-25 00:13:06 +0200 | [diff] [blame] | 408 | /* |
| 409 | * The code used to be very clever and tried to submit |
| 410 | * a SMS during the Location Updating Request. This has |
| 411 | * two issues: |
| 412 | * 1.) The Phone might not be ready yet, e.g. the C155 |
| 413 | * will not respond to the Submit when it is booting. |
| 414 | * 2.) The queue is already trying to submit SMS to the |
| 415 | * user and by not responding to the paging request |
| 416 | * we will set the LAC back to 0. We would have to |
| 417 | * stop the paging and move things over. |
| 418 | * |
| 419 | * We need to be careful in what we try here. |
| 420 | */ |
| 421 | |
| 422 | /* check if we have pending requests */ |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 423 | pending = sms_subscriber_find_pending(net->sms_queue, vsub); |
Holger Hans Peter Freyther | 074b2b2 | 2011-07-25 00:13:06 +0200 | [diff] [blame] | 424 | if (pending) { |
| 425 | LOGP(DMSC, LOGL_NOTICE, |
| 426 | "Pending paging while subscriber %llu attached.\n", |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 427 | vsub->id); |
Holger Hans Peter Freyther | 074b2b2 | 2011-07-25 00:13:06 +0200 | [diff] [blame] | 428 | return 0; |
| 429 | } |
| 430 | |
Holger Hans Peter Freyther | 074b2b2 | 2011-07-25 00:13:06 +0200 | [diff] [blame] | 431 | /* Now try to deliver any pending SMS to this sub */ |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 432 | sms = db_sms_get_unsent_for_subscr(vsub, UINT_MAX); |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 433 | if (!sms) |
| 434 | return -1; |
Vadim Yanitskiy | 24e025e | 2018-11-22 15:42:39 +0700 | [diff] [blame] | 435 | |
| 436 | gsm411_send_sms(net, vsub, sms); |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 437 | return 0; |
| 438 | } |
| 439 | |
| 440 | static int sms_subscr_cb(unsigned int subsys, unsigned int signal, |
| 441 | void *handler_data, void *signal_data) |
| 442 | { |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 443 | struct vlr_subscr *vsub = signal_data; |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 444 | |
| 445 | if (signal != S_SUBSCR_ATTACHED) |
| 446 | return 0; |
| 447 | |
| 448 | /* this is readyForSM */ |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 449 | return sub_ready_for_sm(handler_data, vsub); |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 450 | } |
| 451 | |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 452 | static int sms_sms_cb(unsigned int subsys, unsigned int signal, |
| 453 | void *handler_data, void *signal_data) |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 454 | { |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 455 | struct gsm_network *network = handler_data; |
| 456 | struct sms_signal_data *sig_sms = signal_data; |
| 457 | struct gsm_sms_pending *pending; |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 458 | struct vlr_subscr *vsub; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 459 | |
| 460 | /* We got a new SMS and maybe should launch the queue again. */ |
| 461 | if (signal == S_SMS_SUBMITTED || signal == S_SMS_SMMA) { |
Holger Hans Peter Freyther | 024dc77 | 2014-02-24 14:29:27 +0100 | [diff] [blame] | 462 | /* TODO: For SMMA we might want to re-use the radio connection. */ |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 463 | sms_queue_trigger(network->sms_queue); |
| 464 | return 0; |
| 465 | } |
| 466 | |
| 467 | if (!sig_sms->sms) |
| 468 | return -1; |
| 469 | |
| 470 | |
| 471 | /* |
| 472 | * Find the entry of our queue. The SMS subsystem will submit |
| 473 | * sms that are not in our control as we just have a channel |
| 474 | * open anyway. |
| 475 | */ |
Stefan Sperling | 87cba1f | 2018-01-22 17:05:37 +0100 | [diff] [blame] | 476 | pending = sms_find_pending(network->sms_queue, sig_sms->sms->id); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 477 | if (!pending) |
| 478 | return 0; |
| 479 | |
| 480 | switch (signal) { |
| 481 | case S_SMS_DELIVERED: |
Holger Hans Peter Freyther | 93de8b2 | 2014-02-24 16:13:04 +0100 | [diff] [blame] | 482 | /* Remember the subscriber and clear the pending entry */ |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 483 | network->sms_queue->pending -= 1; |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 484 | vsub = vlr_subscr_get(pending->vsub); |
Stefan Sperling | 6ba2d5a | 2018-01-18 18:55:26 +0100 | [diff] [blame] | 485 | db_sms_delete_sent_message_by_id(pending->sms_id); |
Harald Welte | 37a3ae8 | 2018-04-14 20:53:36 +0200 | [diff] [blame] | 486 | sms_pending_free(pending); |
Holger Hans Peter Freyther | 93de8b2 | 2014-02-24 16:13:04 +0100 | [diff] [blame] | 487 | /* Attempt to send another SMS to this subscriber */ |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 488 | sms_send_next(vsub); |
| 489 | vlr_subscr_put(vsub); |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 490 | break; |
| 491 | case S_SMS_MEM_EXCEEDED: |
| 492 | network->sms_queue->pending -= 1; |
| 493 | sms_pending_free(pending); |
| 494 | sms_queue_trigger(network->sms_queue); |
| 495 | break; |
| 496 | case S_SMS_UNKNOWN_ERROR: |
| 497 | /* |
| 498 | * There can be many reasons for this failure. E.g. the paging |
| 499 | * timed out, the subscriber was not paged at all, or there was |
| 500 | * a protocol error. The current strategy is to try sending the |
| 501 | * next SMS for busy/oom and to retransmit when we have paged. |
| 502 | * |
| 503 | * When the paging expires three times we will disable the |
| 504 | * subscriber. If we have some kind of other transmit error we |
| 505 | * should flag the SMS as bad. |
| 506 | */ |
| 507 | switch (sig_sms->paging_result) { |
| 508 | case 0: |
| 509 | /* BAD SMS? */ |
| 510 | db_sms_inc_deliver_attempts(sig_sms->sms); |
| 511 | sms_pending_failed(pending, 0); |
| 512 | break; |
| 513 | case GSM_PAGING_EXPIRED: |
| 514 | sms_pending_failed(pending, 1); |
| 515 | break; |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 516 | case GSM_PAGING_BUSY: |
| 517 | network->sms_queue->pending -= 1; |
| 518 | sms_pending_free(pending); |
| 519 | sms_queue_trigger(network->sms_queue); |
| 520 | break; |
| 521 | default: |
Holger Hans Peter Freyther | eff40949 | 2012-11-10 19:46:58 +0100 | [diff] [blame] | 522 | LOGP(DLSMS, LOGL_ERROR, "Unhandled result: %d\n", |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 523 | sig_sms->paging_result); |
| 524 | } |
| 525 | break; |
| 526 | default: |
Holger Hans Peter Freyther | eff40949 | 2012-11-10 19:46:58 +0100 | [diff] [blame] | 527 | LOGP(DLSMS, LOGL_ERROR, "Unhandled result: %d\n", |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 528 | sig_sms->paging_result); |
| 529 | } |
| 530 | |
Stefan Sperling | 14e0517 | 2018-01-22 17:31:20 +0100 | [diff] [blame] | 531 | /* While here, attempt to remove an expired SMS from the DB. */ |
| 532 | db_sms_delete_oldest_expired_message(); |
| 533 | |
Holger Hans Peter Freyther | 1716406 | 2010-12-24 21:39:55 +0100 | [diff] [blame] | 534 | return 0; |
Holger Hans Peter Freyther | 11b28f9 | 2010-12-24 13:48:27 +0100 | [diff] [blame] | 535 | } |
Holger Hans Peter Freyther | 81c0e25 | 2010-12-25 14:08:00 +0100 | [diff] [blame] | 536 | |
| 537 | /* VTY helper functions */ |
| 538 | int sms_queue_stats(struct gsm_sms_queue *smsq, struct vty *vty) |
| 539 | { |
| 540 | struct gsm_sms_pending *pending; |
| 541 | |
| 542 | vty_out(vty, "SMSqueue with max_pending: %d pending: %d%s", |
| 543 | smsq->max_pending, smsq->pending, VTY_NEWLINE); |
| 544 | |
| 545 | llist_for_each_entry(pending, &smsq->pending_sms, entry) |
Holger Hans Peter Freyther | 583e9ae | 2010-12-27 20:19:48 +0100 | [diff] [blame] | 546 | vty_out(vty, " SMS Pending for Subscriber: %llu SMS: %llu Failed: %d.%s", |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 547 | pending->vsub->id, pending->sms_id, |
Holger Hans Peter Freyther | 583e9ae | 2010-12-27 20:19:48 +0100 | [diff] [blame] | 548 | pending->failed_attempts, VTY_NEWLINE); |
Holger Hans Peter Freyther | 81c0e25 | 2010-12-25 14:08:00 +0100 | [diff] [blame] | 549 | return 0; |
| 550 | } |
Holger Hans Peter Freyther | 3c6f6c2 | 2010-12-25 14:25:12 +0100 | [diff] [blame] | 551 | |
| 552 | int sms_queue_set_max_pending(struct gsm_sms_queue *smsq, int max_pending) |
| 553 | { |
Holger Hans Peter Freyther | eff40949 | 2012-11-10 19:46:58 +0100 | [diff] [blame] | 554 | LOGP(DLSMS, LOGL_NOTICE, "SMSqueue old max: %d new: %d\n", |
Holger Hans Peter Freyther | 3c6f6c2 | 2010-12-25 14:25:12 +0100 | [diff] [blame] | 555 | smsq->max_pending, max_pending); |
| 556 | smsq->max_pending = max_pending; |
| 557 | return 0; |
| 558 | } |
Holger Hans Peter Freyther | 4dcc5e5 | 2010-12-25 14:38:30 +0100 | [diff] [blame] | 559 | |
Holger Hans Peter Freyther | 994dcbb | 2010-12-25 14:50:50 +0100 | [diff] [blame] | 560 | int sms_queue_set_max_failure(struct gsm_sms_queue *smsq, int max_fail) |
| 561 | { |
Holger Hans Peter Freyther | eff40949 | 2012-11-10 19:46:58 +0100 | [diff] [blame] | 562 | LOGP(DLSMS, LOGL_NOTICE, "SMSqueue max failure old: %d new: %d\n", |
Holger Hans Peter Freyther | 994dcbb | 2010-12-25 14:50:50 +0100 | [diff] [blame] | 563 | smsq->max_fail, max_fail); |
| 564 | smsq->max_fail = max_fail; |
| 565 | return 0; |
| 566 | } |
| 567 | |
Holger Hans Peter Freyther | 4dcc5e5 | 2010-12-25 14:38:30 +0100 | [diff] [blame] | 568 | int sms_queue_clear(struct gsm_sms_queue *smsq) |
| 569 | { |
| 570 | struct gsm_sms_pending *pending, *tmp; |
| 571 | |
| 572 | llist_for_each_entry_safe(pending, tmp, &smsq->pending_sms, entry) { |
Holger Hans Peter Freyther | eff40949 | 2012-11-10 19:46:58 +0100 | [diff] [blame] | 573 | LOGP(DLSMS, LOGL_NOTICE, |
Harald Welte | 2483f1b | 2016-06-19 18:06:02 +0200 | [diff] [blame] | 574 | "SMSqueue clearing for sub %llu\n", pending->vsub->id); |
Holger Hans Peter Freyther | 4dcc5e5 | 2010-12-25 14:38:30 +0100 | [diff] [blame] | 575 | sms_pending_free(pending); |
| 576 | } |
| 577 | |
Holger Hans Peter Freyther | 96e9f08 | 2010-12-28 14:09:07 +0100 | [diff] [blame] | 578 | smsq->pending = 0; |
Holger Hans Peter Freyther | 4dcc5e5 | 2010-12-25 14:38:30 +0100 | [diff] [blame] | 579 | return 0; |
| 580 | } |