module-rtp-sap: only override NTP value in the timing line

That is indeed 0 for nearly any device. However the NTP value in the session identification part plays a crucial role for distinguishing between streams in some implementations, e.g. Dante.
This commit is contained in:
Dmitry Sharshakov 2023-12-18 19:11:01 +03:00 committed by Wim Taymans
parent 420ab8ed89
commit 2e14f3d97a

View file

@ -167,6 +167,7 @@ static const struct spa_dict_item module_info[] = {
struct sdp_info { struct sdp_info {
uint16_t hash; uint16_t hash;
uint32_t ntp; uint32_t ntp;
uint32_t t_ntp;
char *origin; char *origin;
char *session_name; char *session_name;
@ -551,7 +552,7 @@ static int send_sap(struct impl *impl, struct session *sess, bool bye)
user_name, sdp->ntp, src_ip4 ? "IP4" : "IP6", src_addr, user_name, sdp->ntp, src_ip4 ? "IP4" : "IP6", src_addr,
sdp->session_name, sdp->session_name,
dst_ip4 ? "IP4" : "IP6", dst_addr, dst_ttl, dst_ip4 ? "IP4" : "IP6", dst_addr, dst_ttl,
sdp->ntp, sdp->t_ntp,
sdp->media_type, sdp->dst_port, sdp->payload); sdp->media_type, sdp->dst_port, sdp->payload);
if (impl->extra_attrs_preamble) if (impl->extra_attrs_preamble)
@ -695,7 +696,8 @@ static struct session *session_new_announce(struct impl *impl, struct node *node
sess->announce = true; sess->announce = true;
sdp->hash = pw_rand32(); sdp->hash = pw_rand32();
sdp->ntp = pw_properties_get_uint32(props, "rtp.ntp", (uint32_t) time(NULL) + 2208988800U + impl->n_sessions); sdp->ntp = (uint32_t) time(NULL) + 2208988800U + impl->n_sessions;
sdp->t_ntp = pw_properties_get_uint32(props, "rtp.ntp", sdp->ntp);
sess->props = props; sess->props = props;
if ((str = pw_properties_get(props, "sess.name")) == NULL) if ((str = pw_properties_get(props, "sess.name")) == NULL)