paraslash.git
17 months agoparaslash 0.6.2 v0.6.2
Andre Noll [Sat, 30 Jun 2018 08:54:34 +0000 (10:54 +0200)]
paraslash 0.6.2

17 months agoMerge branch 'refs/heads/t/aft_fixes'
Andre Noll [Mon, 25 Jun 2018 18:12:27 +0000 (20:12 +0200)]
Merge branch 'refs/heads/t/aft_fixes'

Two fixes related to the audio file table.

Cooking for three weeks.

* refs/heads/t/aft_fixes:
  aft: Remember current audio file after SIGHUP.
  aft: Honor AUDIO_FILE_REMOVE events.
  server: Update status items on file renames.

17 months agoMerge branch 'refs/heads/t/long-help'
Andre Noll [Sun, 10 Jun 2018 16:59:36 +0000 (18:59 +0200)]
Merge branch 'refs/heads/t/long-help'

This series introduces lsu.c and lsu.h which contain helpers related to
the lopsub library. These helpers are designed to be shared between
the executables. The series starts by implementing a generic help
command for lopsub suites and converts para_server. Subsequent patches
convert audiod and para_play.

The second part of the series adds another lopsub related helper which
merges command line options and config file options. Each executable
is modified to make use of the new helper, getting rid of quite some
code duplication.

The conflict resolution for server.c has been tested for a while.

Cooking for five weeks.

* refs/heads/t/long-help:
  play: Use lsu_merge_config_file_options().
  mixer: Use lsu_merge_config_file_options().
  gui: Use lsu_merge_config_file_options().
  filter: Use lsu_merge_config_file_options().
  audioc: Use lsu_merge_config_file_options().
  audiod: Use lsu_merge_config_file_options().
  client: Use lsu_merge_config_file_options().
  lsu: Add helper to merge config file options, convert server.
  Trivial: Rename completion_result variables.
  play: Implement help --long.
  audiod: Implement help --long.
  Introduce lsu.{c,h}, implement help --long for para_server.

18 months agoMerge branch 'refs/heads/t/listen-address'
Andre Noll [Mon, 28 May 2018 16:29:06 +0000 (18:29 +0200)]
Merge branch 'refs/heads/t/listen-address'

A short series which adds options for para_server that allow the user
to specify the listening addresses of the passive sockets.

The resolution of the conflicts in server.c and send_common.c have
been well tested and were cooking for a couple of weeks.

* refs/heads/t/listen-address:
  server: Add --http-listen-address and --dccp-listen-address.
  server: Implement --listen-address for control service.

18 months agoMerge branch 'refs/heads/t/clean_server_exit'
Andre Noll [Sun, 20 May 2018 09:26:26 +0000 (11:26 +0200)]
Merge branch 'refs/heads/t/clean_server_exit'

This series removes many memory leaks of para_server by refactoring
the shutdown and signal handling code. Most of the leaks happen only
at shutdown and are hence harmless. But it is still good to plug
the leaks because this puts more focus on real memory leaks in the
valgrind output.

The merge conflicted rather badly due to the changes introduced with
the crypt branch that was merged last week. The resolution has been
thoroughly tested, though.

* refs/heads/t/clean_server_exit: (32 commits)
  command.c: Document return value of handle_connect().
  user_list: Make list head static.
  afs: Allow database switching on sighup.
  afs: Free current mood or playlist on exit.
  afs: Free status items on exit.
  afs: Shutdown signals on exit.
  server: Free parse result also in afs.
  afs: Deplete user list at startup.
  server: Free audio file header on exit.
  sender: Deplete ACLs on exit.
  Remove some unused includes from {dccp,http}_send.c.
  server: Make argument of user_list_init() constant.
  server: Deplete user list on exit.
  server: Combine user_list_init() and populate().
  server: Move para_fgets() to user_list.c.
  server: Initialize user list at compile time.
  server: Rename functions related to user lists.
  server: Constify return value of lookup_user().
  server: Let stat command handler perform cleanup on signals.
  server: Have afs process close the current mood on exit().
  ...

18 months agoMerge branch 'refs/heads/t/crypt'
Andre Noll [Sun, 13 May 2018 12:03:20 +0000 (14:03 +0200)]
Merge branch 'refs/heads/t/crypt'

A couple of simple patches which clean up the openssl and gcrypt crypto
backends. The non-trivial parts are the get_public_key() unification,
the memory leak fixes for openssl and the activation of the secmem
pool for gcrypt.

* refs/heads/t/crypt:
  gcrypt: Allocate a secmem pool at startup.
  crypt: Introduce crypt_shutdown().
  crypt: Rename init_random_seed_or_die() -> crypt_init().
  gcrypt: Use ROUND_DOWN() macro.
  crypt: Rename RSA functions.
  crypt: Deduplicate get_public_key().
  crypt.h: Remove two unused constants.
  Rename crypt.c -> openssl.c.

19 months agoMerge branch 'refs/heads/t/doc'
Andre Noll [Sun, 6 May 2018 08:54:47 +0000 (10:54 +0200)]
Merge branch 'refs/heads/t/doc'

Some minor documentation improvements. Cooking for four months.

* refs/heads/t/doc:
  Fix bad grammar "allows <infinitive>".
  doc: Move filter examples to man page.
  doc: Move sender examples to manual page.

19 months agoaft: Remember current audio file after SIGHUP.
Andre Noll [Mon, 16 Apr 2018 18:41:39 +0000 (20:41 +0200)]
aft: Remember current audio file after SIGHUP.

