http_send.c: Never check the client fd for writability.

The fd is set to nonblocking mode, so writes will never
block but fail.

http_send.c: Kill HTTP_SENT_OK_MSG.

We might as well go directly to the HTTP_STREAMING state.

{http,dccp}_send.c: Kill unused variable "self".

http_send.c: Get rid of write_ok().

Just use http_write() instead of write().

dccp_send.c: Fix a header sending bug.

If the dccp sender is asked to send out an empty chunk, it
returns immediately without doing anything. However, we
might want to send the audio file header or the queued chunks
in this case. So always loop over all connected clients to see
if we can send something.

Moreover, if we are not listending on the dccp port, there can
not be any dccp clients, so the loop over all clients will be
an empty loop. Thus we may skip the initial check for an early
return completely.

http_send.c: Get rid of HTTP_READY_TO_STREAM.

The only reason for this state was to know whether we
already sent the audio file header. Store this info in
the new header_sent member of struct http_client.

This simplifies the code a bit and makes it more similar
to the dccp sending code.

[TRIVIAL] dccp_send.c: Decrease scope of header_len.

http_send.c: Introduce http_write.

This allows to get rid of write_ok() and makes
send_queued_chunks() identical to its counterpart
in the dccp sender.

sendall(): Replace send() by write().

With zero flags parameter, send() is  equivalent  to  write(2).

dccp_senc.c: Kill E_DCCP_WRITE.

Use ERRNO_TO_PARA_ERROR() instead.

[DCCP]: Use 8000 as port.

From: Gerrit Renker

As TCP and DCCP are different transport protocols, it is possible
to let both the dccp and the http sender listen on port 8000.

Be more careful on command exit.

If the callback returned zero, there's nothing to do.
If it returned positive, we have to send out the result
buffer and must free it afterwards. Sending the buffer may
well lead to an error condition, so always check the return
value of the corresponding call to send_buffer(). Finally,
it the command handler returned a negative value, we have
to send an error message to the client which might give rise
to another error (if sending the errror message failed). In this
case, ignore the send errror and return the previous error value.

com_sender(): Check return value of send_buffer().

Kill two unnecessary return statements.

dccp_send.c: Reformat comment.

mood.c: Kill unrechable return statement.

Always check the return value of mark_fd_nonblocking().

dccp_recv.c: Fix a fd leak.

If shutdown(fd, SHUT_WR) fails, we miss to close the
open fd returned by makesock().

dccp_recv.c: Reformat comment.

dccp_send.c: Check the return value of mark_fd_nonblocking() also for the client fd.

dccp_send: Fix a fd leak.

If shutdown(fd, SHUT_RD) fails, we miss to close the fd.

dccp_senc.c: Check return value of mark_fd_nonblocking().

Add __must_check annotation for mark_fd_nonblocking() and mark_fd_blocking().

gcc-compat.h: Fix __must_check.

It didn't work for gcc-4.

Simplify http_open().

No need to shutdown the clients if listen failed: There
can not possibly exist any connected clients at this point.

Kill "port" parameter of http_open().

It's always conf.http_port_arg.

http_send.c: Rename server_fd to listen_fd.

To make it consistent with the dccp sender.

Get rid of sender->status.

DRY: For the http and the dccp sender, the status is given by
the value of the listening fd. For the ortp sender, introduce
the new static int sender_status.

Remove unused ortp_pre_select().

aft.c: Fix a memory leak in open_and_update_audio_file().

If make_status_items() fails, we return an error but
do not free the chunk table buffer.

Also get rid of a pointless "goto err".

vss.c: Trivial whitespace cleanup.

vcc.c: Fix a memory leak in recv_afs_result().

This function contained a code path that allocated
the buffer for the chunk table and then returned an error
(due to a failing fstat()) without freeing the chunk table
buffer.

net.c: Fix a memory leak in makesock().

"port" was allocated dynamically but never freed. Fix it by
using a static buffer instead. This buffer can not overflow as
"port_number" is an unsigned short, so its decimal representation
consists of at most 5 digits.

