Commit 53466e2

author

Alessandro Toppi

committed

Reset NACK queue only when receiving a KeyFrame with a highest sequence number.

1 parent 47c8ba7 commit 53466e2Copy full SHA for 53466e2

File tree

-3

lines changed

-3

lines changed

Lines changed: 4 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -2409,15 +2409,16 @@ static void janus_ice_cb_nice_recv(NiceAgent *agent, guint stream_id, guint comp`
`2409`	`2409`	`/* ... unless NACKs are disabled for this medium */`
`2410`	`2410`	`return;`
`2411`	`2411`	`}`
	`2412`	`+ guint16 new_seqn = ntohs(header->seq_number);`
`2412`	`2413`	`/* If this is video, check if this is a keyframe: if so, we empty our NACK queue */`
`2413`	`2414`	`if(video && stream->video_is_keyframe) {`
`2414`	`2415`	`if(stream->video_is_keyframe(payload, plen)) {`
`2415`		`- JANUS_LOG(LOG_HUGE, "[%"SCNu64"] Keyframe received, resetting NACK queue\n", handle->handle_id);`
`2416`		`- if(component->last_seqs_video[vindex])`
	`2416`	`+ if(component->last_seqs_video[vindex] && (int16_t)(new_seqn-rtcp_ctx->max_seq_nr) >0) {`
	`2417`	`+ JANUS_LOG(LOG_HUGE, "[%"SCNu64"] Keyframe received with a highest sequence number, resetting NACK queue\n", handle->handle_id);`
`2417`	`2418`	`janus_seq_list_free(&component->last_seqs_video[vindex]);`
	`2419`	`+ }`
`2418`	`2420`	`}`
`2419`	`2421`	`}`
`2420`		`- guint16 new_seqn = ntohs(header->seq_number);`
`2421`	`2422`	`guint16 cur_seqn;`
`2422`	`2423`	`int last_seqs_len = 0;`
`2423`	`2424`	`janus_mutex_lock(&component->mutex);`

Comments

(0)