The current_aft_row pointer becomes stale when the osl tables are
re-opened because SIGHUP has been received. If the current audio file
is renamed, or its metadate (afs info or afh info) are modified,
the table event dispatcher of the audio file table compares the
stale row pointer against the (non-stale) row pointer of the audio
file that has been obtained by path lookup. The two pointers never
compare equal under normal circumstances due to ASLR, so the file
modification does not trigger an update of the status items.

This patch fixes this inconvenience by remembering the hash of the
current audio file at table close time. On SIGHUP, after the table
has been re-opened, we lookup the stored hash to replace the stale
row pointer.

19 months agoaft: Honor AUDIO_FILE_REMOVE events.
Andre Noll [Tue, 17 Apr 2018 08:05:07 +0000 (10:05 +0200)]
aft: Honor AUDIO_FILE_REMOVE events.

If the current audio file has been removed, we should invalidate the
current_aft_row pointer.

19 months agoserver: Update status items on file renames.
Andre Noll [Mon, 9 Apr 2018 19:05:56 +0000 (21:05 +0200)]
server: Update status items on file renames.

The audio file table currently ignores rename events. This has the
effect that the status items, hence the display of para_gui, is not
updated when the current file has been renamed. Fix this oversight.

19 months agoserver: Remove incorrect __a_unused in com_ff() and com_jmp().
Andre Noll [Mon, 30 Apr 2018 18:38:18 +0000 (20:38 +0200)]
server: Remove incorrect __a_unused in com_ff() and com_jmp().

These subcommands do use the command context pointer.

19 months agoMerge branch 'maint'
Andre Noll [Sun, 6 May 2018 08:37:42 +0000 (10:37 +0200)]
Merge branch 'maint'

This fix for an ancient bug should also be in master.

* maint:
  server: Fix return value of com_ff().

19 months agoMerge branch 'refs/heads/t/gui-sigwinch'
Andre Noll [Tue, 1 May 2018 16:17:32 +0000 (18:17 +0200)]
Merge branch 'refs/heads/t/gui-sigwinch'

A single patch which removes a long standing issue of para_gui.

Cooking for half a year.

* refs/heads/t/gui-sigwinch:
  gui: Catch SIGWINCH.

19 months agoserver: Fix return value of com_ff(). maint
Andre Noll [Mon, 30 Apr 2018 19:18:01 +0000 (21:18 +0200)]
server: Fix return value of com_ff().

We set the VSS_NEXT flag if the amount of seconds to jump forward
would exceed the end of the current audio file. However, in this case
we fail the command by returning -E_NO_AUDIO_FILE, which is incorrect.

This bug was present since day one of the git repo (2006).

19 months agoserver: Fix brown paper bag bug in generic_com_on().
Andre Noll [Mon, 30 Apr 2018 18:59:59 +0000 (20:59 +0200)]
server: Fix brown paper bag bug in generic_com_on().

Due to this bug an the wrong integer is stored in ->listen_fd,
resulting in errors of the form

Apr 30 20:56:15 (4) (31884) accept_sender_client: Socket operation on non-socket

Fixes: ac9f8fc0b4a20 (send_common: Combine generic_com_on() and
open_sender()) from one month ago.

19 months agoplay: Use lsu_merge_config_file_options().
Andre Noll [Thu, 15 Mar 2018 01:31:10 +0000 (02:31 +0100)]
play: Use lsu_merge_config_file_options().

Besides parsing the command line options and the config file,
parse_config_or_die() also checks the key map arguments
syntactically. This check needs to be kept of course.

play.c already includes lsu.h because it calls lsu_com_help().

19 months agomixer: Use lsu_merge_config_file_options().
Andre Noll [Thu, 15 Mar 2018 01:14:37 +0000 (02:14 +0100)]
mixer: Use lsu_merge_config_file_options().

para_mixer is special in the way the config file is handled because
it is the only executable which allows options for subcommands
to be stored in the config file. Therefore the mixer specific
parse_and_merge_config_file() is called twice. First to merge the
global (supercommand) options, then to merge the subcommand options.
This does not cause any problems regarding the conversion, though.

19 months agogui: Use lsu_merge_config_file_options().
Andre Noll [Thu, 15 Mar 2018 00:36:19 +0000 (01:36 +0100)]
gui: Use lsu_merge_config_file_options().

para_gui supports config file reloading, so parse_config_file_or_die()
needs to set MCF_DONT_FREE to keep a reference to the command line
parse result and MCF_OVERRIDE in the reload case. Also, the code to
check the key maps and to initialize the theme needs to be kept.

19 months agofilter: Use lsu_merge_config_file_options().
Andre Noll [Thu, 15 Mar 2018 00:18:12 +0000 (01:18 +0100)]
filter: Use lsu_merge_config_file_options().

Straight forward conversion. We now set the loglevel only once,
after the config file options have been merged.

19 months agoaudioc: Use lsu_merge_config_file_options().
Andre Noll [Wed, 14 Mar 2018 23:59:56 +0000 (00:59 +0100)]
audioc: Use lsu_merge_config_file_options().

This gets rid of the pointless configfile_exists() and the equally
pointless cmd variable.

19 months agoaudiod: Use lsu_merge_config_file_options().
Andre Noll [Wed, 14 Mar 2018 23:04:06 +0000 (00:04 +0100)]
audiod: Use lsu_merge_config_file_options().

Besides reading and merging the config file, parse_config_or_die()
also creates the uid whitelist. This part has to be retained, but we
can reduce the indentation level by returning early if --user-allow
was not given.

19 months agoclient: Use lsu_merge_config_file_options().
Andre Noll [Thu, 15 Mar 2018 01:45:59 +0000 (02:45 +0100)]
client: Use lsu_merge_config_file_options().

The helper is a drop-in replacement for the old code. We only have
to include the lsu header and link with the object.

