Merge commit 'athcx/next' into next

Fix off-by-one bug in mp3_afh.

Kill remaining instances of signal().

Also, make para_sigaction() and para_install_sighandler() return void
and exit on errors.

server: Use para_sigaction() to install the temporary SIGCHLD handler.

Use para_sigaction() in command handlers.

This allows to get rid of an ugly hack for solaris. The patch also adds checks
for whether the signals were reset sucessfully in the command handler and
aborts early on errors.

Introduce para_sigaction().

This wrapper for sigaction() is public and may be used to setup
a handler different from the generic handler that gets installed
for the signal by para_install_sighandler().

signal: Switch from signal() to sigaction.

Use of signal() should be avoided because the behavior of signal() varies
across Unix versions, and has also varied historically across different
versions of Linux.

This patch rewrites para_install_sighandler so that it calls sigaction()
instead of signal(). The implementation is taken from good old APUE.

There are a couple of other users of signal() in the paraslash code. Most
of which are OK because they use signal() only to ignore/reset a signal which
happens to be the only portable use of signal(). All other users of signal()
have to be converted in subsequent patches.

Auto-adjust fecdec output buffer size.

Fix a memory leak in the fecdec filter.

fecdec: Return an error on zero size slices.

INSTALL: Minor improvements.

Fix some memory leaks.

In fact, there was only one "real" leak: An empty info string (100
bytes) could leak on file change.

Destroy the close_on_fork_list in close_listed_fds().

This is called from child context and the child does no longer
need the list once all fds have been closed.

server: Use a temporary SIGCHLD handler to catch afs init failures.

If afs dies immediately on startup (because of dirty tables) para_server
would not notice as the SIGCHLD handler is being intalled after the
call to afs_init().

audiod: Improve display of play time.

Keeping track of the offset, server stream start and the duration
of the audio file length at the time the receiver/writer was started
allows to compute the play time more accurately.

Fix various dead stores found by the clang static analyzer.

udp_send: Also mention the FEC parameters of all targets.

wav: Replace assertion by a proper error message.

This assertion can be hit easily by choosing wav as the first filter
of the filter chain.

GET_NUM_DIGITS: Evaluate macro parameter only once.

Fix possible NULL pointer dereference in afh_get_header().

mp3: Avoid rounding errors during bitrate calculation.

Just add up the bitrates of all frames and divide by the number
of frames afterwards. By using uint64_t for the sum, we should
be safe wrt. overflows.

Fix year in comment to color.c.

The code was actually stolen this year :)

alsa: Split alsa_open().

Only perform what is absolutely necessary in alsa_open(). Call the
real alsa initialization from alsa_post_select(), when we actually
have audio data available to start streaming.

This should get rid of the sound artefacts that happen sometimes
at the beginning of the stream.

write: Get rid of wn->chunk_bytes and wng->max_chunk_bytes.

These are not used anyway. Make all writers return 1 on success.

Minor documentation updates.

alsa: Simplify alsa_open().

No need to call snd_pcm_info() as we do not use it anyway.

audiod: Improve receiver restart logic.

Use the vss status flags and the stream_start time announced by
para_server to decide when to start a new receiver.

This is better than basing this decision only on whether the vss status
equals "playing" as we did before. This allows to reliably detect when it
is safe to start a new receiver while the filter/writer of the current
stream are still active.

The old code would simply wait until filter/writer complete which, due to
buffering, might be much too late.

vss: Take fec group durations into account when computing the eof barrier.

On eof, we must wait at least as long as the duration of the last sent
fec group.

Fix a typo in the documentation to struct writer.

alsa: Revert 596d1d97f90854bd941db02f1353df8e1196ccc0

This was really a brain fart. Instead, introduce a small delay
in alsa_open() that defers audio data writing by 100ms. This seems
to be enough to get rid of the buffer underruns with the udp sender.

audiod: Change return value of open_current_receiver() to void.

Its single caller ignores the return value.

Move struct status_task from audiod.h to audiod.c.

It's only used in audiod.c.

vss: Fix need_audio_header().

This braino caused the audio file header to be included in each FEC group.

vss: Improve EOF handling.

Set the "next" flag only if no more writes to fec clients happen.

vss: Handle errors in setup_next_fec_group() properly.

setup_next_fec_group() might return a negative value.

recv_afs_msg(): Fix indentation.

Make FEC work with oggvorbis streams.

Unfortunately, this required to rewrite large parts of the fec code
in vss.c. The new code behaves better if the FEC parameters are not
suitable for the current audio file, for example if the audio file header
does not fit into the desired number of slices. It detects this situation
and adjusts the given FEC parameters accordingly.

