Update TROUBLESHOOTING.md

Update TROUBLESHOOTING.md

Update RELEASENOTES.md

Update README.md

Fix typo — thanks to [systemcrash](https://github.com/systemcrash).

Update configure.ac

Update shairport-sync.spec

Update shairport-sync.spec

Update RELEASENOTES.md

Update shairport-sync.spec

Fix segmentation fault bug when a client's DACP status is withdrawn. Not enough testing...

Fix segmentation fault bug when a client's DACP status is withdrawn. Not enough testing...

Update shairport-sync.spec

Tiny typo fixes in the release notes.

3.14 to 3.1.5 Fix a bug resumption in iOS 11.2 on software-volume controlled Shairport Syncs to be at 100% volume. Improve sync with movies. Other bug fixes.

3.14 to 3.1.5 Fix a bug resumption in iOS 11.2 on software-volume controlled Shairport Syncs to be at 100% volume. Improve sync with movies. Other bug fixes.

Update RELEASENOTES.md

Update RELEASENOTES.md

More comprhensive saving and restoring of hardware and software level when a session is stopped and started. Does hardware as well as software.

More comprhensive saving and restoring of hardware and software level when a session is stopped and started. Does hardware as well as software.

More comprhensive saving and restoring of hardware and software level when a session is stopped and started. Does hardware as well as software.

Improve the sync between AirPlay and movies like YouTube. Not sure we understand the negotiation for syncing.

Improve the sync between AirPlay and movies like YouTube. Not sure we understand the negotiation for syncing.

Update RELEASENOTES.md

Store the last software volume setting so that it can become the default when the next play sessions starts. This is important if a starting volume is not always given, a new phenomenon in iOS 11.2 and macOS 10.13.2

Store the last software volume setting so that it can become the default when the next play sessions starts. This is important if a starting volume is not always given, a new phenomenon in iOS 11.2 and macOS 10.13.2

Update CONTRIBUTING.md

Update CONTRIBUTING.md

Update CONTRIBUTING.md

Update CONTRIBUTING.md

Update CONTRIBUTING.md

Update CONTRIBUTING.md

Update CONTRIBUTING.md

Update CONTRIBUTING.md

Update RELEASENOTES.md

Make some additions to audio_pa to emit error messages before it dies

Make some additions to audio_pa to emit erro messages before it dies

Update UPDATING.md

Fix a small conflict in mdns_avahi.c

Fiel small metadat bug, make more intermediate progress with d-bus stuff

Fix bug in ooutput of 'flsr' metadata item -- remove the leading equals sign.

Started implementing a scanner with just playstatusupdate message

hooking up a dacp scanner

Intermediate

Fix compilation bug if dbus or mpris not selected

Begin a DACP monitor. Very incomplete -- work in progress only

Quieten a warning from clang on FreeBSD

Don't need to make Shairport Sync an activatable D-Bus service, so don't need this file

Remember to install a dbus system bus access rule for mpris, separate dbus and mpris access rules, move them to scripts folder

Update RELEASENOTES.md

Update RELEASENOTES.md

Update RELEASENOTES.md

Update RELEASENOTES.md

Update configure.ac

Update configure.ac

Update RELEASENOTES.md

Update RELEASENOTES.md

Merge pull request #620 from bootc/cve-2017-12087

Fix CVE-2017-12087 in bundled tinysvcmdns (development branch)

Merge pull request #621 from bootc/cve-2017-12087-master

Fix CVE-2017-12087 in bundled tinysvcmdns (master branch)

tinysvcmdns: fix CVE-2017-12087

This patch incorporates upstream's fixes for a remotely exploitable
buffer overflow bug in the bundled tinysvcmdns library. The following
upstream commits are included:

https://bitbucket.org/geekman/tinysvcmdns/commits/48c73fbb36b7a5584b00538d89c89dd8b15ab2a7
https://bitbucket.org/geekman/tinysvcmdns/commits/29ea1b9fca94dc42d16109e050d2967231b1e341

The changes have been incorporated preserving local changes such as the
check for malloc() returning NULL.

Reported against shairport-sync in Ubuntu Launchpad:
https://bugs.launchpad.net/ubuntu/+source/shairport-sync/+bug/1729668

This commit closes GitHub issue #619.

tinysvcmdns: fix CVE-2017-12087

This patch incorporates upstream's fixes for a remotely exploitable
buffer overflow bug in the bundled tinysvcmdns library. The following
upstream commits are included:

https://bitbucket.org/geekman/tinysvcmdns/commits/48c73fbb36b7a5584b00538d89c89dd8b15ab2a7
https://bitbucket.org/geekman/tinysvcmdns/commits/29ea1b9fca94dc42d16109e050d2967231b1e341

The changes have been incorporated preserving local changes such as the
check for malloc() returning NULL.

Reported against shairport-sync in Ubuntu Launchpad:
https://bugs.launchpad.net/ubuntu/+source/shairport-sync/+bug/1729668

This commit closes GitHub issue #619.

Update RELEASENOTES.md

Update RELEASENOTES.md

Merge pull request #618 from joerg-krause/fix-softvol

Fix usage of ALSA softvol plugin.

Thanks Jörg Krause. If you want to apply it to the `development` branch, I'll pull it in there too; otherwise I'll make the change myself.

Fix usage of ALSA softvol plugin

Somewhere in the version 3.x the softvol plugin got broken as the volume
change is not applied anymore. Turned out for the softvol plugin no
`volume()` and `parameters()` are defined.

Add previous and next controls

Send playing status as soon as the first packet is received

Update RELEASENOTES.md

Clang-format

Handle play / pause / play-pause / stop but primitively

Clean up naming of some files, get the 'stop' command working

Reorganise D-BUS interface code and introduce an MPRIS interface.

Reorganise D-BUS interface code and introduce an MPRIS interface.

It turns out you don't need to add an interface file or a service file for a dbus app that doesn't need to be autostarted, or something.

Forgot documentation updates, again!
Merge branch 'development' of github.com:mikebrady/shairport-sync into development

Fix a variable substitution. Might be problematic for BSD

Update RELEASENOTES.md

dbus-related files taken from directories and renamed

Change the names of some of the files.

Merge branch 'development' of github.com:mikebrady/shairport-sync into development

Forgot the typo fix.

Improve dbus and add some dacp services

clang-format only

Merge pull request #614 from corbinsantin/development

Fix typo in README.md

Update README.md

small typo

Clean up rtp_send_client_command a little

Update RELEASENOTES.md

Merge pull request #612 from belboj/signal-race

Fixed race condition when stopping other threads with SIGUSR1

Fixed race condition when stopping other threads with SIGUSR1

Many threads have a main loop that checks some exit variable in the loop condition. To stop a thread, another thread sets that exit variable, sends a SIGUSR1 to that thread to get it out of blocking calls like read(), write() or select(), and finally joins it.

There was a race condition though: if the signal arrived after the loop condition was checked but before the blocking system function was entered, the target thread essentially ignored the signal, made the blocking call anyway, and could get stuck forever there.

The only way around this race condition is to block SIGUSR1 in the destination thread during the critical time interval. This requires that the blocking system call and the call to unblock SIGUSR1 be an atomic unit. There is only one system call that makes this possible: pselect(), which takes a signal mask to replace the current signal mask with, but only while pselect() runs. Other functions like read(), recv(), write() and sendto() may only be called if we can be sure that they don't block. For this purpose, the file descriptors must be set to non-blocking mode and we must prepend the call to them with a call to pselect() to block until the socket becomes readable/writable.

Update RELEASENOTES.md

Merge pull request #611 from belboj/some-fixes

Some issues found during the development of a DLNA backend.

Many thanks for these.

Fixed loop condition in non_blocking_write()

The while loop should be repeated, if the last write attempt was a partial write only. In this case, however, rc is greater than 0 (should be 1), not equal to 0.

Fixed EINTR handling in rtsp_read_request()

EINTR is an error code written to errno, not returned by read().

Fixed dangling pointer to random state array

The C library function initstate(), at least in macOS El Capitan, not only sets the state array given to it as the second argument, but also accesses the state array given to it in a previous call! (I don't know why, or if that behaviour is correct, but macOS seems to do that anyhow).
Therefore, we cannot pass a local variable to it. As soon as the player thread stops, it will be a dangling pointer, and when a new player thread is started afterwards, initstate() will dereference that dangling pointer with unforeseeable consequences.

handle_announce() waits for playing_conn to stop if it's already shutting down

- If playing_conn is already stopping, handle_announce() waits one second for it to stop. This gives the current RTSP connection a better chance to get the player, if it has been started very quickly after the previous RTSP connection had decided to stop. iTunes can sometimes be very quick in starting a new RTSP connection after tearing down the previous one, but shairport_sync unfortunately needs at least 100ms to shut down the old connection.
- cleanup_threads() sets playing_conn to NULL if it points to the removed thread, otherwise playing_conn would be a dangling pointer, which is bad, because handle_announce() dereferences it unless it is NULL.

Prevent buffer overflow in log functions

die(), warn(), debug() and inform() write the string into a limited internal buffer first. They now use vsnprintf() instead of vsprintf() to prevent an overflow of that buffer.

Update RELEASENOTES.md

Don't disable PulseAudio autospawn

Add an exit diagnostic, clean up a dbus flag

Address a warning about a buffer being too short.

Correct a few warnings that show upo in Clang but not in GCC

Correct a Makefile bug that was silent in Linux but caused problems in FreeBSD