Skip to content

Commit

Permalink
rtp/sess: fix missing srate_tx locking
Browse files Browse the repository at this point in the history
Found by coverity:

CID 443862:  Concurrent data access violations  (MISSING_LOCK)
Accessing "sess->srate_tx" without holding lock "rtcp_sess.lock".
Elsewhere, "rtcp_sess.srate_tx" is written to with "rtcp_sess.lock" held 2 out
of 2 times (2 of these accesses strongly imply that it is necessary).
  • Loading branch information
sreimers committed Dec 11, 2024
1 parent 3d6db77 commit e4ff503
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/rtp/sess.c
Original file line number Diff line number Diff line change
Expand Up @@ -426,10 +426,12 @@ static int encode_handler(struct mbuf *mb, void *arg)
static int mk_sr(struct rtcp_sess *sess, struct mbuf *mb)
{
struct txstat txstat;
uint32_t srate_tx;
int err;

mtx_lock(sess->lock);
txstat = sess->txstat;
srate_tx = sess->srate_tx;
sess->txstat.ts_synced = false;
mtx_unlock(sess->lock);

Expand All @@ -442,7 +444,7 @@ static int mk_sr(struct rtcp_sess *sess, struct mbuf *mb)

dur = jfs_rt - txstat.jfs_rt_ref;
rtp_ts = (uint32_t)((uint64_t)txstat.ts_ref + dur *
sess->srate_tx / 1000000u);
srate_tx / 1000000u);

err = rtcp_encode(mb, RTCP_SR, sess->senderc,
rtp_sess_ssrc(sess->rs), ntp.hi, ntp.lo,
Expand Down

0 comments on commit e4ff503

Please sign in to comment.