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

don't set the initial_delay_barrier if time diff is unreliable

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

Conflicts:

Makefile.in

audiod: Add missing documentation

alsa_write.c: Add missing documentation

audiod.c: Kill unused variable pcd

audiod.h: Kill unused struct audiod_task

The audiod task is an ordinary struct task.

fade.c: [trivial] whitespace cleanup

sdl_gui.ggo: Kill duplicate authorship info

This info is already provided by the HANDLE_VERSION_FLAG macro

gui.c: make command pipe a nonblocking fd

audiod.c: [trivial] move close_stat_pipe() closer to its callers

audiod: kill handle_signal()

It was only called from singal_event_handler(), so move the code there.
Also, remove the redundant check for t->ret != -E_SIGNAL_CAUGHT which
can never happen.

audiod: fix mode cycling

This was quite buggy all the time, but wasn't noticed due to the typo bug
which was fixed in 1b46ae4193cddb7d589b0f2f7d8158b84f7e9f6d.

Now, the status task no longer has an event handler as both its pre_select
and its post_select functions always succeed and the status task is supposed
to never terminate. Unlike the client task whose event handler now does all
the cleanup if the connection to para_server breaks down for whatever reason.

client_common.c: invalidate the private client data pointer on errors.

Not really neccessary, but clean.

use own version handler for all commands.

This makes the man pages generated by help2man look much nicer.

client: combine client_open() and client_parse_config()

Both users (client.c and audiod.c) called client_open() right
after calling client_parse_config(), so call client_open() from
client_parse_config() and make client_open() static.

While we're at it, rename client_open() to client_connect() as this
is what the function does. Also rename client_parse_config() to
client_open() and change audiod.c and client.c to call client_open()
instead of client_parse_config().

This patch fixes a segfault in para_client was it was run without
specifying a command. In this case, the private client data struct
was used by para_log after being freed -- Ouch. Fortunately, this
bug didn't affect para_audiod as it never calls these functions with
no command.

Another bug due to a typo in client_close() is also fixed. This code
needs closer investigation and lacks documentation btw.

para.h: Kill unused LICESE macro

Merge ../paraslash_meins/paraslash

fix typo in FEATURES

string.c: Add missing \return in documentation

new feature list, merge of README and FEATURES

README and FEATURES contained quite some overlap, so merge them
into a single document (README) and change FEATURES to contain only
a bullet list.

remove old server and audiod screenshots

update para_server and para_audiod "screenshots"

use current versions of the startup log messages.

overview.sk: Update to 0.2.15

audiod does not fork and exec anymore for quite some time now.

move s_a_r() and s_a_r_list() from string.c to mysql_selector.c

And make them static as they are only used in mysql_selector.c.

This reduces the stripped code size of all executables (except
para_server of course) by about 1K.

string.c: Documentation update

- Add documentation of return value to all functions.
- Replace \a by \p which looks nicer in the html output.
- Clarify s_a_r documentation

Merge ../paraslash_meins/paraslash

make man dirs before calling help2man and friends

Also, make sure we call help2man on the executables in the
current dir.

Merge ../paraslash.fml/paraslash

clarify documentation of mysql_selector_init()

mysql_selector.c: free(NULL) is OK.

No need to check for NULL. Moreover, stream, meta, atts and dir are
set in any case so kill the NULL initialization.

Makefile.in: minor cleanups

move target para_recv down and kill unneccessary -o option for the
mysql_selector.o target.

check_writer_arg(): change writer arg to const char *

make osx_write_parse_config() static and change arg to const char*

server.ggo: make the documentation match the parameter name

The parameter is called 'name', but in the documentation it was refered to
as 'username'. Similar with 'groupname'.

file_write.c: make file_write_parse_config() static and change arg to const char*

Same comment as for the previous patch to alsa_write.c applies.

alsa_write.c: make alsa_parse_config() static and change parameter to const

alsa_cmdline_parser_string() which is generated by gengetopt takes a const
char *.

change email address from tu-darmstadt to systemlinux.org

alsa_write.c: fix compiler warning on x86_64.

Simple %d -> %zd format string mismatch.

Merge ../paraslash.meins/paraslash

NEWS update

command.c: remove an unused variable

command.c: add documentation of handle_connect()

make another function of command.c static

command.c: make two functions static

rename "synopsis" to "usage".

command.c help: nicify help output

server.cmd: update hup help text

vss_status_tohuman(): change permission status flags

d (dbtool) -> a (audio file selector)
a (audio file sender) -> v (virtual streaming system)

add links to all man pages to documentation web site

Makefile.in: Always make man pages of each binary, supported or not.

Makefile.in: use man pages generated by command_util.sh for tarball

Makefile.in: More trivial cleanups

Makefile.in: Kill unused TGZ_PREFIX

Makefile.in: trivial cleanups

Makefile.in: Add target man

Makefile.in: Kill Changelog target

It's unused for ages.

Makefile.in: add targets for man page generation

audiod.cmd: add SN tag

simplify com_man() of command_util.sh

add SN tag to *_selector.cmd

server.cmd: add SN tag and fix some typos

command_util.sh: improve man page generation

Read the global tag SN (section name) and output simple *roff text.

command_util.sh: Make sure the "handler" pointer is NULL if a line handler exists.

This becomes neccessary because the array of commands is no longer static,
i.e. the standard does not guarantee that uninitialized members are zeroed out.

nuke the AN (array name) header field

The previous patches changed that name to ${file_name%command_list}cmds,
so it is a derived string that does not need to be specified any more.

rename audiod command array from cmds to audiod_cmds

rename mysql selector command array from cmds to mysql_selector_cmds

rename server command array from cmd_struct to server_cmds

switch to a single rule for all command lists

command_util.sh: rename proto to header and array to c_file

move header include info from Makefile.in to *.cmd

make audiod use command_util.sh

make the playlist selector use command_util.sh

Also, add random_selector.cmd which should have been includes already in the
previous patch.

make the random selector use command_util.sh

make the mysql_selector use command_util.sh

para_server: switch to generated command list