gui: Make album/comment layout look nicer.

Merge branch 'master' into next

Conflicts:
ogg_afh.c

NEWS update.

oggdec filter improvements.

Try to open the ogg vorbis callbacks as soon as possible rather
than waiting until the input buffer reaches the given initial buffer
size. If that fails, try again later when more data is available but
fail if the input buffer size is larger than the initial buffer size
and we can still not open the ov callbacks.

Also, if a hole was detected, likely because we started streaming
in the middle of the file, add an additional delay to avoid buffer
underruns.

Complete re-write of the ogg vorbis audio format handler.

The new code is quite a bit smaller, performes much better and chooses
a chunk time dependent on the average ogg page size rather than using
a hardcoded chunk time of 250ms.

fecdec: Defer decoding until the first slice of the second group arrives.

Otherwise, this could lead to buffer underruns in the decoding application
in case slices are missed. So introdce group_completion_status which tracks
whether we already have received the first group sucessfully and are waiting
for the first slice of the next group.

afh: Fix memory leak.

stdin: Increase input buffer size.

16K was a bit small for the ogg_decode filter.

tv_scale(): Avoid integer overflow.

Just use an uint64_t and compute everything in microseconds which even
simplifies the code a bit.

vss: Honor header chunk when computing FEC group start.

The first chunk of an ogg vorbis file is the audio file header
which should not be included in the calculation of the FEC
group timing.

FEC timing improvements.

Currently we compute the time of a FEC group as the number of
containing chunks times the chunk time. The time between sending
two slices therefore depends only on the number of chunks the group
contains. Groups containing many slices are sent with larger delays
than groups containing few slices.

This approach is not optimal for the following reason: Consider a group
containing only few slices which is followed by a group containing
many slices. This happens frequently at the end of VBR MP3 files which
contain some seconds of silence or applause at the end because this
last part is often encoded at a lower bitrate than the preceding part.

In this scenario buffer underruns in the receiving application can
easily occur if the previous FEC group has been decoded and completely
fed to the writer before enough slices of the next group have arrived
to decode the second group.

This patch changes the timing of FEC groups such that all but the
first group use the duration of the _previous_ group as the basis
for the timing.

vss_send(): Fix EOF-check for FEC clients.

If the last chunk has been sent to all http/dccp clients we have
to wait until the last FEC group has been sent before setting the
NEXT flag that causes all senders to shut down its clients. The old
code tested if a slice was sent to any FEC  client during vss_send()
and set the NEXT flag if nothing was sent.

However, this is not sufficient as there may be still slices available
which have to be sent at some future time. This patch teaches
vss_send() to detect this condition. It also renames the boolean
variable sent_something to fec_active, which is more to the point.

configure: Add link to osl homepage.

Merge branch 'master' into next

mood: Do not handle events if no mood is active.

In case a file is being removed with "para_client rm" while in playlist mode,
the following assertion triggers:

