kill file parameter from get_file_info of struct audio format handler

Now that all audio format handlers use mmap(), this parameter is unused.

mp3_afh.c: Switch to memory mapped input file

aac_afh: use the file mapping rather than read()

This simplifies this audio format handler quite a bit.

Also fix a bug in error.h: E_AAC_OVERRUN is an error of the aac decoder,
so move the corresponding define to where it belongs.

vss.c: Use the file mapping instead of fseek()/read()

first draft of the mmap patch series

The purpose of these changes to para_server is to not call
read()/fseek() in the audio format handlers and in vss.c but to use
mmap() instead which should result in less code and should also be
faster and less error-prone.

This first patch implements the basic infrastructure for mmap but
keeps the file pointer so that changes may be merged step by step and
changes the ogg vorbis audio format handler to use the new mapping
of the audio file.

Subsequent patches will change the other audio format handlers and
the virtual streaming system to also use mmap instead of read()/fseek().

vss.c: rename get_song() to vss_get_audio_file()

comment out some noisy debug messages

small documentation cleanup

command.c kill unused declaration of user_list

ogg_afh.c: make two functions static

fix doxygen comment of the init functions of all audio format handlers.

mp3_afh.c: Kill global variable num_chunks and seconds.

num_chunks has the same meaning as chunks_total in struct audio format.
Similarly, mp3.seconds has the same meaning as afi->seconds. So use these.

kill close_audio_file handler from struct audio format.

All audio format handlers just did  fclose(), so do this from vss.c
and kill the handler. This also allows to get rid of the global
infile variable in all audio format handlers.

kill global variable af from all audio format handlers

Also rename the argument of the init function from "p" to "afh".

aac_afh.c: kill global variables inbuf and inbuf_len

Pass them to read_chunk_table() instead.

aac_afh.c: free the inbuffer already in aac_get_file_info()

No need to wait for this until the audio file gets closed. This
patch also fixes two occurences of a bug where the length of the
input buffer was not set correctly.

aac_afh.c: Kill global variable af

It's only used in aac_afh_init.

aac_afh.c: Kill global variable num_chunks

It's already stored in afi->chunks_total

aac_afh.c: Kill global variable inbuf_size

it's constant and equals AAC_INBUF_SIZE.

introduce struct audio_format_info

This further simplifies the audio format handlers. The chunk_table
and the header info is now contained in that struct which allows
to move the cleanup on eof from the audio format handlers to the
virtual streaming system.

ogg_afh.c: Kill tunetable()

A rather crappy piece of code that is (1) broken and (2) unneccessary.

ogg_afh.c: Switch to ogg vorbis callbacks

This allows to call ov_clear() already during ogg_get_file_info() which
in turn simplifies ogg_close_audio_file().

This is a preparation for moving the close function of the audio
format handlers to the virtual streaming system as they all do
essentially the same now.

READMY.mysql: replace --enable-mysql-headers by --with-mysql-headers

The configure option was changed some time ago, but the README file
still uses the old syntax.

audiod: avoid busy loop on startup

The restart barrier of the status task must be also set in case
client_open() failed. This avoids spamming the log file.

ogg_afh.c: Fix format string compiler warning

mysql_selector.c: mark mysql_macro_list[] const

mysql.c: fix -Wshadow warnings

Makefile.in: use CPPFLAGS also for target mysql.o

aac_afh.c: Make "handle" a local variable

Also, clean up the definition of static variables.

mp3_afh.c: Mark frequencies, bitrate and frame_size_index arrays const

mp3_afh.c: Kill unused mp3.filename

ogg_afh.c: Kill unneccessary inclusion of fd.h

ogg_afh.c: Kill global variable num_chunks

The code is more readable if this is returned by ogg_compute_chunk_table()
and passed where neccessary.

ogg_afh.c: make vi_sampling_rate, vi_bitrate local

ogg_afh.c: Kill unused vi_bitrate_nominal

ogg_afh.c: get rid of global variable max_chunk_len

It's only used in ogg_compute_chunk_table(). So make it a local variable.