19 months agolsu: Add helper to merge config file options, convert server.
Andre Noll [Wed, 14 Mar 2018 19:42:57 +0000 (20:42 +0100)]
lsu: Add helper to merge config file options, convert server.

After the command line options have been parsed, most paraslash
executables read options from a config file. The two lopsub parse
result structures are then merged in a way that command line options
take preference over config file options. This logic is duplicated
in all executables.

This patch introduces a generic helper to eliminate the duplication.
The new lsu_merge_config_file_options() will eventually be employed
by all executables which need to parse the config file. This patch,
however, only converts para_server.

19 months agoTrivial: Rename completion_result variables.
Andre Noll [Wed, 14 Mar 2018 01:14:58 +0000 (02:14 +0100)]
Trivial: Rename completion_result variables.

Most completers call the completion result pointer "cr", but some use
"result" instead. Let's be consistent and rename those to cr.

19 months agoplay: Implement help --long.
Andre Noll [Tue, 13 Mar 2018 23:10:42 +0000 (00:10 +0100)]
play: Implement help --long.

Also para_play benefits from the conversion to the generic
lsu_com_help(). After this patch, the three help commands of server,
audiod and play have the same syntax and their output is formatted
in the same way.

19 months agoaudiod: Implement help --long.
Andre Noll [Tue, 13 Mar 2018 22:47:11 +0000 (23:47 +0100)]
audiod: Implement help --long.

This converts the help command of para_audiod to the generic
lsu_com_help(), adding --long and removing quite a few lines of
duplicated code.  This patch also touches audioc.c since para_audioc
needs to learn to complete the new --long option. Moreover the help
text of the help command is made a bit more useful.

19 months agoIntroduce lsu.{c,h}, implement help --long for para_server.
Andre Noll [Tue, 13 Mar 2018 21:37:39 +0000 (22:37 +0100)]
Introduce lsu.{c,h}, implement help --long for para_server.

This adds the --long option to the server help subcommand. The former
help output becomes the long help while the short help text is shown
if --long is not given.

Although only the help command of para_server is converted in this
patch, the new functionality is implemented in a generic way so
that the help commands of para_audiod and para_play can use the same
implementation. Those will be converted in subsequent patches.

t0004 parses the help output and thus needs to be changed to include
--long.

19 months agoaft: Reorder global static variables of aft.c.
Andre Noll [Mon, 16 Apr 2018 17:43:09 +0000 (19:43 +0200)]
aft: Reorder global static variables of aft.c.

These variables constitute the state of the audio file table. The
overall logic is easier to understand if they are combined at one
spot at the top of the file.

The definition of struct ls_data needs to be moved up because of
this. Pure code movement, no semantic changes.

19 months agoafs: Don't log table open error twice.
Andre Noll [Fri, 13 Apr 2018 07:16:16 +0000 (09:16 +0200)]
afs: Don't log table open error twice.

If ->open() fails, open_afs_tables() returns a negative error code,
so there is no need to log the error string.

19 months agoaft.c: Trivial whitespace fix.
Andre Noll [Wed, 18 Apr 2018 18:33:32 +0000 (20:33 +0200)]
aft.c: Trivial whitespace fix.

19 months agoserver: Add --http-listen-address and --dccp-listen-address.
Andre Noll [Sat, 17 Mar 2018 01:11:01 +0000 (02:11 +0100)]
server: Add --http-listen-address and --dccp-listen-address.

Currently the http and the dccp sender accept connections on all IP
addresses. This commit makes it possible to configure the senders to
listen only on the specified subset of addresses.

To make this work, the sender_status structure has to be extended to
store an array of file descriptors rather than just a single one.
Several places need to iterate over all listening sockets of the
sender. The new FOR_EACH_LISTEN_FD macro helps to avoid duplicating
the corresponding loop.

The status part of the sender command now prints the listening
addresses instead of just on/off. This is why we also maintain the
ascii representation of the listening addresses in the sender_status
structure and introduce the format_url() helper to format host/port
pairs.

19 months agoMove MAX_HOSTLEN from server.h to net.h.
Andre Noll [Wed, 7 Mar 2018 18:45:09 +0000 (19:45 +0100)]
Move MAX_HOSTLEN from server.h to net.h.

This generic networking-related constant does not belong to
server.h. However, since server.h uses the constant, we have to make
sure that net.h is included before server.h.

19 months agoserver: Implement --listen-address for control service.
Andre Noll [Wed, 7 Mar 2018 11:32:15 +0000 (12:32 +0100)]
server: Implement --listen-address for control service.

For hosts with multiple IP addresses one might want to configure the
listening sockets so that connections can only arrive on a subset of
the host's addresses. This patch implements this feature.

Unlike para_listen_simple() the new para_listen() receives an optional
argument to let the caller specify the listening address in addition
to the port number. para_listen_simple() is now a simple wrapper
that passes a NULL pointer as the new argument which indicates that
the socket should listen on all local addresses, just like prior to
this change.

The set of listening addresses for the control service of para_server
can be specified via the new --listen-address option. This option
can be given multiple times, once for each listening address.

Due to this change the server needs to maintain more than a single
file descriptor to dispatch incoming connections. Hence the integer
->listen_fd of struct server_command_task has to be replaced by an
array of file descriptors and the {pre,post}_select methods of the
command task have iterate over all descriptors in the array.

The meaning of the --port option has changed due to the new option:
since the argument to --listen-address can also contain a port number,
the argument to --port is only used for addresses with no port number,
or if --listen-address is not given at all.

Although the http and dccp senders also open a listening socket, this
commit affects only the control service of para_server (listening on
TCP 2990 by default). Senders will be covered in the next commit.

