Refuse to add invalid attibute names.

And tell the client rather than the log file about any errors.

fd.c: Remove unused label change_to_orig_dir.

Merge commit 'remotes/fml/v0.3' into v0.3

Add --gc-sections to LDFLAGS.

This, together with -fdata-sections and -ffunction-sections brought
nice size reductions. Before:

-rwxr-xr-x 1 maan fml  29K Oct 11 14:32 para_audioc*
-rwxr-xr-x 1 maan fml  42K Oct 11 14:32 para_client*
-rwxr-xr-x 1 maan fml  33K Oct 11 14:32 para_fade*
-rwxr-xr-x 1 maan fml  38K Oct 11 14:32 para_filter*
-rwxr-xr-x 1 maan fml  54K Oct 11 14:32 para_fsck*
-rwxr-xr-x 1 maan fml  52K Oct 11 14:32 para_gui*
-rwxr-xr-x 1 maan fml  42K Oct 11 14:32 para_recv*
-rwxr-xr-x 1 maan fml 216K Oct 11 14:32 para_server*
-rwxr-xr-x 1 maan fml  38K Oct 11 14:32 para_write*

After:

-rwxr-xr-x 1 maan fml  21K Oct 11 14:31 para_audioc*
-rwxr-xr-x 1 maan fml  33K Oct 11 14:31 para_client*
-rwxr-xr-x 1 maan fml  29K Oct 11 14:31 para_fade*
-rwxr-xr-x 1 maan fml  29K Oct 11 14:31 para_filter*
-rwxr-xr-x 1 maan fml  45K Oct 11 14:31 para_fsck*
-rwxr-xr-x 1 maan fml  47K Oct 11 14:31 para_gui*
-rwxr-xr-x 1 maan fml  30K Oct 11 14:31 para_recv*
-rwxr-xr-x 1 maan fml 208K Oct 11 14:31 para_server*
-rwxr-xr-x 1 maan fml  29K Oct 11 14:31 para_write*

Get rid of E_WAITPID.

Get rid of E_SIGNAL_PIPE.

If creation of the signal pipe fails, we abort anyway. So print
the errno in the fatal log message instead of the fact that the create
failed.

fd.c: Cosmetics.

Get rid of E_FCHDIR.

Return the system error code instead.

Get rid of E_OPENDIR.

Return the system error instead.

para_select(): Return a proper error code on errors.

Remove some system errors from errno.h

No need to have system errors duplicated in error.h. Just return
-ERRNO_TO_PARA_ERROR(errno) in these cases.

Get rid of -E_F_GETFL and -E_F_SETFL.

Return the proper errno instead, using the new ERRNO_TO_PARA_ERROR macro..

error.h: Add more helper macros.

.ico Version of the fancy new logo.

error.h: Make PARA_STRERROR understand system errors.

error.h: Remove some unused errors.

Only declare error codes that are safe to use.

The following type of bug occurred frequently since the introduction
of error.h: An error code was defined for some file a.c and was used
in another file b.c. This is all good if each executable that contains
b.o also contains a.o.

If this is not the case, i.e. if there is an executable that contains
b.o but _not_ a.o, the error texts of all errors defined for a.c are
not included in the executable. This results in a segmentation fault
at runtime if PARA_STRERROR() is used for such an error code.

Unfortunately, this was hard to spot because all error codes were
visible everywhere. This patch gets rid of this flaw by making
only those error codes visible for b.c which are safe to use.

According to the above, the set of error codes which are safe to use
in b.c can be defined as follows. Let E_1, ... E_n be the set of
executables that contain b.o and let, for j=1,...n,  S_j be the
set of files those files contained in E_j. Then, all errors defined
in the intersection

S_1 \cap S_2 \cap ... \cap S_n

are safe to use in b.c. The patch adds a function to configure.ac
That computes this intersection. Unfortunately, it is rather expensive.

The good news is that this simplifies error.h a bit and that it
already revealed a couple of bugs which are also fixed in the patch:

- afs.c used E_SIGNAL_CAUGHT which was only defined in audiod.
  Introduce E_AFS_SIGNAL and change afs.c accordingly.
- audiod uses E_RECV_SYNTAX, which belonged to recv which is
  not contained in audiod. Move it to recv_common instead.
  This bug is also present in v0.2.x and needs a similar fix.
- E_NOTDIR was used in fd.c, but was defined in osl.c. Not all
  executables that include fd.o also include osl.o. Move E_NOTDIR
  to the set of errors for fd.

INSTALL: Update to v0.3.

Still needs more work.

mood.c: Fix comment for the scoring function.

Add more documentation.

