diff options
| author | Pablo Neira Ayuso <pablo@soleta.eu> | 2014-08-28 14:28:23 +0200 |
|---|---|---|
| committer | Pablo Neira Ayuso <pablo@soleta.eu> | 2014-08-28 14:30:53 +0200 |
| commit | 0b44016ef66788795dab051f4c9408cbc79d4978 (patch) | |
| tree | 9336162410f13c34f8f67360abb0f6256e5ca583 | |
| parent | 3789791b6b5b0bdbcb03e50d521040c813c659c0 (diff) | |
osmux: fix leak in osmux_replay_lost_packets()
The original RTP header has been already sanity check. Therefore,
we can assume the clone is sane.
| -rw-r--r-- | src/osmux.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/src/osmux.c b/src/osmux.c index 61e3fc4..9288464 100644 --- a/src/osmux.c +++ b/src/osmux.c @@ -400,9 +400,8 @@ static void osmux_replay_lost_packets(struct batch_list_node *node, memcpy(clone->data, last->data, last->len); msgb_put(clone, last->len); + /* The original RTP message has been already sanity check. */ rtph = osmo_rtp_get_hdr(clone); - if (rtph == NULL) - return; /* Adjust sequence number and timestamp */ rtph->sequence = htons(ntohs(rtph->sequence) + i); |