19 months agonet: Combine para_listen() and para_listen_simple().
Andre Noll [Wed, 7 Mar 2018 11:00:58 +0000 (12:00 +0100)]
net: Combine para_listen() and para_listen_simple().

The only caller of para_listen() is para_listen_simple(), since
flowops are only used for active sockets.

19 months agonet: Improve error diagnostics of makesock_addrinfo().
Andre Noll [Thu, 22 Mar 2018 01:09:43 +0000 (02:09 +0100)]
net: Improve error diagnostics of makesock_addrinfo().

Since the function iterates over all addresses in the passed
addressinfo structure, errors can be non-fatal and may even be
expected. Therefore the function does not log any errors from
socket(2), setsockopt(2), connect(2) or bind(2), but only returns
a generic -E_MAKESOCK error code if none of the addressinfo members
worked. Unfortunately, this means it's impossible to tell from the
log message which of these system calls has failed.

This patch changes the function to also log the errors from the above
system calls, but only with loglevel notice. This way the non-fatal
errors are not shown by default (since the default loglevel is
"warning"), but one can easily activate them by specifying a lower
loglevel.

19 months agosend_common: Improve error diagnostics of generic_com_on().
Andre Noll [Sat, 17 Mar 2018 01:10:55 +0000 (02:10 +0100)]
send_common: Improve error diagnostics of generic_com_on().

Currently the function returns an error code but does not log a message
on errors. The callers, http_send_init() and dccp_send_init(), print
the strerror text with no indication that it is was generic_com_on()
which caused the error.

This commit moves the log messages from the callers to generic_com_on()
and changes the return type of generic_com_on() to void because both
callers now ignore the return value. The new log messages include a
text which tells the user what went wrong.

19 months agosend_common: Combine generic_com_on() and open_sender().
Andre Noll [Sat, 17 Mar 2018 00:57:43 +0000 (01:57 +0100)]
send_common: Combine generic_com_on() and open_sender().

The latter function is only called by the former, and both are short.

20 months agofd: Let readv_nonblock() recover from EINTR.
Andre Noll [Mon, 26 Mar 2018 22:39:48 +0000 (00:39 +0200)]
fd: Let readv_nonblock() recover from EINTR.

No need to fail the operation if the read was interrupted by a signal.

The patch also fixes some minor issues in the documentation:
a typo ("The" was incorrectly capitalized) and a missing \ref
for xwrite().  We use the opportunity to get rid of the \a and \p
font annotations. This improves the readability of the source code,
which is more imporatant than nice looking web pages. However, we
only touch those parts of the documentation which are modified anyway.

20 months agomixer: Improve sleep subcommand.
Andre Noll [Fri, 5 Jan 2018 15:11:36 +0000 (16:11 +0100)]
mixer: Improve sleep subcommand.

This avoids a pointless exec of "para_client stop" in case no fade-out
mood is given, and another pointless "para_client play" exec which
was executed when already playing.

Other than that behaviour should be identical.

20 months agohttp_recv: Improve error diagnostics.
Andre Noll [Wed, 21 Mar 2018 12:56:04 +0000 (13:56 +0100)]
http_recv: Improve error diagnostics.

If the http receiver is started when the stream is stopped or paused,
para_recv exits silently with no error message. Let's tell the user
what is going on.

20 months agoafh_recv: Improve error diagnostics.
Andre Noll [Tue, 9 Jan 2018 23:19:13 +0000 (00:19 +0100)]
afh_recv: Improve error diagnostics.

There are many reasons for afh_recv_open() to fail. For example, the
afh receiver could be unable to open its input file, or the given
begin chunk was larger than the end chunk. At the moment the error
reporting is a bit scarce since only the string of the error code is
printed. This commit makes afh_recv_open() print also the reason for
the error and the name of the file that caused the error.

20 months agoafs: Document ->lpr of struct afs_callback_arg.
Andre Noll [Mon, 26 Mar 2018 23:23:07 +0000 (01:23 +0200)]
afs: Document ->lpr of struct afs_callback_arg.

This should have been documented when the ->lpr member was introduced
two years ago in commit ba0a963f (server: Convert com_add() to lopsub).

20 months agobase64.h: Add Copyright and Doxygen header.
Andre Noll [Tue, 20 Mar 2018 02:58:09 +0000 (03:58 +0100)]
base64.h: Add Copyright and Doxygen header.

20 months agodoxygen: Exclude config.h.
Andre Noll [Tue, 20 Mar 2018 03:00:14 +0000 (04:00 +0100)]
doxygen: Exclude config.h.

This file is generated by autoconf and contains no doxygen
comments. Therefore it should not be part of the web documentation.

20 months agorecv: Mention default for --receiver.
Andre Noll [Wed, 21 Mar 2018 12:40:23 +0000 (13:40 +0100)]
recv: Mention default for --receiver.

It's not obvious that http is the default receiver, so let's mention this.

20 months agorecv: Add description of para_recv.
Andre Noll [Mon, 12 Mar 2018 17:32:52 +0000 (18:32 +0100)]
recv: Add description of para_recv.

para_recv is the only executable whose man page has no short
description at the beginning.

20 months agoclient: Remove unused ->config_file member.
Andre Noll [Wed, 14 Mar 2018 19:39:28 +0000 (20:39 +0100)]
client: Remove unused ->config_file member.

Currently we store a copy of the config file path in the client task
structure for no reason at all. Remove it.

20 months agoclient: Remove duplicate error message.
Andre Noll [Wed, 14 Mar 2018 19:33:58 +0000 (20:33 +0100)]
client: Remove duplicate error message.

client_parse_config() returns an error code, so the caller is responsible
for printing a suitable error message.

