Update AIRPLAY2.md - Version introduced + Supported devices

The introduction paragraph now more clearly mentions the hard facts/requirements with "as of v4.1 and newer" and the supported devices are displayed in bullet list format for better readabilty.

Update RELEASENOTES-DEVELOPMENT.md

Build scripts/shairport-sync.service, scripts/shairport-sync.service-avahi and scripts/shairport-sync unconditionally for Linux to facilitate different installation methods.

Update RELEASENOTES-DEVELOPMENT.md

Merge pull request #1811 from klemensn/keep-password-private

Omit password in debug (syslog) output

Merge pull request #1810 from klemensn/printf-null-string

Avoid passing NULL pointer to printf(3) %s

Merge pull request #1803 from klemensn/initial-pledge

initial pledge(2) on OpenBSD

Omit password in debug (syslog) output

Configuration files may contain it, but those can be protected with
filesystem permissions.

Debug log, however, may end up in syslog(3) which is usually accessible
by all users on a system.

Avoid disclosing user secrets by simply not printing the password.

Avoid passing NULL pointer to printf(3) %s

OpenBSD issues syslog(3) warnings about these as they may hint at bugs:
shairport-sync: vfprintf %s NULL in "metadata socket address is "%s" port %d."

Metadata listener socket is empty by default so use the existing `strnull`.

initial pledge(2) on OpenBSD

Limit the set of system calls shairport-sync is allowed to make,
most importantly fork(2)/execve(2) if and only if user defined commands
are run.

OpenBSD's official audio/shairport-sync port already ships with this
patch.

https://man.openbsd.org/pledge.2

Merge pull request #1800 from klemensn/rtsp-listener-exit-segfault

Fix crash on double pthread_cancel(3) on exit (detected on OpenBSD).

Fix crash on double pthread_cancel(3) on exit

On OpenBSD 7.4-current, failure to listen on the RTSP socket(s) results
the `rtsp_listener_thread` being pthread_cancel(3)'ed twice, once through
`rtsp_listen_loop()` and again via atexit(3) handler `exit_rtsp_listener()`:

```
$ nc -4l 5000 &
$ nc -6l 5000 &
$ shairport-sync -c/dev/null
warning: could not establish a service on port 5000 -- program terminating. Is another instance of Shairport Sync running on this device?
Segmentation fault (core dumped)
```
```
Program terminated with signal SIGSEGV, Segmentation fault.
433             if (tib->tib_canceled == 0 && tid != 0 &&
[Current thread is 1 (process 290061)]
```

`die()` -> `exit(EXIT_FAILURE)` normally in this case, thus forgoing the
first cancel and relying on the atexit handler alone.

With Mike Brady.

Update RELEASENOTES-DEVELOPMENT.md

Merge pull request #1797 from klemensn/openbsd-doc-nits

OpenBSD doc nits

Merge pull request #1798 from klemensn/config-perm-segfault

Exit on config read error to avoid crash on EACCES

Exit on config read error to avoid crash on EACCES

Failure to read does not exit despite
`    /* Read the file. If there is an error, report it and exit. */`

EACCES (e.g. insufficient filesystem permissions) is enough to crash
on access through later `config_*()` such as those when either of
either of D-Bus, MPRIS or MQTT is used.

Seen `--with-mpris-interface` and
```
$ ls -l /etc/shairport-sync.conf
-rw-r-----  1 root  _shairport  28114 Jan 25 01:53 /etc/shairport-sync.conf
$ shairport-sync
Segmentation fault (core dumped)
```

clarify root privilege requirements

NQPTP works and runs as _shairport user on OpenBSD.

Linux capabilities(7) allow starting as unprivileged user in the first
place, otherwise programs only need to start and perform privileged
actions such as bind(2)ing sockets as root, after that they can and
should! drop privileges.

Instead of updating/repeating supported systems, drop listings.

Sprinkle OpenBSD news in README.md

fix [-Wunused-variable] for keep* variables

Update RELEASENOTES-DEVELOPMENT.md

Merge pull request #1795 from klemensn/manual-section

Move shairport-sync manual from section 7 to 1

Merge pull request #1794 from klemensn/per-socket-keepidle

No per-socket TCP keepidle on OpenBSD

Merge pull request #1793 from klemensn/control-socket

Connect to NQPTP control socket on localhost

