diff options
| author | Jacob Erlbeck <jerlbeck@sysmocom.de> | 2015-01-05 18:57:32 +0100 |
|---|---|---|
| committer | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2015-01-18 18:22:47 +0100 |
| commit | bce2061b43c1e38b766cc3738b4ad669211dc6a4 (patch) | |
| tree | 0df33aa2f88e57fd4a693f531180e31a909c780f /openbsc/src/gprs/gprs_gsup_messages.c | |
| parent | 9aa9991efe207f5c12bbcf655b0660e3466c02e0 (diff) | |
gprs: Let GSUP parser functions return GMM causes on errors
Currently the GSUP message handling function in gprs_subscriber.c and
the functions in gprs_gsup_messages.c are not consistent with respect
to the return codes if an error happens. Albeit all error return
codes are negative, the semantics of the absolute value are not
clearly defined. In addition, some return codes are not passed to the
calling function.
This path changes these functions to always return a negated GMM
cause value in case of errors. Return values of called parser
functions are not longer ignored.
Sponsored-by: On-Waves ehf
Diffstat (limited to 'openbsc/src/gprs/gprs_gsup_messages.c')
| -rw-r--r-- | openbsc/src/gprs/gprs_gsup_messages.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/openbsc/src/gprs/gprs_gsup_messages.c b/openbsc/src/gprs/gprs_gsup_messages.c index f47ad5545..1dad74ffe 100644 --- a/openbsc/src/gprs/gprs_gsup_messages.c +++ b/openbsc/src/gprs/gprs_gsup_messages.c @@ -74,7 +74,7 @@ static int decode_pdp_info(uint8_t *data, size_t data_len, rc = gprs_shift_tlv(&data, &data_len, &tag, &value, &value_len); if (rc < 0) - return rc; + return -GMM_CAUSE_PROTO_ERR_UNSPEC; iei = tag; @@ -116,7 +116,7 @@ static int decode_auth_info(uint8_t *data, size_t data_len, while (data_len > 0) { rc = gprs_shift_tlv(&data, &data_len, &tag, &value, &value_len); if (rc < 0) - return rc; + return -GMM_CAUSE_PROTO_ERR_UNSPEC; iei = tag; |