diff options
author | Sylvain Munaut <tnt@246tNt.com> | 2013-05-18 11:34:13 +0200 |
---|---|---|
committer | Sylvain Munaut <tnt@246tNt.com> | 2013-05-23 08:22:48 +0200 |
commit | 25293204200f4dadcd5ffcbe0f884b06829a49c7 (patch) | |
tree | 8f561bf925ccf00b5b4ba8ffde6ad46746ebc9bb /src/host/layer23/src/mobile/gsm411_sms.c | |
parent | fde82f554b4d3d1534b13b6d42f528eb933fd2f5 (diff) |
host/mobile: Fix trans_assign_trans_id() users error checking
The returned value of trans_assign_trans_id() must be stored into a
'int' rather than uint8_t to allow proper testing of negative return
values (errors).
Thanks to Bhaskar <niceguy108@gmail.com> for the original
patches.
Signed-off-by: Sylvain Munaut <tnt@246tNt.com>
Diffstat (limited to 'src/host/layer23/src/mobile/gsm411_sms.c')
-rw-r--r-- | src/host/layer23/src/mobile/gsm411_sms.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/host/layer23/src/mobile/gsm411_sms.c b/src/host/layer23/src/mobile/gsm411_sms.c index 9de1259e..655fe535 100644 --- a/src/host/layer23/src/mobile/gsm411_sms.c +++ b/src/host/layer23/src/mobile/gsm411_sms.c @@ -625,7 +625,7 @@ static int gsm411_tx_sms_submit(struct osmocom_ms *ms, const char *sms_sca, uint8_t *data, *rp_ud_len; uint8_t msg_ref = 42; int rc; - uint8_t transaction_id; + int transaction_id; uint8_t sca[11]; /* max len per 03.40 */ LOGP(DLSMS, LOGL_INFO, "..._sms_submit()\n"); |