Merge pull request #1782 from vs49688/master

Makefile.am: allow manually installing config files

Makefile.am: allow manually installing config files

Adds a "install-config-files" Makefile target that installs the config
files and dbus policies to DESTDIR=, regardless of whether
--with{,out}-configfiles is specified.

Move shairport-sync manual from section 7 to 1

From mandoc man(1) [-s] section:
                   1         General commands (tools and utilities).
                   [...]
                   7         Miscellaneous information.

From GNU man(1) DESCRIPTION:
     1   Executable programs or shell commands
     [...]
     7   Miscellaneous (including macro packages and conventions), e.g.
         man(7), groff(7)

shairport-sync is an executable, shairport-sync(1) ought to be its
documentation, not merely miscellaneous information.

OpenBSD adjusts this in its audio/shairport-sync port/package ever since.

No per-socket TCP keepidle on OpenBSD

A system-wide sysctl(2) net.inet.tcp.keepidle exists, but there is no
setsockopt(2) equivalent.

Connect to NQPTP control socket on localhost

nqptp.c listens on "localhost", but shairport-sync connectes to the wildcard
address 0/0.  This apparently works on Linux and FreeBSD, but OpenBSD fails:

```
$ ktrace shairport-sync -v -u
[...]
         0.000038288 "ptp-utilities.c:243" *fatal error: error sending timing_peer_list to NQPTP
         0.000021868 "shairport.c:1728" emergency exit
$ kdump
[...]
  2319 shairport-sync STRU  struct sockaddr { AF_INET, 0.0.0.0:9000 }
  2319 shairport-sync RET   sendto -1 errno 51 Network is unreachable
[...]
```

Resolve and connect to "localhost" just like NQPTP does, resulting in
127.0.0.1 or ::1 as socket addresses.

This is required to run when configured `--with-airplay-2`.
Tested on OpenBSD/amd64 7.4-current with shairport-sync 2.4.3 and nqptp
1.2.5-dev cfa8315 (plus OpenBSD fixes).

I expect Linux and FreeBSD to work as before, but have not tested it.

Quieten some debug messages.

Update RELEASENOTES-DEVELOPMENT.md

Update CAR INSTALL.md

Use pthread_rwlock_wrlock instead of the incorrect use of pthread_rdlock_wrlock when tearing down a connection.

Update RELEASENOTES-DEVELOPMENT.md

A bit late, but tag this as 4.3.3-dev. Also merge in the commit history of the master branch.

merge back in 4.3.2, a bit lae...

Update CAR INSTALL.md

Update CAR INSTALL.md

Update CAR INSTALL.md

Update RELEASENOTES-DEVELOPMENT.md

Update RELEASENOTES-DEVELOPMENT.md

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

Stop some warnings during build for Docker.

Update RELEASENOTES-DEVELOPMENT.md

Update RELEASENOTES-DEVELOPMENT.md

Bookworm and simplifications

 Update for Bookworm and make some simplifications.

Update CAR INSTALL.md

Update for Bookworm and simplify some steps. Add some information about updating.

Update CAR INSTALL.md

Update CAR INSTALL.md

Merge branch 'development'

Update to 4.3.2
**Enhancements**
* A totally new PipeWire backend featuring full synchronisation.