simplify ogg_compute_chunk_table()

it rewinds the input file on entry and on exit. Do this in the caller instead
which allows to get rid of the file parameter in ogg_compute_chunk_table().

ogg_afh.c: Kill global variable infile

Pass it where neccessary instead.

ogg_afh.c: Kill superflous vi_channels

ogg_afh.c: Kill unused inbuf and inbuf_size

mp3_afh.c: Kill unused inbuf and inbuf_size;

simplify vss_read_chunk()

All its parameters are known globally within vss.c, so omit them.
Also, return an int instead of the pointer to the inbuffer which is
also known to the caller.

This patch also introduces para_fseek() in fd.c which is used by
mp3.c and vss.c and changes the mp3 audio format handler so that it
uses the new function and always check the return value.

kill the read_chunk handler of struct audio_format handler

chunks are now read by vss_read_chunk(), i.e. the same function is
used for all audio formats.

make get_file_info() return the chunk table

This is a preparation for the moving the read_chunk handler to
the vss layer.

vss.c: Kill check for get_file_info == NULL

This can never happen.

kill the reposition handler in struct audio_format_handler

All three audio format handlers provided only a dummy function. Kill
these as well.

unify all audio format handlers

This changes the ogg vorbis audio format handler such that its read_chunk and
reposition_stream functions coincide with those of the other audio format
handlers.

ogg_afh.c: Rename oggbuf_len to inbuf_size

so that it matches the naming of the other audio format handlers

ogg_afh.c: Rename oggbuf to inbuf

To make naming consistent with the aac and mp3 audio format handler.

unify aac and mp3 audio format handlers

This makes the mp3 audio format handler use a chunk table just like
m4a and ogg vorbis already do. Moreover, the aac and mp3 audio format
handlers now have identical read_chunk and reposition_stream handlers.

Next step is to tweak the ogg vorbis audio format handler so that
it also fits into this scheme. In the end, this allows to move
read_chunk() and reposition_stream() from the audio format handlers
one layer up to the virtual streaming system and to kill the function
pointers in struct audio_format.

The advantage of this change that it avoids duplicated/similar code, and
that new audio format handlers can added easier.

string.c para_realloc(): report requested size on realloc failures

ogg_afh.c: chunk_table is an array of type size_t

simplify ogg_compute_header_len()

dccp_recv.c, stdout.h: change year in GPL header to 2007

These two files have changed in 2007, so..

rename DB_READ, DB_WRITE to AFS_READ, AFS_WRITE

Merge /fml/ag-raetsch/home/maan/scm/paraslash_meins/paraslash/

FEATURES update

osx_write.c: Kill sleep() on buffer underruns

This was a leftover from the mpg123 patch for mac os which does
more harm than good. It could cause para_write/para_audiod to
block under certain circumstances.

NEWS update

stdout.h: Kill unused bufsize

filter.c: Register tasks in proper order

The order is important as it determines the order in which
the pre_select functions are going to be called by the scheduler.

