diff options
author | Harald Welte <laforge@gnumonks.org> | 2017-12-03 10:29:13 +0100 |
---|---|---|
committer | Harald Welte <laforge@gnumonks.org> | 2017-12-03 21:22:25 +0000 |
commit | be4a2088098a7b4fbfc1a5046e4c2ceb21ce1b1d (patch) | |
tree | 268b7216fdf2fd212f76f2a6b0a992a41208507d /src/common/rsl.c | |
parent | 5112c45d224f99d0bc65e52f6cb5269feb7315dc (diff) |
Put useful information in RTCP SDES.
The RTCP Sender Description is supposed to contain useful information
to identify the source of the RTP stream. So far, it only contained
compile-time default data of libortp. Let's put the BTS UnitID, the
lchan number and the OsmoBTS version in there.
This change requires libosmo-abis Change-Id Ice794f9e0c6caeea1c67520c12efbfa375d1fb82
Change-Id: Id6ce7188354d3a0517661c9648854ec829ef1cac
Related: OS#2701
Diffstat (limited to 'src/common/rsl.c')
-rw-r--r-- | src/common/rsl.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/src/common/rsl.c b/src/common/rsl.c index 461b21d1..60f964e5 100644 --- a/src/common/rsl.c +++ b/src/common/rsl.c @@ -20,6 +20,8 @@ * */ +#include "btsconfig.h" /* for PACKAGE_VERSION */ + #include <stdio.h> #include <errno.h> #include <netdb.h> @@ -1668,6 +1670,7 @@ static int rsl_rx_ipac_XXcx(struct msgb *msg) } if (dch->c.msg_type == RSL_MT_IPAC_CRCX) { + char cname[32]; char *ipstr = NULL; if (lchan->abis_ip.rtp_socket) { LOGP(DRSL, LOGL_ERROR, "%s Rx RSL IPAC CRCX, " @@ -1736,6 +1739,12 @@ static int rsl_rx_ipac_XXcx(struct msgb *msg) return tx_ipac_XXcx_nack(lchan, RSL_ERR_RES_UNAVAIL, inc_ip_port, dch->c.msg_type); } + /* Ensure RTCP SDES contains some useful information */ + snprintf(cname, sizeof(cname), "bts@%s", ipstr); + osmo_rtp_set_source_desc(lchan->abis_ip.rtp_socket, cname, + gsm_lchan_name(lchan), NULL, NULL, + gsm_trx_unit_id(lchan->ts->trx), + "OsmoBTS-" PACKAGE_VERSION, NULL); /* FIXME: multiplex connection, BSC proxy */ } else { /* MDCX */ |