diff options
author | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2017-11-16 17:11:28 +0100 |
---|---|---|
committer | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2017-11-27 17:54:23 +0800 |
commit | 89009751ea7ceda542c254633f4ceef23b3f8f90 (patch) | |
tree | 04376e26f8072dbe1ade8d6637d874538ce2fb45 /src/host/layer23/include | |
parent | 14598ac88d816168e037d701b6f43a2a4cf02f42 (diff) |
mobile: Avoid msg_ref going out of sync
It seemed like msg_ref could go out of sync. In some places we are
using sms->msg_ref in other cases we pass it as parameter (e.g. when
sending the SMS) or we get it out of the gsm411_rp_hdr.
Instead of hardcoding 42 for all messages make it configurable and
pass the parameter from the caller.
Change-Id: I4bac5f06921b5fd85a98d97770d42d4858ca1c42
Diffstat (limited to 'src/host/layer23/include')
-rw-r--r-- | src/host/layer23/include/osmocom/bb/mobile/gsm411_sms.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/host/layer23/include/osmocom/bb/mobile/gsm411_sms.h b/src/host/layer23/include/osmocom/bb/mobile/gsm411_sms.h index d14e6db8..0d0578a3 100644 --- a/src/host/layer23/include/osmocom/bb/mobile/gsm411_sms.h +++ b/src/host/layer23/include/osmocom/bb/mobile/gsm411_sms.h @@ -11,7 +11,6 @@ struct gsm_sms { uint8_t ud_hdr_ind; uint8_t protocol_id; uint8_t data_coding_scheme; - uint8_t msg_ref; char address[20+1]; /* DA LV is 12 bytes max, i.e. 10 bytes * BCD == 20 bytes string */ time_t time; @@ -28,6 +27,6 @@ void sms_free(struct gsm_sms *sms); struct gsm_sms *sms_from_text(const char *receiver, int dcs, const char *text); int gsm411_rcv_sms(struct osmocom_ms *ms, struct msgb *msg); int sms_send(struct osmocom_ms *ms, const char *sms_sca, const char *number, - const char *text); + const char *text, uint8_t msg_ref); #endif /* _GSM411_SMS_H */ |