20 months agoclient: Fix loglevel setting.
Andre Noll [Sat, 17 Mar 2018 22:29:12 +0000 (23:29 +0100)]
client: Fix loglevel setting.

If --loglevel is given in the config file but not at the command line,
the value from the config file should be used. However, currently we
use the default value in this case. Moreover, client_parse_config()
sets the loglevel *after* it already printed some log messages.

This commit changes client_parse_config() to initialize the loglevel
pointer right after the command line and config file options have
been merged.

20 months agocommand.c: Document return value of handle_connect().
Andre Noll [Mon, 26 Mar 2018 23:09:35 +0000 (01:09 +0200)]
command.c: Document return value of handle_connect().

Kind of pointless, but gets rid of the following Doxygen warning:

/home/maan/scm/paraslash/server.h:112: warning: return type of member handle_connect is not documented

20 months agoFix help text of the dccp receiver.
Andre Noll [Mon, 12 Mar 2018 19:18:07 +0000 (20:18 +0100)]
Fix help text of the dccp receiver.

CCIDs are no longer reported in the server info output for more than
three years, see commit 435ad905 (Add sender subcommand "status").

20 months agosend_common: Remove outdated comment.
Andre Noll [Thu, 8 Mar 2018 20:39:24 +0000 (21:39 +0100)]
send_common: Remove outdated comment.

Commit 8dab386f (Rework para_accept()) from eight years ago changed
the function to receive also a pointer to the read fd set and pass
this pointer to para_accept() which performs the check for readability
of the socket file descriptor. Since then the comment which says that
the caller must check the fd set is stale.

20 months agouser_list: Make list head static.
Andre Noll [Sun, 18 Feb 2018 14:48:42 +0000 (15:48 +0100)]
user_list: Make list head static.

It is only used in user_list.c.

20 months agoafs: Allow database switching on sighup.
Andre Noll [Wed, 3 Jan 2018 01:49:22 +0000 (02:49 +0100)]
afs: Allow database switching on sighup.

Currently the database dir is set at startup and stays unmodified even
after the sighup handler has reloaded a configuration which specifies
a different database path.

This patch makes it possible to switch to a different database after
SIGHUP was received. To make this work, we only have invalidate the
database_dir pointer after the database has been closed.

20 months agoafs: Free current mood or playlist on exit.
Andre Noll [Wed, 3 Jan 2018 00:07:45 +0000 (01:07 +0100)]
afs: Free current mood or playlist on exit.

Another harmless memory leak in the afs process which only occurs
on exit.

20 months agoafs: Free status items on exit.
Andre Noll [Wed, 3 Jan 2018 00:01:42 +0000 (01:01 +0100)]
afs: Free status items on exit.

The server process frees the memory allocated for the status items
on exit but the afs process does not. To avoid the leak we have to
make free_status_items() public.

20 months agoafs: Shutdown signals on exit.
Andre Noll [Mon, 1 Jan 2018 03:35:53 +0000 (04:35 +0100)]
afs: Shutdown signals on exit.

This frees the signal task structure, avoiding a (harmless) memory
leak on exit.

20 months agoserver: Free parse result also in afs.
Andre Noll [Mon, 1 Jan 2018 03:28:29 +0000 (04:28 +0100)]
server: Free parse result also in afs.

Both the server and the afs process need to access the lopsub parse
result which is allocated and initialized prior to the fork(2) that
creates the afs process. Hence both processes should free this memory
on exit.

The new public free_lpr() frees the memory allocated by both parse result
structures.

20 months agoafs: Deplete user list at startup.
Andre Noll [Mon, 1 Jan 2018 03:13:17 +0000 (04:13 +0100)]
afs: Deplete user list at startup.

The afs process does not need the user list, so we may free the list
in server.c before calling init_afs().

20 months agoserver: Free audio file header on exit.
Andre Noll [Tue, 2 Jan 2018 23:05:06 +0000 (00:05 +0100)]
server: Free audio file header on exit.

This fixes a harmless memory leak which occurs when para_server terminates
while a header stream is active.

20 months agosender: Deplete ACLs on exit.
Andre Noll [Thu, 16 Nov 2017 02:35:01 +0000 (03:35 +0100)]
sender: Deplete ACLs on exit.

This teaches the dccp and the http sender to empty their access control
lists during shutdown, avoiding some (harmless) memory leaks at exit.

20 months agoRemove some unused includes from {dccp,http}_send.c.
Andre Noll [Thu, 16 Nov 2017 01:41:31 +0000 (02:41 +0100)]
Remove some unused includes from {dccp,http}_send.c.

The access control and close on fork lists are handled in send_common.c,
so the two senders do not need to know anything about them.

The dccp sender does not perform chunk queuing, so it does not need to
include chunk_queue.h.

20 months agoserver: Make argument of user_list_init() constant.
Andre Noll [Sun, 31 Dec 2017 22:37:16 +0000 (23:37 +0100)]
server: Make argument of user_list_init() constant.

The function does not modify the memory pointed to by user_list_file.

20 months agoserver: Deplete user list on exit.
Andre Noll [Sun, 22 Oct 2017 12:46:16 +0000 (14:46 +0200)]
server: Deplete user list on exit.

This eliminates a bunch of valgrind memory leak warnings when
run with --leak-check=full --show-leak-kinds=all.

20 months agoserver: Combine user_list_init() and populate().
Andre Noll [Sun, 13 Aug 2017 20:07:57 +0000 (22:07 +0200)]
server: Combine user_list_init() and populate().

The latter is only called by the former, and both are short enough.

20 months agoserver: Move para_fgets() to user_list.c.
Andre Noll [Sun, 13 Aug 2017 00:36:14 +0000 (02:36 +0200)]
server: Move para_fgets() to user_list.c.