afh: Introduce afh_get_largest_chunk_size().

This will be needed by the fec code in order to validate the
fec parameters.

oggdec: Abort on input errors.

fec: Fix comment to invert_mat().

alsa_write: Handle EPIPE errors properly.

There's no need to throw away the data if a buffer underrun occurs.
Just try to write the same data again after the snd_pcm_prepare().

Avoid division by zero in setup_next_fec_group().

This needs more work.

Fix initialization of fec_client_list.

The list must be initialized _before_ the init functions of the
senders are called, because the udp_send_init() might add fec clients
to that list.

NEWS, CREDITS, FEATURES update.

Recognize the eof packet also in the udp receiver.

This fixes two bugs: The last fec group of an audio file was not
decoded at all, and "para_recv -r udp" did not exit at the end of
a multicast stream.

Fix some gcc warnings on x86_64.

Merge commit 'meins/master'

vss.c: Improve the fec timing code.

This patch

- Fixes a the bug where assert(start_buf) caused para_server
to abort.
- Gets rid of buffer underruns by introducing a quick hack that
sends all but the first slice a bit early.
- Moves some code from compute_next_fec_slice() to
setup_next_fec_group() where it belongs.

Simplify dispatch_slice().

fecdec: Add some more documentation, minor improvements.

fecdec: Minor cleanups.

fecdec.c: Add some more sanity checks to find_group().

Make fec parameters configurable.

This patch allows to specify the fec parameters in the add subcommand
like this:

para_client sender udp add 224.0.1.38:8000/1400:63:64

More source code documentation.

vss.h: Kill unused VSS_CHANGE.

fecdec.c: Don't warn about incomplete slices at close time.

Move that warning to free_oldest_group().

fecdec_filter.c: Kill useless UNUSED_GROUP_NUM.

A group is complete iff num_received_slices >= data_slices_per_group.

fecdec_filter cleanups and documentation improvements.

Rename struct fec_group to fecdec_group as we already have struct
fec_group in vss.c. Also kill useless INPUT_BUFFER_SIZE define.

Add some more FEC source code documentation.

Trivial doxygen fixes.

Define _GNU_SOURCE to get the ucred structure.

This is needed for glibc >= 2.8.

Simplify compute_slice_timeout().

vss.c: Get rid of some casts.

Merge branch 'gerrit'

Add forward error correction code to the udp sender/receiver.

This patch adds the first draft of a FEC implementation based
on code by Luigi Rizzo.

On the server side, the FEC encoding is done within the virtual
streaming system which also contains the timing routines for sending
a FEC-encoded audio stream. Senders my request such an encoded stream
by calling vss_add_fec_client() with a fec_client_parms structure
that contains the FEC parameters and a callback function which is used
to actually send the data.

On the receiver side, the new fecdec filter is introduced which must
be used to decode a FEC-encoded stream. As the fec parameters are
contained in the header of each data slice of the encoded stream, no
options to this filter are necessary.

ATM, FEC is only used by the udp sender/receiver, but other protocols
can be easily changed to use FEC as well.

This new code is still experimental, lacks documentation and the fec
parameters can currently only be changed by editing the source code.

URL/UDPv6 support for the UDP sender.

This adds support for URL strings to support IPv4/IPv6 addresses, hostnames,
and optional port numbers to the UDP sender. The syntax is based on RFC 3986,
example URL strings to be used with the 'add'/'delete' commands are:

     10.10.1.2:8000 # host:port
     10.10.1.2 # host with default port 8000
     localhost # hostname (for IPv4 or IPv6 host)
     localhost:8001 # hostname with port
     [::1]:8000  # same, but different syntax
     [badc0de::1] # IPv6 host with default port 8000

The 'delete' command now also supports port-wildcarding, i.e. not specifying
the port in the 'delete' command removes all targets whose host names match.

The UDP sender has been tested to function with IPv6, including multicasting:
 * server: para_server --udp_target=[ff05::beef]:8000 --udp_mcast_iface=eth1
 * client: para_audiod -F -D -r 'ogg:udp -i ff05::beef -I eth0'

Auxiliary functions to parse and validate parts of URIs.

To support URI syntax in the manner of RFC 3986, this adds a self-contained set
of functions to parse (parts of) URIs and to validate such parts using basic
syntax checks.

To avoid blocking while resolving DNS strings, strings that do not have the
characteristics of native IPv4/v6 address strings always pass; a wrong hostname
will then be detected later when connecting.

Note: strictly speaking, this does not deal with URIs, since the 'scheme' part
      is missing and only the "locator" part is parsed. It is possible to extend
      this later, using something like dccp://host[:port], http://host[:port],
      or udp://host[:port] to specify the transport protocols of paraslash.

