nils/shairport-sync
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Check timer values a bit more carefully and ignore crazy values.

Browse Source
This commit is contained in:
Mike Brady
2018-11-14 12:25:56 +00:00
parent 67083665ad
commit 29a37f44b9
1 changed files with 10 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
rtp.c
+10 -5
View File
@@ -621,9 +621,14 @@ void *rtp_timing_receiver(void *arg) {
distant_transmit_time = (uint64_t)nctohl(&packet[24]) << 32;
distant_transmit_time += nctohl(&packet[28]);
uint64_t remote_processing_time = distant_transmit_time - distant_receive_time;
uint64_t remote_processing_time = 0;
if (distant_transmit_time >= distant_receive_time)
remote_processing_time = distant_transmit_time - distant_receive_time;
else {
debug(1, "Yikes: distant_transmit_time is before distant_receive_time; remote processing time set to zero.");
}
// debug(1,"Return trip time: %" PRIu64 " uS, remote processing time: %" PRIu64 "
// uS.",(return_time*1000000)>>32,(remote_processing_time*1000000)>>32);
@@ -632,10 +637,10 @@ void *rtp_timing_receiver(void *arg) {
// remove the remote processing time from the record of the return time, as long at the
// processing time looks sensible.
if ((remote_processing_time > 0) && (remote_processing_time < return_time))
if (remote_processing_time < return_time)
return_time -= remote_processing_time;
else
debug(1, "Non-sensical remote processing time -- ignored.");
debug(1, "Remote processing time greater than return time -- ignored.");
int cc;
for (cc = time_ping_history - 1; cc > 0; cc--) {