It's only used there, so we can make it static and dedoxify its
documentation.

20 months agoserver: Initialize user list at compile time.
Andre Noll [Sat, 12 Aug 2017 23:05:22 +0000 (01:05 +0200)]
server: Initialize user list at compile time.

This allows the removal of the "initialized" variable, and it
simplifies user_list_init().

20 months agoserver: Rename functions related to user lists.
Andre Noll [Sat, 12 Aug 2017 23:10:01 +0000 (01:10 +0200)]
server: Rename functions related to user lists.

The common user_list prefix of the two public functions declared in
user_list.h indicates that these functions are defined in user_list.c.

20 months agoserver: Constify return value of lookup_user().
Andre Noll [Sat, 12 Aug 2017 23:02:17 +0000 (01:02 +0200)]
server: Constify return value of lookup_user().

Command handlers only need to read this structure.

20 months agoserver: Let stat command handler perform cleanup on signals.
Andre Noll [Sat, 12 Aug 2017 21:27:55 +0000 (23:27 +0200)]
server: Let stat command handler perform cleanup on signals.

The stat command is special because it is the only command which may
run for a very long time, and because it catches SIGUSR1 which is sent
by the server to notify the command handler process when the status
items have changed and the command handler should update the client.

The stat command handler catches neither SIGINT nor SIGTERM, though,
so these signals terminate the process. This happens when para_server
is started without --daemon and CTRL+C is pressed (causing SIGINT
to be sent to the all processes in the foreground process group)
or when the server is about to exit, because in this case the main
process sends SIGTERM to all child processes.

If the command handler process terminates because it received SIGINT
or SIGTERM, valgrind reports various memory leaks. Although the leaks
are harmless, they clutter the valgrind output and make real memory
leaks hard to spot.

This commit changes com_stat() to ignore SIGINT and catch SIGTERM in
addition to SIGUSR1. We don't need to care about SIGINT in com_stat()
because in the above scenario, the main process sends SIGTERM to all
command handlers after it received SIGINT. With the patch applied, if
com_stat() received SIGTERM, it returns and the normal cleanup on exit
is performed.

We have to employ pselect(2) to avoid race conditions.

20 months agoserver: Have afs process close the current mood on exit().
Andre Noll [Sat, 12 Aug 2017 20:02:48 +0000 (22:02 +0200)]
server: Have afs process close the current mood on exit().

This eliminates a few (harmless but annoying) warnings about memory
leaks from the valgrind output.

20 months agoserver: Deplete close on fork list on exit.
Andre Noll [Sat, 12 Aug 2017 19:32:28 +0000 (21:32 +0200)]
server: Deplete close on fork list on exit.

We empty the list in the command handler process by calling
close_listed_fds(), but the server process leaks the memory
allocated for the entries of the close on fork list.

This commit introduces deplete_close_on_fork_list() to empty the
list without closing any file descriptors. It is called from main()
to avoid the leak.

With the patch applied, valgrind --show-reachable=no no longer
complains about possibly lost blocks for the server process.

20 months agodaemon: Introduce log mutex.
Andre Noll [Mon, 7 Aug 2017 22:02:03 +0000 (00:02 +0200)]
daemon: Introduce log mutex.

Currently the server processes append messages to the log file without
coordination. This usually does not cause problems, but sometimes
the log messages of the main server process and the afs process get
interleaved due to the lack of serialization.

This patch serializes access to the common log file by employing a
new lock: the log_mutex. Like the mmd_mutex, it is realized as an
one-element System V semaphore set.

Logging is performed by the daemon subsystem implemented in daemon.c.
This subsystem is also used by para_audiod which is single-threaded
and thus does not need to care about serialization of log messages.
To keep daemon.c working for both para_server and para_audiod,
struct daemon gains two optional methods, ->pre_log_hook and
->post_log_hook. If the function pointers are not NULL, the methods
are called before and after a message is logged. For para_server
the methods call back to the server code to take and release the
lock. para_audiod does not need to be modified because if the new
function pointers are left at their default value NULL, so no hooks
are called.

20 months agoserver: Exit cleanly on SIGINT/SIGTERM.
Andre Noll [Mon, 7 Aug 2017 19:41:00 +0000 (21:41 +0200)]
server: Exit cleanly on SIGINT/SIGTERM.

Currently signal_post_select() kills all child processes and then
calls exit(3) if SIGNINT or SIGTERM was received. This leaves all file
descriptors open and memory blocks allocated, which makes debugging
memory leaks difficult because the valgrind output is hard to read.

This patch changes the server to cleanly shutdown the scheduler and
deallocate resources (close file descriptors, free memory, destroy
locks and shared memory areas) before exit(3) is called.

20 months agoserver: Cleanly shut down senders on exit.
Andre Noll [Mon, 7 Aug 2017 19:11:26 +0000 (21:11 +0200)]
server: Cleanly shut down senders on exit.

This introduces ->shutdown for struct sender. Each of the three senders
implements the new method to close file descriptors and to deallocate
the resources occupied by the sender. The method is only called on
exit via the new vss_shutdown() from both server and command handler
context after schedule() returns.

We need to introduce another helper, process_is_command_handler(), to
distinguish between the two callers.

20 months agoserver: Shutdown the scheduler before handling commands.
Andre Noll [Mon, 7 Aug 2017 01:44:07 +0000 (03:44 +0200)]
server: Shutdown the scheduler before handling commands.

Currently the command handlers are called from the ->post_select
method of the command task after the main server process has forked.
When the command handler is done, it exits with the scheduler still
active. This is not a problem per se, and it has been like this
for more than a decade, but it does make it harder to debug memory
leaks because we leak some resources. Valgrind complains about this,
cluttering the output with pointless warnings.