In the previous code, the scheduler called stdin->pre_select _before_
filter_chain->pre_select() which gave rise to the following scenario
(note that there's no filter_chain->post_select()):

stdin->pre_select(): set stdin for reading
filter_chain->pre_select(): nothing to do
stdin->post_select(): read buffer
stdin->pre_select(): do _not_ set stdin for reading as stdin buffer is full

For para_filter, after the last there are no fds set which causes the
scheduler to use the default timeout of 1s. This had the effect that
decoding of aac files started only after a few seconds.

With the roles of interchanged tasks the scenario is as follows:

filter_chain->pre_select(): nothing to do
stdin->pre_select(): set stdin for reading
stdin->post_select(): read buffer
filter_chain->pre_select(): convert buffer
stdin->pre_select(): set stdin _again_ as stdin buffer is not full

Alternative fix might be to use filter_pre_select() also for post_select.

rename db.c to afs.c and db.h to afs.h

db.c find_audio_files(): use int rather than char for cwd_fd

Merge /home/maan/scm/paraslash_fml/paraslash

simplify get_atts()

remove pointless ret variable and clarify comment.

fix typo in dccp_writer

Again, a serious typo that causes the dccp writer to not error
out under certain circumstances.

simplify setup_default_wng()

The default writer is known at compile time, so replace the code
that computes it at runtime by a #define in configure.ac.

http_recv.c: Fix typo in http_recv_open()

Quite serious as it causes the http receiver not to error
out if get_host_info() fails.

new codename, reset version to git

the paraslash-0.2.15 release tarball

paraslash 0.2.15

Makefile.in: fix target web/misc

It depends on @PACKAGE_TARNAME@-@PACKAGE_VERSION@.tar.bz2 rather than on
paraslash-git.tar.bz2.

Makefile.in: only build and install man pages for supported binaries

This makes it possible again to install from the git repo even if not
all binaries can be build, e.g. Mac OS which always lacks para_fade
(reverts a858de7cf9437cf9a3569eb4bc297c5b0e768994).

Also, don't refuse to install and don't complain if $VARDIR
(/var/paraslash) can not be created, as this is non-fatal.

configure.ac: Detect os type, add -Wl,-bind_at_load to oggvorbis ld flags on Darwin.

This gets rid of zillions of warnings of the kind

ld: warning suggest use of -bind_at_load, as lazy binding may result in errors or different symbols being used

on Mac OS X (10.4.6)

rc4.h: add doxygen file comment

fix client args info memory leak

filter_chain.c, recv_common.c: Fix another memory leak

There was still one leak left: argv[0] gets strdupped, but never freed.

dccp_recv.c: Fix memory leak in dccp_recv_open()

Missing freeaddrinfo().

audiod.c: fix memory leak in audiod_status_dump()

missing free() for decoder flags. Quite serious: We leaked some bytes on each
status change.

filter_chain(): Fix memory leak in parse_filter_args

Same bug as in parse_receiver_args(): argv must be freed (see previous patch).

recv_common: fix memory leak and invalid free in error path

As split_args() returns pointers that point within the given string,
it is wrong to free the individual pieces. However, the array of
substrings must be freed. parse_receiver_args() got both of these
wrong :(

cosmetics

crypt.c: fix memory leak in para_decrypt_buffer()

The client side had more or less the same memory leak as the server side:
The rsa structure wasn't freed.

fix a memory leak in the RSA key handling

RSA keys must be freed by using openssl's RSA_free() rather than the
usual free(). This leak turns out to be more serious as the amount
of leaked memory increased by about 300 bytes every time para_server
reread the user list (i.e. the hup command was executed or para_server
received SIGHUP).

user_list.c: Fix a memory leak

get_rsa_key() already allocates space for the rsa key pointer. Moreover,
u->perms was set to zero twice. Kill one instance.

simplify lookup_user()

It's easier to just pass the name of the user to be looked up. This
fixes a memory leak btw. because u.name was strdupped and never freed.
That leak wasn't serious though as is happened in the child process.

list.h, ortp.h: Add missing documentation

mysql_selector.c: Fix mbox command

An additional space crept in.

para.h fix a typo

README: minor cleanups

add para.h to the files used by doxygen

move gui-related symbols from para.h to new gui.h

more doxygen docu, some small cleanups

para.h: Add documentation of PARA_MAX, PARA_MIN, PARA_ABS

kill STRINGSIZE macro

It's kind of silly. Just open code the desired value.

playlist_selector.c, random_selector.c: make update_audio_file() static

command_util.sh: Make it read the function prototype from the source file

This is much cleaner as the commands differ by gcc attribute usage,
e.g. __a_unused.  But it also means that in the source code

(a) the opening bracket for the function body must not be on
the same line (com_hist got this wrong),

(b) for all command handlers, the parameters must be called
fd, argc and argv,

(c) If a command handler uses __noreturn, that attribute must
be the first word, see audiod's com_term().

So change the few occasions where this was not the case.

compress.c, wav.c: add documentation

Merge /home/maan/scm/paraslash_fml/paraslash

add more missing documentation