send.h: Simplify sender command enum.

Also add GPL header while we're at it.

server.c: Initialize vss _after_ afs.

This way, afs doesn't inherit the open fds of the senders. This fixes

sender http off

Without this patch, para_audiod wouldn't notice that the http sender
has closed its listening fd because afs still has this fd open.

http_send.c: Rename open_tcp_port() to http_open().

ringbuffer improvements.

Introduce struct ringbuffer. This allows to get rid of the void* pointers
of the ringbuffer implementation.

ringbuffer.h: Add GPL header.

Merge commit 'remotes/meins/master'

acl.h: Add GPL header and doxygen file comment.

Add pointer to documentation of the new acl functions.

doxify acl.c.

Rename acl functions.

Prefix public functions with acl.

Move acl functions to separate file acl.c.

http_send: Make add_perm_list_entry() and del_perm_list_entry() generic.

http_send.c: Add get_acl_contents() helper.

http_send.c: Make host_in_acl() generic.

Add an additional parameter for the list head that defines the acl.

http_send.c: Only pass the fd to host_in_access_perm_list().

No need to pass a pointer to struct http_client as only its ->fd member
is used. Rename this function to host_in_acl while we're at it.

http_send.c: Make init_access_control_list() generic.

http_send: Rename access_perm_list to http_acl.

new codename, reset version to git

the paraslash-0.3.0 release tarball

paraslash 0.3.0

make all: Also build man pages.

Without this patch, man pages are built during make install. It's more
natural to build them at make time.

make clean: Remove man dir.

INSTALL: Replace "para" by "para_client".

The para alias is only introduced in README.afs. Reported by
Johannes Wörner.

Remove man pages from tarball.

Since they are re-created at make install time anyway and help2man
is now considered a required tool for building paraslash, there is
no reason to ship the pre-generated man pages with the tarball. This
reduces the size of the tarball by about 2k.

Thanks to Johannes Wörner for pointing out this inconsistency.

REQUIREMENTS: Move help2man to the list of required software.

The man pages depend on the executables, i.e. make rebuilds a man page
whenever the corresponding executable changes. Therefore help2man is
required even when installing from tarballs.

REQUIREMENTS: Link to the english version of the linphone pages.

fade.c: Don't include curses.h.

para_fade does not depend on curses. Including curses.h leads to an
error at compile time on linux if the curses headers are not installed.

Thanks to Johannes Wörner for reporting.

NEWS: Fix a typo.

Change year in Copyright comment from 2007 to 2008.

configure.ac: Improve alsa checks.

On non-Linux systems, do not check for alsa, and do not print the
warning message stating that alsa could not be found.

Also, skip further alsa-checks on the first failure.

Merge commit 'remotes/yangtse/master'

Fix gcc-warning on Solaris.

net.c:334: warning: missing braces around initializer

Merge commit 'remotes/yangtse/master'

Fix configure on Solaris.

Without this patch, configure fails on Solaris because it doesn't find
inet_ntoa() and socket(). These are in libnsl and libsocket
respectively, but AC_CHECK_FUNCS() doesn't know this.

gui.c: Change para command to "para_client --".

Reformat gui.ggo.

gui.ggo: Kill --auto_decode.

This option is unused for ages..

com_ls(): Always print the basename as the subject if -lv was given.

Fix a doxygen warning.

Cosmetics.

Simplify para_basename().

No need to allocate additional memory. This saves a malloc/free
in the hot path of com_ls(), the only caller. Also take care for
the case basename == NULL.

Update bash_completion to v0.3 commands.

It doesn't care about options, and it's slow.  But it's better than
what was there before.

Turn PARA_STRERROR into an inline function.

Add documentation and an assertion for (num > 0).

Fix typo in the help text of the ls command.

com_lsatt(): Fix error message.

We must not use PARA_STRERROR(-ret) if ret is zero.

Makefile.in: Fix OS detection.

The uname_s variable was never defined. Define it and rename SYSTEM
to UNAME_RS while we are at it.

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.