From a89da7fded202a36bdaaa5e2008d4ec5246a6500 Mon Sep 17 00:00:00 2001 From: Mke Brady Date: Tue, 6 Oct 2015 13:39:05 +0100 Subject: [PATCH] Fix a bug whereby it would not start to play the stream -- visible when used as system sound. Improperly initialised first_packet_timestamp --- configure.ac | 2 +- player.c | 1 + rtsp.c | 2 +- 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index b4febda9..f6f0f70f 100644 --- a/configure.ac +++ b/configure.ac @@ -2,7 +2,7 @@ # Process this file with autoconf to produce a configure script. AC_PREREQ([2.50]) -AC_INIT([shairport-sync], [2.5.0.7], [mikebrady@eircom.net]) +AC_INIT([shairport-sync], [2.5.0.8], [mikebrady@eircom.net]) AM_INIT_AUTOMAKE AC_CONFIG_SRCDIR([shairport.c]) AC_CONFIG_HEADERS([config.h]) diff --git a/player.c b/player.c index b98776b4..557eb52a 100644 --- a/player.c +++ b/player.c @@ -911,6 +911,7 @@ static void *player_thread_func(void *arg) { silence = malloc(OUTFRAME_BYTES(frame_size)); memset(silence, 0, OUTFRAME_BYTES(frame_size)); late_packet_message_sent = 0; + first_packet_timestamp = 0; missing_packets = late_packets = too_late_packets = resend_requests = 0; flush_rtp_timestamp = 0; // it seems this number has a special significance -- it seems to be used // as a null operand, so we'll use it like that too diff --git a/rtsp.c b/rtsp.c index e2eb53e9..c03bb72f 100644 --- a/rtsp.c +++ b/rtsp.c @@ -722,7 +722,7 @@ static void handle_setup(rtsp_conn_info *conn, rtsp_message *req, rtsp_message * if (config.latency==-1) { // this means that no static latency was set, so we'll allow it to be set dynamically - config.latency=88200; // to be sure, to be sure + config.latency=88198; // to be sure, to be sure -- make it slighty different from the default to ensure we get a debug message when set to 88200 config.use_negotiated_latencies = 1; } char *hdr = msg_get_header(req, "Transport");