**Bug Fixes**
* Stability improvements for the PulseAudio backend.
* Fix a crash when the Avahi subsystem became disconnected. This is normally a rare occurrence, but Shairport Sync was not dereferencing obsolete data correctly when it happened.
* Set and reset Bonjour flags correctly when it's a Classic Airplay session in AirPlay 2 operation.
* Fix a number of FreeBSD compilation errors and warnings.
* Fix various errors when breaking into an existing session to terminate it. Thanks again to [aaronk6](https://github.com/aaronk6).
* Fix some debug message errors, sigh. Thanks to [Nathan Gray](https://github.com/n8gray).

Update CAR INSTALL.md

Add note about Bookworm

Update CAR INSTALL.md

Add note about Bookworm

Update RELEASENOTES-DEVELOPMENT.md

Change response to a /feedback message in a Classic AirPlay session to 501 Not Implmemented instead of 200 OK.

Update RELEASENOTES-DEVELOPMENT.md

Return code 200 for a POST of type /feedback, and continue to return 500 for everything else.

Typo

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

Online update.

Move the PipeWire stanza up in the configuration file.

Update RELEASENOTES-DEVELOPMENT.md

Add PipeWire sink target setting. Fix PipeWire node name code, duh, remove code that was double-deallocating settings strings.

Remove code that was double-deallocating settings strings.

Update RELEASENOTES-DEVELOPMENT.md

Update RELEASENOTES-DEVELOPMENT.md

Change node_name default to "Shairport Sync", parameterise a few settings.

Update RELEASENOTES-DEVELOPMENT.md

Add two PipeWire backend ("pw") settings -- application_name (default: "Shairport Sync") and node_name (default: "shairport-sync").

Deallocate settings strings on exit.

Update RELEASENOTES-DEVELOPMENT.md

Update RELEASENOTES-DEVELOPMENT.md

Send silence when no audio is coming through from Shairport Sync. This eliminates (?) xrun errors. Clean up a few imprefections.

Quieten a debug message.

Update RELEASENOTES-DEVELOPMENT.md

Complete replacement of the PipeWire backend with a new implementation with full synchrnoisation. Works well, but still preliminary.

Stability improvements -- take more care when the fifi is full, fix a logical error in the use of the buffer mutex.

stop linker errors if built alongside audio_pa

Replace the pipewire backend completely. Play writes to a buffer. The on_process() function reads from the buffer and updates timing information for the delay() function. Barebones -- can't set the app name or the volume. Assumes no further delays and no buffers when on_process is called. But it works!

mutex lock was protecting the wrong variable, duh.

Fix potential bug if the timing data is inconsistent for 10 tries.

Initial tone-generator based pipewire backend compiling and apparently working, valgrind is happy too.

Update RELEASENOTES-DEVELOPMENT.md

Don't mute if the volume control is at minimum (-144.0) and ignore_volume_control is seelcted.

Actually _do_ change the latency to 200000 in the PipeWire backend as previously promied, duh.

Update RELEASENOTES-DEVELOPMENT.md

Very very rarely, the avahi client becomes disconnected.

This usually means there are problems in the system as a whole rahter than with Shairport Sync.

However, Shairport Sync was not cleaning up properly before deleting the now-disconnected avahi client and
creating a new one in an attempt to reconnect. That caused Shairport Sync to crash even if the new avahi client
was created successfully.

So this commit has code to delete the avahi group and broswer callback before deleting the disconnected avahi client.

Update RELEASENOTES-DEVELOPMENT.md

Make the pipewire latency 200000 (uS?) instead of 20000.

Update RELEASENOTES-DEVELOPMENT.md

Add missing arguments to some debug messages. They could potentially cause crashes just when you need them to work properly, sigh!

Update RELEASENOTES-DEVELOPMENT.md

When a connection termiates abruptly while is it the principal_conn, make sure it sets the principal_conn
to NULL and cleans up the bonjour flags, if appropriate.

Simplify the TEARDOWN handlers and the thress teardown functions by incporporating
the above code in the teardown_phase_two (for AP2) and teardown (fpr AP1) functions.

It means that closing a connection will block on the principal_conn_lock, so if
you have the principal_conn_lock, closing will not complete until you release it.

Maybe we need a principal_conn_acquisition_lock for that...

Update RELEASENOTES-DEVELOPMENT.md

Quieten a debug message.

clang format

Remove a few debug messages.

Set and reset bonjour flags correctly when it's a Classic Airplay session. Reintroduce a 1 second delay for the prempted session to go away.

Remove redundant (and now faulty) check to see if the player is also principal conn.

update to correspond with the move of  principal_conn_lock to a pthread_rwlock.

Change principal_conn_lock from a regular mutex to a read-write mutex, so it can be used to check and hold the current principal_conn unless it's being altered in get_play_lock or a release_play_lock.
Only allow access to the config.airplay_statusflags, build_bonjour_strings(NULL), mdns_update(NULL, secondary_txt_records) when read_lock is acquired on the principal_conn_lock.
Only allow changes to config.airplay_statusflags and only allow access to mdns_update if you are the principal conn.

The bonjour status flags and the info response plist calculations are still a dirty rotten hack.

Add a pthread_cleanup compatible function to release a pthread_rwlock.

move the declaration of principal_conn_lock from player.h to rtsp.h where it should be.