aacdec: Do not fail on emtpy buffers.

alsa: Do not fail if sw params could not be installed.

This is not fatal, so just print a warining.

Simplify client_open().

Use the file_exists() helper and add missing documenation for
the loglevel arg.

Merge commit 'fml/master'

Put IPv4-specific parts inside the ACL subsystem

The ACL subsystem performs access validation based on IPv4 address /
netmask pairs. Since the internals of this system are exposed via
struct sender_command_data, other subsystems can only use IPv4.

This patch migrates the IPv4-specific parts of the ACL subsystem inside
that system and replaces the 'struct in_addr' address with a host
string. A subsequent patch converts the remaining subsystems to also
use strings instead of AF-dependent addresses.

For the present purpose the string size (fixed) is over-dimensioned,
since a valid IPv4 address contains at most 15 = 4*3 + 3 + 1 bytes
(including the 3 dots and the '\0'). The size should also be sufficient
for hostnames, since RFC 1123, 2.1 suggests a maximum length of 255.

Since the conversion string -> struct in_addr now happens later,
syntax checks have been added for argument strings.

Typos

Found when rtfm.

Allow to specify UDPv4/6 multicast interface

Using a Posix function, this allows to specify outgoing/incoming
multicast interfaces, e.g.

  para_server --udp_target=224.0.1.38:8000 --udp_mcast_iface=eth1

or

  para_audiod -D -r 'ogg:udp -i 224.0.1.38 -I eth0'

The option simplifies multicast streaming, which is now possible without
having to set a multicast route when there is more than one network interface.

This option works unconditionally for UDPv6. For UDPv4 it is only enabled
on hosts that support `struct ip_mreqn', support for which is detected
via configure.

On OSes that do not support this struct, a warning message is printed;
while it is possible to add the same functionality also in those cases,
it would complicate the implementation and thus has been left out.

Support for UDPv4/v6 multicast streaming

This provides self-contained support for UDPv4/v6 multicast:
 * receiver joins multicast v4/v6 address;
 * sender, on encountering a multicast address, sets TTL/IPv6
   number of multicast hops, and (as a precaution) enables looping
   back of multicast packets.

The use of the sender is still restricted to UDPv4, since it is also
necessary to perform changes to the (IPv4-based) udp_target struct,
which should be done in a separate patch.

Fix a typo in gui.c.

Merge commit 'fml/master'

Fix the ROUND_UP macro.

Doooh.

Do not allocate too much space for the mp3 chunk table.

The chunk_table contains an array of uint32_t values.

Merge commit 'fml/master'

Merge commit 'meins/master'

Merge commit 'meins/master'

afs.cmd: Add documentation of ls -lc.

attribute.c: Trivial docu fix.

Ignore SIGUSR1 before forking the afs process.

Otherwise, afs might get killed by SIGUSR1 before it gets a chance
to ignore this signal.

afs: Fix a shm leak.

Even if the command handler stops to read the command output
afs must continue to read shmids from server fd so that it can
destroy all shared memory areas that the server created for afs.

Call shm(destroy) on the mmd shared memory area right after creating it.

It will only be destroyed when the last reference is gone, which
is when para_server exits. This allows to get rid of a global
variable. Moreover, the shared memory area will be destroyed even
in case para_server crashes.

Simplify ggo/makefile.

Use a define rather than shell code to compute the options to gengetopt.

Fix config file handling for audiod and clean up init functions.

As noted by Gerrit Renker, audiod refused to start if the --config_file
option was not given and the default config file does not exist.

Fix this bug and rename some of the init functions to make the naming
more consistent.

Merge commit 'fml/master'

Fix build as root.

As noted by Gerrit Renker, when trying to extract help text from
./para_audiod, the build stops when logged in as root because
para_audiod refuses to start as root if --user and --group are
not given.

This patch moves the check for UID==0 after the check for --help and
--detailed-help so that the help text may be extracted as root even
if no --user and --group options are given.

Build problem when no audiod.conf exists

This avoids stopping the build process and allows users to read
the detailed help before getting the error message about a missing
audiod.conf file - fixed by parsing after testing for the help text.

Don't let make write the filename into the output

The 'cd' output in the command echoed the directory into the
output, which is not welcome in the output file (gengetopt
did not like that).

Fix para_client's --loglevel option.

Without this fix, para_client always used loglevel debug.

Fix compile warning on NetBSD.

This fixes

color.c:71: warning: array subscript has type 'char'

osx_write.c: Add missing include.

Merge commit 'meins/master'

Merge commit 'athcx/master'

make_wav_header(): Add some comments.

server.c: Only try to print errors when not in daemon mode.