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/gsm480_ss.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/gsm480_ss.c')
-rw-r--r-- | src/host/layer23/src/mobile/gsm480_ss.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/host/layer23/src/mobile/gsm480_ss.c b/src/host/layer23/src/mobile/gsm480_ss.c index fda62881..ab8e164c 100644 --- a/src/host/layer23/src/mobile/gsm480_ss.c +++ b/src/host/layer23/src/mobile/gsm480_ss.c @@ -564,7 +564,7 @@ static int gsm480_tx_ussd(struct gsm_trans *trans, uint8_t msg_type, int ss_send(struct osmocom_ms *ms, const char *code, int new_trans) { struct gsm_trans *trans = NULL, *transt; - uint8_t transaction_id; + int transaction_id; /* look for an old transaction */ if (!new_trans) { |