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.

open_and_update_audio_file(): Interchange two arguments.

And add documentation of the "score" parameter.

REQUIREMENTS: Kill an incomplete sentence, add git.

Introduce vss_stopped().

Fix com_pause().

If the vss status is stopped, we must not switch to pause status.

fade.c: Fix client_cmd()

We have to wait for children to terminate because otherwise the
client commands may be executed in random order.

Since no caller checked the return value of client_cmd(), make it
abort on errors and change the return value to void.

Also, each caller only passed one string, so make it take only a
char * pointer.

stat.c: Cosmetics.

make_empty_status_items(): Add spaces after status item identifiers.

Identifiers without spaces confuse para_gui.

Fix tv_divide().

The old code contained an integer overflow which caused the mp3 audio
format handler to compute the chunk of large mp3 files incorrectly.

Moreover, the code was more complicated than necessary. So replace it
by a simple calculation which just transforms the given struct timeval
into its number of microseconds, does the division and transforms
the quotient back to a struct timeval.

Thanks to Gerrit Renker for pointing out the problem.

Fix com_nomore.

Avoid some gcc warnings.

Avoid empty lines in the status output.

Introduce make_empty_status_items().

At the end of the stream we have to reset the status items
because audiod would start a new receiver otherwise.

Add more status items.

Use FOR_EACH_STATUS_ITEM everywhere.

handle status items via autoconf.

Store chunk info in the afhi column rather than in the chunk column.

As the afhi column is mmapped, the chunk info is now available to
the ls command. The disk-storage column "chunks" only contains the
chunk table which allows to get rid of load_chunk_info() and
save_chunk_info().

aft.c: Fix a possible memory leak.

README, INSTALL: Small improvements.

More status item renames.

SI_LENGTH_MIN -> SI_BITRATE
SI_LENGTH_SEC -> SI_FREQUENCY

SI_ATTRIBUTES -> SI_ATTRIBUTES_BITMAP
SI_UPTIME -> SI_ATTRIBUTES_TXT

Don't report server uptime as status item.

This belongs to the server info.

Rename audio_info status items and get rid of duplicate afh info.

SI_AUDIO_INFO1 -> SI_AUDIO_FILE_INFO
SI_AUDIO_INFO2 -> SI_TAG_INFO1
SI_AUDIO_INFO3 -> SI_TAG_INFO2

The mp3 and the ogg afh stored bits of information in the info
string. Only store there what is not already contained in afh_info.

Interchange two lines in load_afhi().

Just to keep the oder consistent with safe_afhi().

Make the status item list globally visible.

audiod.c: Add missing space.

score.c: Use the new OSL_DONT_FREE flag for the aft column.

The score tables stores in one of its columns a pointer to
a row in the audio file table. This means for each admissible
file we're allocating space for a void* pointer to store the
aft pointer.

It's simpler and probably faster to store the afs pointer directly,
but of course that pointer must not be freed by the osl.

osl: Introduce volatile columns that need not be freed.

If the new OSL_DONT_FREE flag is given, osl_close_table()
will not try to free such objects.

Fix duplicate afs event add.

This bug hits if a request to update an existing audio file
is made. The old code caused an AFHI_CHANGE event and then
did "goto out;" which raises nother event (AUDIO_FILE_ADD).

Fix it by moving the event-raising code some lines up.

Always print a space after each status item identifier.

Get rid of afd->afsi and rename afs_status_info to verbose_ls_output.

Kill afd->current_play_mode.

It's unused.

kill afd->score.

This is also contained in afd->afs_status_info.

Get rid of afd->path.

All that info is contained in afd->afs_status_info.

Rename length status item to seconds_total and print it only once.

Fix number of whitespaces in duration status item.

Always write image_name and lyrics_name status output.

If there is no image_name, print "(none)". Same with
lyrics.

Split lyrics and image lines in stat output.

It's better to have the id and the name of the corresponding
blob as two different status items.