command_util.sh: Fix man page format.

It really looked ugly as it were. Remove leading tabs in help texts.

afs.cmd: Beautify help texts.

com_add(): Write messages to the client rather than to the log.

afs: Handle SIGHUP.

This required moving around some functions but was otherwise
straight forward.

com_add(): Only accept absolute paths.

osx_write.c: Add two missing headers.

Switch from inet_aton() to inet_pton().

Solaris doesn't have inet_aton().

Make it compile on Solaris.

This required a bit of work as Solaris has  gethostbyname() and
socket-related functions not in libc but in libsocket and libnsl.
So add corresponding checks to configure.ac.

Moreover, make para_gui optional and add configure options for
specifying the directory containing the curses headers and curses
lib.

README: Mention dependence on gnu make.

Solaris make fails miserably.

FEATURES: Update.

audioc: Get rid of the temporary socket file.

This allows to kill the --tmpdir option as well.

afs.c: Remove some dead code.

Add command line options --mood and --playlist.

And replace the temporary strings which were hardcoded
in afs.c.

afs.c: Introduce client list for afs (fixes dosable bug).

With the old code, a malicious local user could easily cause a DOS
by connecting to the local afs socket but never sending any data
through that socket. This caused afs to block on the read() from the
new client fd.

This patch fixes this flaw by introducing a doubly liked list of
connected clients.  Each entry in this list contains an open file
descriptor and the time the client connected.

In command_pre_select() we add each fd in the list to the set of
fds to be checked for readability. In command_post_select() we loop
through that list and check each fd again. Whenever the fd for a
client is readable we read from that fd and execute an afs command
if the client sent the valid magic cookie.

However, if the fd is not ready, we close the connection if more than
AFS_CLIENT_TIMEOUT seconds (default: 3) have passed since the client
connected. This hopefully avoids another possible DOS: A malicious
user could try to flood us with connections until the limit of open
fds is reached.

Make it compile on FreeBSD.

This was easier than expected. Only two items needed fixing:

- signal.h has to be included for several files that use signals.
- The __used and __packed macros are already defined on FreeBSD which
  leads to an error at compile time. Fortunately paraslash doesn't use
  these macros at all, so we can simply remove them.

gcc-compat.h: Remove compatibility code for gcc-2.

gcc-2 is not supported any longer.

fsck.c: Fix several bugs in dump mode.

- Don't abort if a  directory can't be created because it already exists.
- Use correct variable for the disk storage name column.
- If more than one table is to be dumped, use a directory for each table.
- Remove __a_unused for main()'s argc which _is_ being used.

Implement com_cpsi().

Merge commit 'remotes/meins/v0.3' into v0.3

com_add(): Always check the return value of send_va_buffer().

And abort on send errors.

for_each_file_in_dir(): Ignore permission errors for subdirectories.

error.h: Add CHDIR_PERM error code.

fd.h: Add declaration of para_chdir().

fd.c: Introduce para_chdir().

Also, fix a bug in para_open() which would return -1 instead of
a proper error code.

Implement add -a (add all audio files).

mood.c: Remove debug log message.

Implement mailbox output format for ls.

Alos, fix a bug in com_lsblob_callback() which caused incorrect
detection of the given options.

mood.c: Introduce DEFINE_MOOD_METHOD and DEFINE_MOOD_METHOD_WITH_CLEANUP.

This simplifies the definition of the mood_methods array.

attribute.c: Fix com_rmatt().

We have to get the bit number _before_ we remove the row.

com_ls: Don't use FNM_PATHNAME.

It's just annoying.

afs.cmd: Fix typo in setatt help.

attribute.c: Clear attribute bits already at remove time.

If an attribute gets removed, the old code deferred clearing the
corresponding attribute bit in the afsi of all audio files until
that bit got reclaimed by add_att.  This is problematic because of
the new no_attribute_set mood method.

So clear the bit already in the rmatt callback.

mood.c: Add no_attributes_set mood method.

Also fix a bug in mm_played_rarely_parser(): If no arg was
specified in the mood definition, a NULL pointer is passed to that
function, so it must not blindly dereference that pointer.

Speed up mood loading.

The old code passed a pointer to the row of the audio file table to the
mood scoring functions. The scoring functions had to obtain the afsi,
afhi, path from that row pointer. Thus, at mood load time, the afsi,
afhi, path was extracted N times for each audio file if there are N
mood lines that used this info.

This patch changes the mood scoring functions so that they take
three pointers to afsi, afhi, path instead of the row pointer.
add_item_score(), the single caller of the mood scoring functions,
provides these pointers, i.e. they get extracted from the row pointer
only once per audio file.

