diff options
author | Oliver Smith <osmith@sysmocom.de> | 2019-04-08 14:35:30 +0200 |
---|---|---|
committer | osmith <osmith@sysmocom.de> | 2019-04-11 08:29:20 +0000 |
commit | 7d53ae1db8d96ba609eddea1dee8d8779607a19f (patch) | |
tree | 38aac0dd52137a5147c78b8d989d74fa0dbfb30e | |
parent | 95abc2be17f06b28225e19c198340d14d7848c47 (diff) |
USSD: don't use gsm0480_msgb_alloc_name()
We have nothing to do with GSM 04.80 at the HLR - it's only used to
encapsulate the SS payload between MS and MSC. This is not that
critical, but may be misleading.
Also, gsm0480_msgb_alloc_name() allocates a smaller buffer:
return msgb_alloc_headroom(1024, 128, name);
than osmo_gsup_client_msgb_alloc() does:
return msgb_alloc_headroom(4000, 64, __func__);
Change-Id: Icdab40c6a933888eb9f51bee9c5264c8919dbf7b
-rw-r--r-- | src/hlr_ussd.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/hlr_ussd.c b/src/hlr_ussd.c index 56a5a95..50a0167 100644 --- a/src/hlr_ussd.c +++ b/src/hlr_ussd.c @@ -277,7 +277,7 @@ static int ss_tx_to_ms(struct ss_session *ss, enum osmo_gsup_message_type gsup_m resp.ss_info_len = msgb_length(ss_msg); } - resp_msg = gsm0480_msgb_alloc_name(__func__); + resp_msg = msgb_alloc_headroom(4000, 64, __func__); OSMO_ASSERT(resp_msg); osmo_gsup_encode(resp_msg, &resp); msgb_free(ss_msg); |