This commit cleans up the memory handling of the child process on
exit. The command task of the child process notifies all tasks in its
->post_select method and then returns the new pseudo error code
-E_CHILD_CONTEXT. This causes the scheduler to return to main().

The main() function checks via the new ->child_fd of struct
server_command_task whether it is running in child context after
schedule() has returned and calls the command handler in this case.

20 months agoserver: Make array of senders constant.
Andre Noll [Mon, 7 Aug 2017 15:44:15 +0000 (17:44 +0200)]
server: Make array of senders constant.

Currently we define an array of static non-constant sender structures
in vss.c with only ->init being initialized at compile time.
At startup the ->init method of each sender must fill out the other
function pointers of the structure. This approach is kind of pointless,
as these pointers are known at compile time and never change.

This patch changes the three senders to define a non-static constant
sender structure instead. The vss maintains an array of pointers to
these structures.

Since the new sender structures are non-static, they need to be
doxified. We use the opportunity to provide an overview of each sender.

The patch also introduces FOR_EACH_SENDER(), a macro which is used
several times to replace the open-coded loops we had before.

20 months agoserver: Define server command task structure in main().
Andre Noll [Sun, 6 Aug 2017 21:39:02 +0000 (23:39 +0200)]
server: Define server command task structure in main().

This way it does not need to be static, and we can look at the
contents after schedule() returns.

Preparatory patch for shutting down cleanly on exit.

20 months agobuild: Compile with -Wdiscarded-qualifiers.
Andre Noll [Wed, 3 Jan 2018 17:50:15 +0000 (18:50 +0100)]
build: Compile with -Wdiscarded-qualifiers.

This option is unsupported (but warnings are invariably enabled) for
old gcc versions including 4.2.2. It is supported but not enabled by
default for vanilla gcc-5.5.0, but is enabled for the gcc versions
which ship with Ubuntu-14.04 (4.8.4) and Ubuntu-16.04 (5.4.0).

This patch enables the warning everywhere. We have to employ the
cc-option macro to determine whether the option is supported.

20 months agoacl: Fix/improve acl_del_entry log message.
Andre Noll [Wed, 3 Jan 2018 13:12:00 +0000 (14:12 +0100)]
acl: Fix/improve acl_del_entry log message.

The given address/netmask can match any number of entries. Currently
we log one message per removed address but all messages contain the
address/netmask to match rather than the removed one that was stored
in the acl.

This patch changes acl_del_entry() to log both, with different
severities.

20 months agoacl: Avoid undefined behaviour due to 32-bit shift.
Andre Noll [Thu, 16 Nov 2017 02:24:47 +0000 (03:24 +0100)]
acl: Avoid undefined behaviour due to 32-bit shift.

On x86 a shift by 32-bits is a nop, but this is not backed by any
standard. Moreover, not shifting at all results in an effective netmask
of 32, which is unexpected for for v4_addr_match(), as the function
is supposed to answer the question "do the first zero bits of these
two addresses match?". This should always be true, regardless of the
two addresses.

Fix this, and change the return type of the function while at it.

20 months agoafs: Fix typo in error message.
Andre Noll [Wed, 3 Jan 2018 00:48:09 +0000 (01:48 +0100)]
afs: Fix typo in error message.

It was the ->open() method that failed when this message is logged.

20 months agoafs: Fix memory leak in make_status_items().
Andre Noll [Tue, 2 Jan 2018 23:55:03 +0000 (00:55 +0100)]
afs: Fix memory leak in make_status_items().

The leak occurs when the second call to print_list_item() fails. In
this case we leak the memory pointed to by the lpr pointer. This patch
also streamlines the code a bit by introducing the free_status_items()
helper.

20 months agofd.c: Simplify para_mmap().
Andre Noll [Sun, 14 Jan 2018 17:57:22 +0000 (18:57 +0100)]
fd.c: Simplify para_mmap().

Both callers pass in a zero offset, so we can get rid of this
parameter.

21 months agogcrypt: Allocate a secmem pool at startup.
Andre Noll [Wed, 3 Jan 2018 03:33:02 +0000 (04:33 +0100)]
gcrypt: Allocate a secmem pool at startup.

The client side loads private keys. This patch makes sure the memory
which contains such key material is never swapped out.

21 months agocrypt: Introduce crypt_shutdown().
Andre Noll [Fri, 29 Dec 2017 15:22:37 +0000 (16:22 +0100)]
crypt: Introduce crypt_shutdown().

This plugs a few harmless memory leaks in the openssl crypto
backend. The leaks occur on exit and are only reported by valgrind
if it is run with --leak-check=full --show-leak-kinds=all.

The gcrypt backend has similar problems, but there is no way to
provide a similar patch for libgrypt. The newly added comment in
gcrypt.c explains why.

21 months agocrypt: Rename init_random_seed_or_die() -> crypt_init().
Andre Noll [Thu, 28 Dec 2017 20:57:23 +0000 (21:57 +0100)]
crypt: Rename init_random_seed_or_die() -> crypt_init().

At least the gcrypt implementation does more than just seed the
PRNG. The new name is shorter and more descriptive.

21 months agogcrypt: Use ROUND_DOWN() macro.
Andre Noll [Thu, 28 Dec 2017 20:48:10 +0000 (21:48 +0100)]
gcrypt: Use ROUND_DOWN() macro.

21 months agocrypt: Rename RSA functions.
Andre Noll [Thu, 28 Dec 2017 14:52:31 +0000 (15:52 +0100)]
crypt: Rename RSA functions.

