mood.c: Only print log message if we are actually reloading a mood.

ROUND_UP: Add missing parentheses().

chunk_queue.c: Fix some typos.

aft.c: Rename save_audio_file_info() and enum aft_row_offsets.

aft.c: Improve documentation of the add command.

server.ggo: Remove unused selector option.

Change syntax for selecting moods or playlists.

Use m/foo, p/bar instead of m:foo and p:bar. This is much friendlier
to bash completion.

Cosmetics.

Makefile.in: Change type of variables to simply expanded type.

configure.ac: Add some missing checks.

Noted by autoscan.

Make client_pre_select() and client_post_select() static.

Move functions around a bit to avoid forward-declarations.

client_common.c: Trivial cosmetic cleanups.

Change default loglevel for para_fsck from 4 to 3.

fsck.c: Improve log messages.

fsck.c: Fix NULL-pointer dereference.

fsck_cleanup() might be called with t == NULL.

fsck.c: Loglevel adjustments.

client_common.c: Kill invalid FIXME.

Short writes can not happen here.

client_common.c: Add a comment to client_connect().

README.afs: Add some information on para_fsck.

Fix para_client hangs.

In the client event handler we have to set the eof variable
to a negative value in order to shut down the stdout task.

Rename eof to error while at it.

Replace funny statement in para_next_signal() by readable code.

This happened to be correct only by coincidence. Quoting Linus:

Assignments inside tests should probably be relegated entirely
to loop constructs, where doing them outside the test changes
semantics.

03_STAT_no-clients.diff

Avoids messages like
Dec 15 20:01:25 _tiptop 1 stat_client_write: 0 client(s)
which would otherwise flood the debug log.

02_HTTP_access-list.diff