This is still not optimal as most scoring functions use only one of
the three pointers. But let's not over-engineer the thing.

mood.c: Switch to fnmatch() in mood method name_like.

Also, change doxygen documentation to the new style.

grab_client.c: Change doxygen comments to the new style.

Even more typo fixes.

Still more typo fixes.

FEATURES, INSTALL: Fix some more typos.

README,INSTALL: Fix some spelling errors.

Make com_lsblob() and com_lsatt() more similar to each other.

afs.cmd: Update help output.

Implement pattern matching for com_lsatt().

attribute.c: Implement pattern matching for com_rmatt().

A straight forward task with the infrastructure already in place.

Implement pattern matching for com_touch().

Also, add -p and -v options and fix converting the last played
time which is a 64 bit intger, so strtol() will fail do it on 32
bit machines after 2039 ;)

To fix this, introduce para_atoi64() which calls strtoll() which
is 64 bit even on 32 bit machines and define para_atoi32() which
calls para_atoi64() and checks the boundaries.

com_afs_rm(): Implement pattern matching and add -v, -p, -f flags.

The -r flag is still to be implemented though..

Gerneralize for_each_matching_blob().

It's not only useful for blobs. For instance the rm and the touch
commands might also want to use something similar. So move it to
afs.c and call it for_each_matching_row().

sha1.h: Typo.

attribute.c: Add doxygen file comment.

Move enum ls_sorting_method from afs.h to aft.c and add documentation.

Add documentation of enum blob_table_columns.

Fix table init.

The previous patch introduced two instances of a bug which weren't noticed by
the compiler because attribute_table and score_table were declared as void *
pointers. Change that to struct osl_table *, and add the missing '&' operator.

void * pointers a evil.

Get rid of the "table" member in struct table_info.

afs.c has no busyness in dealing with the table, so do not
expose it to afs.c.

addblob: Overwrite existing blobs.

Previously, an attempt to add a blob with a name of an already
existing blob failed (siltenly). Just replace the existing blob
with the new contents instead.

afs ls -lv: Also print the tag info.

com_check(): Implement mood checking.

make get_playlist_data() generic.

Other blob types might use an analogous function as well, so introduce
blob_get_name_and_def_by_row() in blob.c, and use some preprocessor
magic to define such a function for each blob type.

Change all users of get_playlist_data() to use the new
pl_blob_get_name_and_def_by_row() and remove get_playlist_data().

mood.c: Rename mood_loop() to load_mood_loop_func().

And add a comment that describes why this function returns negative on success,
and positive on errors.

Update Doxyfile to doxygen-1.5.3.

Automatic update, done with doxygen -u.

afs: Move some ls-related stuff from afs.h to aft.c.

Also, add GPL header and documentation of struct afs_info to afs.h.

More mood cleanups.

- Avoid use of global current_mood pointer where possible.
- Change name of non-static functions that use the current mood pointer:
mood_open() -> change_current_mood()
mood_close() -> close_current_mood()
mood_reload() -> reload_current_mood()

compute_mood_score(): Add additional mood * parameter.

Thid reduces the use of the global current_mood pointer.

compute_mood_score(): Rename "row" parameter to "aft_row".

mood.c: Use current_mood as data pointer for parse_mood_line().

mood.c: Add GPL header.

mood.c: Change type of rows from "void *" to "struct osl_row *".

score.c: Remove an unused function declaration.

afs.cmd: Add documentation of the remaining commands.

And improve the formating of command options.

blob.c: Implement pattern matching.

Most of the blob functions now take a pattern list as arguments,
and the function does the job for all blobs that match any given
pattern in the list.

The only exceptions are addblob and mvblob where a pattern list
does not make sense.

Make rbtree_loop() and rbtree_loop_reverse() safe against removal of an entry.

Also, fix the documentation of osl_rbtree_loop() which claimed that an
osl_rbtree_loop_func takes two void pointers which is not true any more.

aft.c: Add GPL header.

afs.c: Avoid noisy log message.

para_server regularly sends SIGUSR1 to all of its children. afs
just ignores that signal, so only print a log message if a signal
different from SIGUSR1 was caught.

afs.cmd: Add documentation of all blob commands.

command_util.sh: Fix man output for templates.

afs.c: Fix documentation of stdin_command().

It doesn't read from stdin but from the given fd.

stdin.h, stdout.h: Trivial documentation cleanup.

stdout.[ch]: Make two functions static.

stdin.[ch]: Make two functions static.

stdin.c: Fix a typo in documentation.

Implement check for invalid playlist entries.

Also some minor improvements for the audio file table checks.