mood.c:521: del_afs_statistics: Assertion `n' failed.

This happens because the file is contained in the score table but statistics are not enabled.
Fix it by ignoring events if cuurent_mood is NULL.

filter: Register the filter chain as the last task.

Without this patch, the command

para_filter -f mp3dec < foo.mp3 > /dev/null

would take several minutes because the stdin buffer is usually full and the stdout buffer empty
which causes select() to be called with empty fd sets, resulting in a one second timeout.

Refuse to load unprotected private keys.

When loading a private key owned by the same user that accesses the key, check
the permissions of the key file. Error out if mode & 077 is non-zero.

Use RSA key blinding to protect against timing attacks.

Not that it matters much, but it doesn't hurt either.

Merge commit 'meins/next' into next

Merge branch 'master' into next

Fix check for short rsa keys.

And add a comment that explains the magic constant 41.

Merge branch 'master' into next

fade: Avoid a gcc warning.

The following warning is harmless and gcc is just being stupid:

fade.c: In function 'main':
fade.c:111: warning: 'val' may be used uninitialized in this function
fade.c:111: note: 'val' was declared here

Merge branch 'master' into next

Conflicts:
aft.c
command.c

Kill the quick hack that changed timing for the first fec goup.

This did never really work.

mp3dec: Improve bad main_data_begin pointer error handling.

These errors from mad_frame_decode() are non-fatal and happen if the
stream is started at the middle of the file, e.g. when para_audiod
is started while para_server is already streaming.

If libmad encounters such an error it throws away the first (and
probably the second) frame which messes up the timing in udp/fec mode,
causing an audible buffer underrun after the remaining frames of the
first fec group have been decoded and fed to the writer.

This patch makes the mp3dec filter keep track of bad main_data_begin
pointer errors that happen at the start of the stream. In this case
decoding is deferred until more data has arrived or 60ms have passed.

com_stat(): Be more anal in checking command line options.

Also check the number of arguments and use para_atoi32 to convert
the argument into an int.

Fix off-by-one bug in fixup_info_string().

This caused the chunk time not being properly displayed due to a
missing newline in the status output.

Merge branch 'master' into next

NEWS update.

Alsa timing improvements.

This moves the computation of the select timeout from
alsa_write_post_select() to alsa_write_pre_select(). The code now
computes when the next buffer underrun would occur and uses that
value to set the timeout for the next select call. This decreases
the number of writes to the alsa handle and therefore also the CPU
usage of para_write/para_audiod.

Merge remote branch 'fml/next' into next

NEWS update.

Fix syntax in documentation of the touch command.

Merge branch 'master' into next

daemon: Don't set the umask to zero.

Clearing umask creates more problems that it solves.

Fix two typos in daemon.c.

gui: Fix a bug in align_str().

This bug was introduced in 01117585. It messes up the display and can lead to a
segfault.

para.css: Remove superflous '#' chars.

Merge commit 'meins/master'

Merge branch 'master' into next

[filter]: Do filter conversion during post_select.

This is more natural and allows to get rid of the hack that
forced a minimal timout in case something was converted.

Merge branch 'master' into next

[gui]: Use the new -p option for the default stat command.

Revamp para_fade.

- Rename options
- Improve documentation
- Make --mode an enum option
- Cleanup ggo file indentation

Print an error message if the filter config file contains invalid options.

Merge branch 'master' into next

Teach para_fade to accept a mixer channel to fade.

This allows to fade not only the volume channel but all other channels that are
supported by oss.

Make some functions/variables static.

Found by sparse.

Move blob table macros from afs.h to blob.c.

They are only used there.

Rename LIST_HEAD to INITIALIZED_LIST_HEAD.

This fixes the following warning on NetBSD:

In file included from udp_send.c:25:
list.h:37:1: warning: "LIST_HEAD" redefined
In file included from /usr/include/net/if.h:84,
 from udp_send.c:14:
/usr/include/sys/queue.h:88:1: warning: this is the location of the previous definition

Merge branch 'master' into next

Kill unused para_reap_children().

Found by cppcheck.

configure.ac: Fix --with-osl-libs option.

Quite a few typos in there..

Fix a gcc warning on 64 bit systems.

client_common.c:175: warning: format '%d' expects type 'int', but argument 4 has type 'size_t'

Merge commit 'meins/next' into next

Add num_played mood method.

Remove the likely/unlikely macros.

These also slow down the compress filter, its only user, by about 10%.

Micro optimizations for the amp filter.

Use memcpy in the special case amp==0 (no amplification) and optimize the code in
the performance-critical loop. Intrestingly, using the likely()/unlikely() macros made the
code slower.

Results (three runs on identical input data on a 32bit x86 machine under Linux, gcc-4.4.0):

old with --amp 3:
        0m0.776s 0m0.790s 0m0.812s, avg: 792
new with --amp 3:
        0m0.456s 0m0.492s 0m0.477s, avg: 475
speedup: 1.67

old with --amp 0:
        0m0.791s 0m0.808s 0m0.810s, avg: 803
new with --amp 0:
        0m0.100s 0m0.103s 0m0.094s, avg: 99
speedup: 8.1

Don't include osl.h from the audio format handlers.

They don't need this header, and including it broke compilation with -O0.

Introduce LIST_HEAD macro and use it in audiod_command.

Using initialized list heads where possible allows to get rid of the checks whether the
list is already initialized.

Move functions only needed by audiod from stat.c to audiod_command.c.

This was 4 out of 5 and thus should decrease the size of executables for non-Linux
systems a bit as we don't use -fdata-sections -ffunction-sections and -Wl,--gc-sections
there.

Is also allows to remove some entries from para.h.

Merge branch 'master' into next

Fix memory leaks in [alsa,file,oss]write.c.

Small and harmless but anyway.

audiod: Increase default select timeout to 1s.

This was set to 99ms only for debugging purposes three years ago...

audiod: Only call audiod_status_dump twice per second.

audiod_status_dump() is quite expensive and is called in a rather hot path.

client_post_select(): Defer decrypting of server challenge.

The old code worked only by pure luck: We allocated the crypt buffer on the stack as
an ordinary automatic variable. This buffer was filled right after we received the challenge
from the server, but it was used in a _subsequent_ call to client_post_select(). There's
no guarantee that the content of the crypt buffer stays the same between these two calls.

So defer the decryption until the client status is CL_RECEIVED_CHALLENGE, i.e. until
the socket file descriptor is known to be ready for sending back the SHA1 of the decrypted
challenge.

Remove unused field challenge_nr from struct client_task.

A relict from 0.3.x.

Merge branch 'master' into next

Conflicts:
audiod_command.c

Remove unused label "out".

Merge branch 'master' into next

Conflicts:
audiod_command.c

audiod: Clean up status output.

It's a mess. Fortunately, this is much cleaner in 0.4.x..

audiod: Properly prefix uptime string.

The prefix was missing the first time the uptime was printed in the status output.

Merge branch 'master' into next

Conflicts:
audiod_command.c

audiod: Fix some memory leaks in the com_stat() error paths.

Merge branch 'master' into next

Fix a memory leak in filter.c.

We must free the filter configs as well.

Fix a typo in the udp_recv help.

Fix memory leaks in xxx_recv_init().

Not serious but it's good to make valgrind happy.

Merge branch 'master' into next

Conflicts:
client_common.c

Make RSA keys larger than 512 bit work.

This allows to use the 2048 bit keys which are necessary for paraslash-0.4.x also for
paraslash-0.3.5.

NEWS update.

Merge commit 'meins/next' into next

Add bitrate, frequency, and channels mood methods.

This isolates the number comparing code from the year mood method so that it can also
be used by the new methods.

Add mood methods {artist, title, album, comment}_matches.

Document the new mood methods and remove the documentation of the played_rarely
mood method that has been deleted recently.

Merge branch 'master' into next

Fix com_init() in case arguments are given.

As the ->name field of struct afs_table was only initialized in the afs process,
com_init(), which gets forked from the server process, did not see the table
names and would segfault due to a NULL pointer dereference if table names
were given as arguments.

Fix it by initializing the ->name fields in the definition of the afs_tables array.

This bug was introduced in commit 53d503ce back in 2007...

Scoring performance enhancements.

The old scoring code computed the afsi, afhi, and the path of the audio file for each
item of the mood,  which is unnecessary.  This patch moves these computations from
get_item_score() into compute_mood_score() so that afsi, afhi and path are only
computed once  per audio file.

As a result, get_item_score() can no longer fail, so we may skip the checks for negative
return values.

Merge branch 'master' into next

Conflicts:
mood.c

Scoring fixes.

Rename add_item_score to get_item_score(), properly check its return value,
and use the computed score only for admissible files.

Move mood methods to a separate file.

As more mood methods will be added in subsequent patches, it's good to separate
moods from mood methods.

This also removes the played_rarely mood method, since it never really worked: Whether
or not a file was played rarely depends on the afs statistics which have not been
computed at the time the mood scrore function is executed.

gui: Output "xxx tag not set" for unset tags rather than empty strings.

Add documentation to struct rc4_context.

Merge commit 'meins/next' into next

Merge commit 'fml/master' into next

Add the new "year" mood method.

This includes a Y2K heuristic for year tags that contain only two digits. If that two-digit
number N is less or equal than the current year minus 2000, assume that it means 2000 + N.
Otherwise, assume 1900 + N.

Examples: 0 -> 2000, 42 -> 1942, 9->2009.

Add missing regex include to generated command_list files.

Merge branch 'master' into next

Merge commit 'meins/next' into next

Introduce para_regcomp.

A wrapper for regcomp() that logs an error message if the regcomp() failed
and uses a return value according to the paraslash rules. Currently there is
only one user of regular expressions, is_v4_dot_quad(), which is converted
to para_regex(), but new callers will be added soon.

Unfortunately, this change made it necessary to include regex.h in all .c files
that use string.h. Clean up the order in which headers are included a bit while
we're at it.

Maybe I should rethink the rule "Only .c files shall include header files"...