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

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

Update RELEASENOTES.md

Adjust the debug level of buffers exhausted and network condition messages.

Oops -- a little typo.

Update RELEASENOTES.md

Update RELEASENOTES.md

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

Oops.

Add the 'dapo' metadata message, giving the DACP port number when found

Generalise volumeup to any command.

Update RELEASENOTES.md

Make the volumeup command work with IPv6 -- use the scope_id

Hook up "volumeup" remote control command -- doesn't work on ipv6, it seems

Checnk the order of the commands a little

Use the DESTDIR variable to make packaging easier

Bump to indicate cleanup of configuration files and fixup of installation for dbus stuff.

Don't need it any more.

Add in code to install the xml file in the local interfaces directory

Stop using the DBUS_SERVICES_DIR stuff

Remove the need for the AS_AC_EXPAND macro and remove the m4 subdirectory containing it

Tighten up the ownership policy

Tidy up configure.ac and drop old pulseaudio back end

A little experimentation with dbus

Some spring cleaning using clang-format

Add avahi code to look for the rmeote control information for the client and get the port number from it.

Add avahi code to look for the rmeote control information for the client and get the port number from it.

Update README.md

Update README.md

Update RELEASENOTES.md

Merge pull request #594 from nbgl/dev-linear-volume

Fix volume bug.

Many thanks for this. I think that in the past the program used to terminate in this case and I changed it to simply continuing and, inadvertently, to failing silently!

Minor cleanup

Attempt to fix hardware volume bug on devices without a linear scale

3.2d8

Try to regenerate the PID directory if it doesn't exist

Try to regenerate the PID directory if it doesn't exist

Merge branch 'master' into piddircreate

Add documentation updates.

Update README.md

Update README.md

Update README.md

Update README.md

Make PID directory path creation recursive and tidy up some code formatting.

Create the directory for the PID file, if necessary

Create the directory for the PID file, if necessary

3.1.2

Remove the "rc0" tag, duh.

Update README.md

Merge pull request #587 from p3ck/development

Development

Update spec file for new upstream release

Merge branch 'upstream-development' into development

Update RELEASENOTES.md

Add a hint about where to find glib libraries

Add a few "get" examples.

tidying up a few little thing

Add very very experimental and exploratory systembus dbus support -- subject to change or removal

Add very very experimental and exploratory systembus dbus support -- subject to change or removal

3.1.2 becomes 3.2d6

3.1.2 becomes 3.2d6

3.1.2

Update RELEASENOTES.md

Update RELEASENOTES.md

3.1.2 pretty-print using clang-format

3.1.2rc0

Move 3.2d5 to 3.1.2

Move 3.2d5 to 3.1.2

Merge branch 'master' into development

Merge any updates made to master