From cf56dc37075b37297e7e278a36307e2117bddc11 Mon Sep 17 00:00:00 2001 From: Jief Date: Wed, 2 Nov 2016 14:58:02 +0100 Subject: [PATCH 1/2] Using Macro instead of LL to make it portable. --- rtp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rtp.c b/rtp.c index 8c34bb9e..576e7ba0 100644 --- a/rtp.c +++ b/rtp.c @@ -501,7 +501,7 @@ void *rtp_timing_receiver(void *arg) { if (reference_timestamp>=play_segment_reference_frame) frame_difference = (uint64_t)reference_timestamp-(uint64_t)play_segment_reference_frame; else // rollover - frame_difference = (uint64_t)reference_timestamp+0x100000000LL-(uint64_t)play_segment_reference_frame; + frame_difference = (uint64_t)reference_timestamp+UINT64_C(0x100000000)-(uint64_t)play_segment_reference_frame; uint64_t frame_time_difference_calculated = (((uint64_t)frame_difference<<32)/44100); uint64_t frame_time_difference_actual = remote_reference_timestamp_time-play_segment_reference_frame_remote_time; // this is all done by reference to the sources' system clock // debug(1,"%llu frames since play started, %llu usec calculated, %llu usec actual",frame_difference, (frame_time_difference_calculated*1000000)>>32, (frame_time_difference_actual*1000000)>>32); From 21025440afeadeb8cee56e6bf47988ab13590370 Mon Sep 17 00:00:00 2001 From: Jief Date: Fri, 11 Nov 2016 19:16:47 +0100 Subject: [PATCH 2/2] clock_drift could be left uninitialized. clock_drift redeclared instead of initialized with "uint64_t clock_drift = 0". I think this is the correction. --- rtp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rtp.c b/rtp.c index 576e7ba0..8d5b46df 100644 --- a/rtp.c +++ b/rtp.c @@ -474,7 +474,7 @@ void *rtp_timing_receiver(void *arg) { if (first_local_time==0) { first_local_time = local_time_chosen; first_remote_time = remote_time_chosen; - uint64_t clock_drift = 0; + clock_drift = 0; } else { uint64_t local_time_change = local_time_chosen - first_local_time; uint64_t remote_time_change = remote_time_chosen - first_remote_time;