Doxify send_common.c.

Sender code consolidation, on/off commands for the dccp sender.

This patch moves a bunch of code from the http sender to
send_common.c to make it available also for the dccp sender.

To this aim a new structure sender_status is introduced which
holds the configuration and the current status of the http/dccp
sender. Most of the new functions in send_common.c take a pointer
to such a structure.

This allows to implement the off/on/allow/deny/info commands
as a one-liner for both the dccp and the http sender.

The new server config option dccp_max_clients allows to restrict the
number of simultaneous connections to the dccp sender.

acl.c: Trivial whitespace cleanups.

user_list: Fix doxygen \param.

init_random_seed(): Use para_open() to open /dev/urandom.

Clean up this function while at it.

Doxyfile: Remove HAVE_MYSQL define.

Remove duplicate error code.

We had AFS_SHORT_READ and SHORT_AFS_READ.

Remove para_mkstemp().

It's unused.

NEWS update.

Speed up audiod_pre_select.

Catch the common case early in the function to minimize the
number of checks for that case.

compress.c: Get rid of min_gain.

Always use no amplification, i.e. 1 << pcd->conf->inertia_arg as
the minimal gain.

string.c: Documentation cleanups.

Get rid of E_MKSTEMP and E_FCHMOD.

Use the system errno instead.

Speed up open_current_receiver().

To my surprise, this function showed up in the profiling info.
This is due to the fact that the old code was parsing the
FORMAT status item in audiod_pre_select() which gets called
on each iteration of the sheduler. However, this status item
gets only modified on audio file change.

So parse the status item string only once, when we receive it,
and store the current audio format number in the new field of
struct status_task.

compress_filter.ggo: Decrease block size from 16 to 15.

This means twice as many volume adjustments.

compress.c: Simplify volume adjusting code.

There's no need to distinguish the cases sample > 0 and
sample < 0. Work with its absolute value instead and get
rid of some similar code in the hot path.

Also print a log message in case clipping occurs.

compress.c: Cosmetics.

user_list.c: Minor documentation impreovements.

populate_user_list(): Do not abort if the rsa key of a user is unavailable.

Skip this line of the user list file instead.

user_list.c: Simplify populate_user_list().

user_list.c: Fix documentation of populate_user_list().

afs.c: Fix fd number in log output.

Kill unused error code WRITE_OK.

osl_core.h: Replace static inline by _static_inline_.

Remove gui_common.[ch].

As the only gui in paraslash-0.3.x is the curses-based para_gui
there is no point in having common gui functions.

More trivial cleanups.

error.h: Kill two unused alsa error codes.

Trivial documentation cleanups.

Fix a typo on the Documentation web page.

Print codename in version info.

net.c: Fix error messages in makesock().

alsa_write.c: Kill deprecated snd_pcm_sw_params_{get,set}_xfer_align.

Fixes

warning: 'snd_pcm_sw_params_get_xfer_align' is deprecated

for alsa-lib-1.0.16.

Implement access control for the dccp sender.

Introduce new acl functions to avoid duplicate code
in the http/dccp sender.

http_send.c: Minor post_select() cleanups.

Defer allocating memory for the client until
we will accept it.

http_send.c: Kill unused variable "i".

http_send.c: Fix a bug in http_post_select().

If the accept() on the listen_fd fails, we don't have a valid
file descriptor but tried to close it anyway. Even worse, the
old code dereferenced and then freed the "sc" pointer which
isn't inititialized at that point.

dccp_send.c: Simplify dccp_info().

NEWS update.

dccp_senc.d: No need to use static variable in dccp_info().

Make the http sender use the generic sender code.

Intoduce send_common.c and use it from the dccp sender.

This new file contains code that was identical for the
dccp and the http sender. A subsequent patch will convert
the http sender to also use the shared code in send_common.c

send.h: Cosmetic cleanups.

http_send.c: Kill check_r member from struct http_client.

It was set to one if and only if the client is in state
HTTP_CONNECTED.

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.