This renames the functions and constants of the public crypto API
which deal with RSA to have the common "apc" (asymmetric pubkey
cryptosystem) prefix.  This hides RSA as an implementation detail
and makes it clear that the functions/constants are related to apc.

Pure renaming, no semantic change. The only other changes are a new
comment to crypt.h and a \ref statement in the doxygen comment of
apc_free_pubkey().

21 months agocrypt: Deduplicate get_public_key().
Andre Noll [Thu, 28 Dec 2017 00:41:14 +0000 (01:41 +0100)]
crypt: Deduplicate get_public_key().

The openssl and grypt implementations of this function share quite
some code. This patch factors out the common code into the new
decode_ssh_key() helper of crypt_common.c. Both implementations are
changed to call the new helper.

As a side effect of this change, all callers of is_ssh_rsa_key()
and check_ssh_key_header() now reside in crypt_common.c, so we can
make these two functions static and remove their declarations from
crypt_backend.h.

21 months agocrypt.h: Remove two unused constants.
Andre Noll [Wed, 27 Dec 2017 20:59:33 +0000 (21:59 +0100)]
crypt.h: Remove two unused constants.

These should have been removed already last year in commit d9f54f43
(crypto: Remove support for ASN public keys), since that commit made
the constants obsolete.

21 months agoRename crypt.c -> openssl.c.
Andre Noll [Wed, 27 Dec 2017 20:47:40 +0000 (21:47 +0100)]
Rename crypt.c -> openssl.c.

The old name made sense back when we had only one crypto backend. These
days paraslash can be compiled against either openssl or gcrypt,
so the new name makes it clear that the file is only compiled in if
the openssl library was selected as the crypto backend.

21 months agoserver: Avoid use of uninitialized memory.
Andre Noll [Mon, 29 Jan 2018 22:21:11 +0000 (23:21 +0100)]
server: Avoid use of uninitialized memory.

change_current_mood() receives an errmsg pointer which the callers
expect to be initialized with an error string if (and only if) the
function returns negative.

However, most error paths miss to initialize the pointer which results
in undefined behaviour in the caller which attempts to free(3)
uninitialized memory. The gcc AddressSanitizer and valgrind both
catch this:

gcc:

==14788==ERROR: AddressSanitizer: attempting free on address which was not malloc()-ed: 0x081af250 in thread T0

valgrind:

==4410== Invalid free() / delete / delete[] / realloc()

The bug was introduced half a year ago when version 2 moods were
introduced in commit 3d3a2f50.

21 months agodaemon: Fix log reload for relative paths.
Andre Noll [Sun, 31 Dec 2017 20:56:35 +0000 (21:56 +0100)]
daemon: Fix log reload for relative paths.

If the argument to --logfile is a relative path, it is interpreted
as relative to the current working directory. In daemon mode, the
current working directory is changed to / during startup. Hence,
when para_server re-opens the log file after it received SIGHUP, the
logfile path will now be interpreted as relative to the the root of
the file system.

Fix this by remembering the original current working directory.
Opening "." as recommended in getcwd(3) is not an option here since the
whole point of changing the cwd to / is to prevent the daemon from
keeping the cwd busy.

21 months agoImprove daemon_open_log_or_die().
Andre Noll [Sun, 31 Dec 2017 19:32:52 +0000 (20:32 +0100)]
Improve daemon_open_log_or_die().

If the log file can not be re-opened, the error message is lost
because the log file has already been closed when PARA_EMERG_LOG() is
called. We can do better by deferring the call to daemon_close_log()
until the new log file has been opened.

With the patch applied, the reason why the (new) log file could not
be opened is logged to the old file.

21 months agoserver: Close logfile later on SIGHUP.
Andre Noll [Sun, 31 Dec 2017 19:27:57 +0000 (20:27 +0100)]
server: Close logfile later on SIGHUP.

The call to daemon_open_log_or_die() will close the log file just
before it is reopened. This is better because that way any errors
which occur in between won't get lost.

21 months agoserver: Simplify afs socket cookie code.
Andre Noll [Sun, 31 Dec 2017 15:27:25 +0000 (16:27 +0100)]
server: Simplify afs socket cookie code.

Currently, the afs_socket_cookie value is passed to afs_init() and
is stored in the commmand task structure of the afs process. This
is unnecessary since the variable defined in server.c is non-static,
and declared as extern in afs.c. We may thus refer to this variable
from afs.c.

The variable was also documented twice. Get rid of one comment.

21 months agomixer: Remove default moods for sleep.
Andre Noll [Tue, 30 Jan 2018 18:56:16 +0000 (19:56 +0100)]
mixer: Remove default moods for sleep.

The documented behavior of the sleep subcommand is to not fade out
if no fade out mood was given. The current code, however, tries to
switch to the default m/fade. This fails if this mood does not exist,
causing the sleep subcommand to terminate. By removing the default
value we get the documented behaviour.

The --fo-mood and --sleep-mood suffer from the same issue and can be
fixed in the same way.

21 months agobuild: Error out early if flex or bison are not found.
Andre Noll [Thu, 11 Jan 2018 08:05:13 +0000 (09:05 +0100)]
build: Error out early if flex or bison are not found.

The configure script tests for these executables, but we happily
proceed even if they are not found. For the m4 executable we got
the check right, however. Introduce a general helper that calls
AC_PATH_PROG() to locate an executable and aborts if it is not found.

21 months agoFix bad grammar "allows <infinitive>".
Andre Noll [Tue, 7 Nov 2017 22:29:03 +0000 (23:29 +0100)]
Fix bad grammar "allows <infinitive>".

In standard English, the verb "allows" can never take an infinitive as
its direct object. This patch corrects these mistakes.