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.

Merge commit 'remotes/fml/master'

Define XOPEN_SOURCE and __EXTENSIONS__ on Solaris.

These are needed for passing fd through a socket.

Don't garbage-collect sections.

This does not work on Solaris, NetBSD and MacOS. Leave it in as
a comment though.

Rename mark_fd_nonblock() to mark_fd_nonblocking().

Just to be consistent with mark_fd_blocking().

Rename sched() to schedule().

We already have struct sched..

REQUIREMENTS: Add pointers to software packages.

net.c: Return proper error value in recv_bin_buffer().

In handle_connect() we really need a blocking fd for the client.
At least on NetBSD the fd returned by accept() might be in non-blocking
mode. So introduce mark_fd_blocking() and use it in handle_connect().

Replace shebang in bash scripts.

NetBSD does not have /bin/bash.

afs.c: Simplify activate_mood_or_playlist().

This also gets rid of

warning: `mode' might be used uninitialized in this function

on gcc-3.3.3.

Add wrapper for isspace().

This avoids

warning: subscript has type `char'

on netbsd.

gitweb.css: Improve readability.

Add some gitweb-releated stuff.

overview.sk: Minor improvements.

Remove another link to the live-demo.

Kill live-demo web page.

Add final 0.2.17 stuff.

the paraslash-0.2.17 release tarball

Fix html header.

The location of the cgi-bin dir on systemlinux is different from
paraslash.org. This broke the gitweb link in the left menu.

download.in.html: Replace paraslash.org in clone command.

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

Announce NetBSD Port.

aft.c: Simplify print_list_item().

aft.c: Do not print directory if LS_FLAG_FULL_PATH was not given.

aft.c ls -lm: Fix unmapping of lyrics file.

Fix handling of invalid entries in the score table.

It may happen that the score table contains references to files that
no longer exist in the file system. The current code fails badly in
this case.

In open_next_audio_file(), if the file corresponding to the next
entry in the score table can not be opened, delete this entry from
the score table and try the next entry.

If the score table becomes empty, send NO_ADMISSIBLE_FILES back to para_server
and make recv_afs_result() switch to stop mode in this case.

Add additional sanity checks to osl_get_rank().

Check whether the riven rank is greater than zero and not greater than
the number of rows in the table.

Fix osl_del_row().

Only free volatile columns of the deleted row if OSL_DONT_FREE
is not set.