After adding IPv6 support the access list matching routine was left in
a broken state. This patch fixes the problem by continuing the support
for IPv4 address/netmask access lists.
The following problems are fixed:
 * support for a server listening to both IPv6 (AF_INET6) and IPv4 (AF_INET)
   address (since one can not assume that the server's address has AF_INET);
 * explicit support for addresses in network-byte-order (without having to reverse
   the netmask);
 * a routine to extract v6-mapped IPv4 addresses ("::FFFF:x.x.x.x") for IPv4
   clients connecting to a (dual-stack) IPv6 server.

When the access list is in use, the connection mode is automatically switched to
IPv4, i.e. IPv6 clients connecting to the server will be treated as if they were
in the deny-list.

01_DCCP_shutdown-unnecessary.diff

This calls shutdown(2) for unused communication paths in the DCCP sender/receiver:
 * the sender does not read data from the client and so calls shutdown(SHUT_RD);
 * the client does not send data to the server and so calls shutdown(SHUT_WR).
The advantage that this buys is a performance optimisation: using shutdown(2) in
DCCP means disabling the corresponding CCID kernel modules, which reduces the
processing costs, i.e.
 * when using SHUT_RD, the receiver congestion-control module is de-activated;
 * when using SHUT_WR, the sender congestion-control module is de-activated.

More information can be found on
http://www.erg.abdn.ac.uk/users/gerrit/dccp/notes/shutdown/

a/command_util.sh: Convert template members also in the short description.

Minor documentation improvements.

filter.h: Trivial documentation cleanups.

More error cleanups.

error.h: Remove unused E_CHDIR.

error.h: Remove some unused errors.

Kill unused E_FSTAT.

error.h: Kill unused DCCP_ERRORS.

Replace eof by error in receivers/filters/writers.

This way it's possible to tell at a later time why
the receiver/filter/writer terminated.

This allows to increase the delay for reconnecting in case
the receiver failed to connect to para_server: Let the receivers
set the error value to -E_RECV_EOF in case a normal end of file
event occurred and check this value when calculating the restart
barrier.

Fix some typos.

Merge commit 'remotes/fml/master'

Fix a gcc warning on MacOS.

Makefile.in: Avoid gcc warnings on non-linux systems.

-fdata-sections, -ffunction-sections, -Wl,--gc-sections, -Wstrict-prototypes
and -Wshadow produce quite a lot of warnings on at least NetBSD and MacOS. So
use these flags only on Linux.

Fix SIGUSR1 handling on Solaris.

para_client dies the second time it receives SIGUSR1, because the signal action
is restored to the default state once the signal handler has been called.

Merge commit 'remotes/fml/master'

server.c: Check return value of signal().

And replace the ugly code that counted the number of calls to
para_install_sighandler() by equivalent, but less ugly code.

Simplify SIGUR1 handling.

There's no need to catch this signal in server.c as it is unly used to notify
its children. So just ignore it.

stat.c: Simplify stat_client_write().

Use write_ok() instead of open coding.

Merge commit 'remotes/yangtse/master'

Merge commit 'remotes/fml/master'

afs.h: Doxify comment for struct audio_file_data.

afs.h: Kill unused function declaration close_audio_file().

Add new "screenshots".

Add new status items num_chunks and chunk_time.

Merge commit 'remotes/fml/master'

alsa_write.c: Don't abort on EAGAIN.

Just retry a bit later.

command_util.sh: Add missing newline.

Fix some gcc warnings on Solaris:

warning: int format, pid_t arg
warning: int format, long int arg
int format, uid_t arg
int format, gid_t arg

CREDITS update.

net.c: Make host_and_port() static.

It's only used in net.c. Some more trivial cleanups also.

http_send.c: Fix another doxygen warning.

Found \endcond command without matching \cond

net.c: Fix a doxygen warning and trivial coding style cleanups.

No functional changes. Fixes the following doxygen warning:

Warning: documented function `getsockname for local or fn getpeername' was not declared or defined.

Cosmetic cleanups include:

- Reduce line width to 80 chars in most cases.
- Replace @param by \param.
- Replace tabs by spaces.

Trivial whitespace cleanups.

Merge commit 'remotes/fml/master'

Fix gcc warning on FreeBSD.

daemon.c:184: warning: long int format, int arg (arg 2)

Mention IPv6 support in NEWS and FEATURES.

Fix compilation on NetBSD.

At least on NetBSD-3.1, AI_V4MAPPED, AI_ALL, and AI_ADDRCONFIG
are all undefined. The fix is to #define these to zero if they
are not defined in netdb.h. This means that no hints will be
given in this case which is the best we can do.

alsa_write: Print more detailed error message.

06_Unix-Domain-Sockets.diff

[LOCAL SOCKETS]: Tidy up and share common code

This patch just suggested itself - after all the other changes have been made,
the only remaining customers of PARA_CONNECT were the Unix domain sockets.

The patch generates a counterpart to create_local_socket(), for the peer talking
over a Unix domain socket.

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

05_DCCP-Remove-Unused.diff

Since DCCP now also uses the makesock function from net.c, it does no longer
need the functions from dccp.c -- the file is thus removed by this patch.

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

04_DCCP-socket-functions.diff

This converts the DCCP streaming unit to use the makesock interface.

As much as possible, the format of the system calls was modelled after /
adapted to that of the HTTP sender/receiver.

This includes the use of a `name' field for the remote socket name of
the connecting client.

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

03_TCP-socket-functions.diff

This converts the control connection and the HTTP streaming module to use the makesock() interface.

The specific changes are:
 * replace tcp_connect() directly with makesock();
 * replace tcp_listen() with para_listen() (in turn a wrapper around makesock());
 * init_sockaddr() becomes obsolete;
 * replaced use of sockaddr_in in para_accept() with NULL sockaddr argument
  -- the `remote' name of the socket is always available via getpeername(2),
-- this is exploited by replacing the reverse-name lookup with remote_name();
 * the sockaddr_in field `addr' of `struct http_client' has been replaced with `name' string
  -- name is filled in as in para_accept() above, using remote_name(),
-- unlike above, the remote_fd() call can later not be done, due to close_listed_fds().

The one thing I am really not sure about is host_in_access_perm_list():
 * It needed to be converted to allow address-independent HTTP streaming.
 * The conversion represents the obvious choice for doing this.
 * However, I have not (yet) tested it.
 * A problem that may arise is that when listening using AF_UNSPEC then _all_ addresses appear
   as IPv6 addresses (i.e. the test "family == AF_INET" would never hold true). In this case,
   a possible solution is to switch between AF_UNSPEC and AF_INET dependint on whether the
   configurable access-list is empty or non-empty, respectively.
 * A possible place for this chance would be http_send_init() or any other function calling
   open_tcp_port().

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

02_Reverse-lookup-support.diff

In some parts of the code a reverse-lookup (e.g. "connection from xxx") is used.
An address-family independent reverse-lookup function is provided by this patch,
exploiting that a socket is described by {src addr, src port, dst addr, dst port}:
 * host_and_port(): prints host and port number of given socket,
 * local_name():    pretty-prints src-addr/src-port,
 * remote_name():   pretty-prints dst-addr/dst-port;

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

01_IPv6-Basic-Support.diff

This patch provides an algorithm which, given a hostname and a numeric port identifier,
will look up all matching IPv4/IPv6 addresses and either bind it (for passive sockets)
or connect to it (for active sockets).

Certain socket options need to be set before a connection is established. Since in
paraslash the demand for such options is not very big at this time, the only present
case (setting SO_REUSEADDR on passive sockets) has been integrated into the main loop.

A more sophisticated variant for setting pre-connection socket options exists and is
available as part of the DCCP library. The current solution is extensible, i.e. if one
wants to later add more options, the alternative (more sophisticated approach) can be
used. But for now, that would just have meant code bloat.

The subsequent patches in this set will step by step replace the old infrastructure
with the use of this algorithm.

Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>

INSTALL: Use $LOGNAME instead of hardcoding the user name.

Fix link to gitweb on the main page.

replace PUBLIC_KEY.

The new key adds my email address of tuebingen, and deletes
the tud and the www.paraslash.org email addresses.

Remove key.anonymous.

It was only needed for the live demo.

com_addatt: Minor improvements.

- Use get_attribute_bitnum_by_name() instead of open-coding.
- Use new error code E_ATT_TABLE_FULL.
- Move definition of objs[ATTCOL_NAME] down to where it is needed.
- Kill FIXME comment: We can not use find_greatest_att_bitnum() here.

string.h: for_each_line_ro() takes a _pointer_ to the line_handler.

afs.h: Add more documentation.

afs.h: Fix typo in comment.

afh_common.c: Add documentation of afh_init().

error.h: Cosmetics.

The setatt command needs at least two arguments.

Fix a couple of typos in error messages.

com_setatt(): Generate an afs event if attributes were modified.

giweb.css: Make backround color for commit messages a bit darker.

net.c: Typo.

afs.h: Make doxygen happy again.

It interprets <foo> as html.

README.afs: Minor improvements.

Rename init_tcp_socket() to tcp_listen() and further net.c cleanups.

Cleanups consist in the following:

- get rid of setserversockopts(). It had only one caller
  within net.c, so incorporate it into tcp_listen().

- Kill E_BIND, E_LISTEN, E_SETSOCKOPT and E_SOCKET. Return
  converted errno instead.

Introduce tcp_connect().

This replaces get_host_info(). The two callers of the latter,
client_connect() and http_recv_open(), did exactly the same, namely
initializing a struct sockaddr_in which was then used for PARA_CONNECT().

So move this common code to tcp_connect(). Additional benefit of this
is that the only remaining callers of init_sockaddr() are in net.c, so
make this function static.

net.c: Cosmetics.

Kill E_ACCEPT.

Get rid of E_SEND.

This patch also simplifies sendall() a bit and fixes two bugs in
add_one_audio_file():

- If the check for existence of a hash sister failed, we missed
  to unmap the audio file.
- If an error occurred we send out a message to the client. If
  this message can not be sent (because the client terminated
  the connection) we returned success anyway and happily
  tried to add the next audio file.

Some documentation improvements as well.

Get rid of E_RECV.

recv_bin_buffer() returns the converted errno on errors.
So there's no point in having a paraslash error number for recv errors.

Minor doxygen improvements.

ogg_afh.c: Increase eof timeval.

gui: Reorder display of status items.

gui.c: Handle empty status items properly.

gui.c: Cosmetics.

gui.c: Increase numbers of color pairs not related to themes.

So that they do not overlap with status items color pairs.

Rename mood_method "name_like" to "path_matches".

Fix rename detection.

If a file is renamed and re-added the old code would segfault because of
a NULL pointer dereference in save_chunk_table(). Fix is trivial.

com_lsblob(): Fix sorting order.