paraslash.git
4 months agoaudiod: Demote severity level of command errors.
Andre Noll [Sun, 17 Jul 2022 19:33:24 +0000 (21:33 +0200)]
audiod: Demote severity level of command errors.

handle_connect() returning negative is a normal condition which
occurs, for example, when the user specifies an invalid argument to
a command. Therefore the "notice" severity level is more appropriate
than the current "error" severity.

4 months agoaudiod: Fix time diff warning log message.
Andre Noll [Mon, 13 Jun 2022 18:31:23 +0000 (20:31 +0200)]
audiod: Fix time diff warning log message.

The format string contains %lu to print a long unsigned, but we
possibly multiply the value with -1, which can lead to output like

Jun 13 13:36:37 baader (3) compute_time_diff: time diff jump: 4294808018ms

Fix this by replacing the multiplication with an additional format
string directive to print the leading +/- explicitly. This is easy
since we already have the sign in a variable, and it avoids any
integer conversion/overflow issues.

6 months agoRemove para_dirname() and para_basename().
Andre Noll [Fri, 11 Mar 2022 18:32:24 +0000 (19:32 +0100)]
Remove para_dirname() and para_basename().

The former has only a single caller, the second only two, open-coding
these is actually simpler and more performant because we no longer
scan each path twice and avoid the temporary copy of the path.

6 months agoRevert "Makefile: Fix compilation after header removal."
Andre Noll [Mon, 30 May 2022 19:06:32 +0000 (21:06 +0200)]
Revert "Makefile: Fix compilation after header removal."

This reverts commit 3bc858ee0d9b929f4cbca61cf5ed8d3184d0cf78.

It is broken because the recipe for the .d files does not honor
CPPFLAGS since we only modify the CPPFLAGS variable for the .o
targets. Adding the corresponding .d targets would be too ugly,
so revert the commit for now until a better solution has been found.

6 months agomood.c: Improve documentation of reload_current_mood().
Andre Noll [Thu, 19 May 2022 16:58:53 +0000 (18:58 +0200)]
mood.c: Improve documentation of reload_current_mood().

The old text was a bit clumsy and too short.

6 months agoRemove E_PERM.
Andre Noll [Tue, 17 May 2022 14:17:32 +0000 (16:17 +0200)]
Remove E_PERM.

There is only one user which may as well use the standard error code.

6 months agoMakefile: Fix compilation after header removal.
Andre Noll [Mon, 14 Mar 2022 21:28:27 +0000 (22:28 +0100)]
Makefile: Fix compilation after header removal.

When switching from an older git version which still contains some
header file to a newer version where it got removed, a .d file remains
in build/deps which lists the no longer existing header file as a
prerequisite. This causes the build to fail because the prerequisite
cannot be created. The problem can be worked around by removing the
stale .d file, for example by running make clean, but this is no real
fix, and is inefficient.

The root of the matter is that .d files depend on their .c counterpart,
but this dependency is not stated anywhere in the Makefile. Thus, we
need a rule for the .d target with the same prerequisites and the same
recipe as the object file target. GNU make supports multiple targets,
but the feature does not seem to work as advertised, regardless of
whether rules with independent targets or rules with grouped targets
(using the &: separator) are employed. Thus we bite the bullet and
use two separate rules.

6 months agoAdd test to check whether empty mood arguments are rejected.
Andre Noll [Sat, 12 Mar 2022 03:29:03 +0000 (04:29 +0100)]
Add test to check whether empty mood arguments are rejected.

To make sure a recent bug won't be re-introduced. The test succeeds
but fails with commit

2c2d3a6ec34d983cb698b245facb8d6240aec1b3

reverted.

We have to add at least one mood to trigger the bug. Since addmood is
a stdin command we must redirect stdin for the para_client command. It
does not hurt to do this for all commands, not just the new one.

6 months agoMinor log severity adjustments.
Andre Noll [Wed, 16 Mar 2022 18:21:24 +0000 (19:21 +0100)]
Minor log severity adjustments.

Two small changes which improve the error output of para_audiod. The
first makes sure we get a message when hitting ctrl+c when audiod
is running in forground mode. The second decreases the severity of
the log message which is shown when a client connection terminates,
since this is not necessarily an error.

6 months agoserver: Early vss shutdown for command handlers.
Andre Noll [Wed, 20 Oct 2021 19:13:58 +0000 (21:13 +0200)]
server: Early vss shutdown for command handlers.

Command must communicate with the vss through the shared memory area
(mmd). Deallocating all resources early in the command handler makes
the code more robust and saves some memory.

6 months agoMerge branch 'maint'
Andre Noll [Sat, 14 May 2022 15:21:42 +0000 (17:21 +0200)]
Merge branch 'maint'

To get the single fix that was just merged to maint.

* maint:
  server: Fix memory leak at exit.

7 months agoMerge branch 'maint'
Andre Noll [Mon, 4 Apr 2022 19:28:37 +0000 (21:28 +0200)]
Merge branch 'maint'

A single urgent fix for a server crash.

* maint:
  vss: Reset afhi.chunks_total on eof.

8 months agoafh: Limit chunk numbers to 32 bit.
Andre Noll [Fri, 27 Aug 2021 13:12:34 +0000 (15:12 +0200)]
afh: Limit chunk numbers to 32 bit.

The number of chunks and the chunk offsets are stored in the audio
file table as 32 bit unsigned integers. Thus, chunk numbers and
sizes cannot exceed 2^32 - 1. Make this fact obvious by changing
the corresponding parameters of aac_afh_get_chunk() from size_t or
unsigned long to uint32_t.

8 months agoaudiod: Stop stream immediately when being switched off.
Andre Noll [Wed, 9 Mar 2022 22:56:56 +0000 (23:56 +0100)]
audiod: Stop stream immediately when being switched off.

When para_audiod enters off/standby mode, we stop the receiver
but continue to filter and write data already received until the
input queue has drained. This can take several seconds, which may
be annoying.

To stop the stream immediately, stop the writer(s) rather than the
receiver. We still only stop the receiver when the server has paused
or stopped the stream because we don't want to discard any data in
this case.

8 months agoserver: Fix memory leak at exit.
Andre Noll [Tue, 22 Mar 2022 20:05:19 +0000 (21:05 +0100)]
server: Fix memory leak at exit.

If command_post_select() returns failure because of a notification we
leak the array of listening fds. No big deal, but worth to fix anyway.

8 months agovss: Reset afhi.chunks_total on eof.
Andre Noll [Sat, 19 Mar 2022 22:48:18 +0000 (23:48 +0100)]
vss: Reset afhi.chunks_total on eof.

Without this, the server can be crashed by running

para_client nomore # set the nomore flag
# wait for eof
para_client jmp 50 # set the repos flag, clear the next flag
para_client play # boom

The problem is that the command handler of the jmp command checks
afhi.chunks_total to determine if some audio file is currently
open. Since vss_eof() did not reset chunks_total, the command handler
wrongly believes that there is an open audio file, sets the repos
flag and clears the next flag. When streaming is resumed later, the
virtual streaming system attempts to access the chunk table which
was freed earlier in vss_eof().

8 months agoafs.c: Improve documentation of callback_query.
Andre Noll [Thu, 17 Mar 2022 21:44:11 +0000 (22:44 +0100)]
afs.c: Improve documentation of callback_query.

It had some language issues and focussed too much on details rather than
explaining the big picture. The new text is shorter and should be much easier
to understand.

8 months agoafs.c: Improve documentation of activate_mood_or_playlist().
Andre Noll [Thu, 17 Mar 2022 19:36:06 +0000 (20:36 +0100)]
afs.c: Improve documentation of activate_mood_or_playlist().

It is not obvious when arg and current_mop can alias each other so
add an explanatory comment.

8 months agoafs.c: Replace underscore by space in log message text.
Andre Noll [Wed, 16 Mar 2022 18:31:55 +0000 (19:31 +0100)]
afs.c: Replace underscore by space in log message text.

8 months agoafs.c: Improve documentation of score table.
Andre Noll [Wed, 9 Mar 2022 21:43:26 +0000 (22:43 +0100)]
afs.c: Improve documentation of score table.

The text focussed on implementation details rather than describing the
purpose of the table. It was also slightly incorrect because playlists
also use the score tables, albeit in a much simpler way and without
"Gaussian normal distributions".

8 months agoscore.c: Don't try to document possible error codes.
Andre Noll [Fri, 18 Mar 2022 19:55:48 +0000 (20:55 +0100)]
score.c: Don't try to document possible error codes.

Every single one of these was incomplete or outdated.

8 months agoscore.c: Remove pointless comments.
Andre Noll [Mon, 7 Mar 2022 21:30:26 +0000 (22:30 +0100)]
score.c: Remove pointless comments.

Static functions don't need doxygen comments. The two comments removed
by this commit did not add any value anyway.

8 months agoscore.c: Improve documentation of score_compare().
Andre Noll [Tue, 15 Mar 2022 20:47:08 +0000 (21:47 +0100)]
score.c: Improve documentation of score_compare().

Reword the text to focus on why we dance twice. Also put it on dedox
because it is static.

8 months agoscore.c: Shorten documentation of get_score_of_row().
Andre Noll [Mon, 7 Mar 2022 21:25:24 +0000 (22:25 +0100)]
score.c: Shorten documentation of get_score_of_row().

This function is static, and its implementation is shorter than its
documentation, which mostly consists of pointless boilerplate text.

8 months agoattribute.c: De-doxify static functions.
Andre Noll [Tue, 15 Mar 2022 20:33:03 +0000 (21:33 +0100)]
attribute.c: De-doxify static functions.

Because static functions don't need doxygen comments. The existing
comments for attribute_close() and attribute_open() did not contain
useful information, so remove these,

8 months agoMerge branch 'maint'
Andre Noll [Thu, 24 Mar 2022 18:58:12 +0000 (19:58 +0100)]
Merge branch 'maint'

A single fix for a benign bug.

* maint:
  afs: Update dummy mood assumptions to reflect the reality.

8 months agoafs: Update dummy mood assumptions to reflect the reality.
Andre Noll [Mon, 14 Mar 2022 18:52:46 +0000 (19:52 +0100)]
afs: Update dummy mood assumptions to reflect the reality.

The code in afs.c assumes that loading the dummy mood always succeeds,
and this is even documented in change_current_mood(). However, this
has never been true because we call into osl library functions which
may fail for various reasons. In particular, if the server is started
without a database the attempt to load the dummy mood fails because
the audio file table does not exist.

The current code was not prepared to handle this case, and does stupid
things like storing the negative error code in *num_admissible and
returning success.

Fix this confusion by adjusting the documentation and letting
activate_mood_or_playlist() fail early. One of its callers,
init_admissible_files(), needs also be adjusted because it asserted
in its error path that the mood which failed to load was not the
dummy mood.

This is a benign bug because the most common way to hit this is
at startup on a fresh install when the database does not exist. In
this case the caller, init_admissible_files(), ignores the negative
num_admissible value.

8 months agocom_ls(): Drop "audio file table syntax error" messsage.
Andre Noll [Sun, 13 Feb 2022 20:29:03 +0000 (21:29 +0100)]
com_ls(): Drop "audio file table syntax error" messsage.

This error message is printed if ls is called with an invalid argument
for the listing mode or sort order. That's not a syntax error, so
use the standard EINVAL error code instead.

8 months agoaac_afh: Be more lenient about zero sized reads.
Andre Noll [Wed, 18 Aug 2021 14:44:15 +0000 (16:44 +0200)]
aac_afh: Be more lenient about zero sized reads.

These actually happen for example when the file contains a meta
tag with an empty string value. POSIX says that the read() function
shall return zero and have no other results, so don't return -1 if
the number of bytes read is zero.

Similarly, return zero if the file offset is beyond EOF.

8 months agoparaslash 0.7.0 v0.7.0
Andre Noll [Sat, 12 Mar 2022 16:31:50 +0000 (17:31 +0100)]
paraslash 0.7.0

8 months agoMerge branch 'refs/heads/t/logo'
Andre Noll [Sat, 12 Mar 2022 16:22:15 +0000 (17:22 +0100)]
Merge branch 'refs/heads/t/logo'

The fancy new svg logo and a couple of changes which streamline
the look of the web pages.

Cooking for a month.

* refs/heads/t/logo:
  web: Add horizontal space between table columns.
  web: Clean up style sheet.
  web: Avoid bold face on main page.
  web: Add padding to all pages.
  web: Move navigation into top level table.
  web: Link to "./index.html" rather than to ".".
  web: Remove selected APIs page.
  web: Avoid "integrated" in feature list of main page.
  web: Shorten slogan.
  web: The new logo for paraslash-0.7.
  manual: Avoid vertical space between coding style items.

8 months agomood.c: Fix memory leak in error path of load_mood().
Andre Noll [Thu, 10 Mar 2022 20:32:22 +0000 (21:32 +0100)]
mood.c: Fix memory leak in error path of load_mood().

When an existing mood definition happens to be invalid so that
mp_init() fails, we return negative and leak the mood structure
just allocated.

This bug was introduced by the commit which removed version 1 moods.
It was found by code inspection.

Fixes: 184ea897a9b4

8 months agoMerge branch 'maint'
Andre Noll [Sat, 12 Mar 2022 15:55:45 +0000 (16:55 +0100)]
Merge branch 'maint'

A couple of urgent fixes for old bugs, the critical one being the
patch to reject empty mood arguments.

Cooking only for a few days to get these fixes merged ASAP.

* maint:
  com_addatt(): Fix memory leak.
  mood.c: Fix memory leak in change_current_mood().
  mp.c: Fix memory leak on parse errors.
  Reject empty mood arguments.

8 months agocom_addatt(): Fix memory leak.
Andre Noll [Mon, 7 Mar 2022 21:02:11 +0000 (22:02 +0100)]
com_addatt(): Fix memory leak.

We miss to free the lopsub parse result on exit. Found by valgrind.

8 months agomood.c: Fix memory leak in change_current_mood().
Andre Noll [Tue, 8 Mar 2022 22:37:08 +0000 (23:37 +0100)]
mood.c: Fix memory leak in change_current_mood().

In two error cases we return without freeing the bison mood parser
and the temporary array. Worse, in these cases we also expose the
partially loaded mood via the global current_mood.

The good news is that these errors should be "impossible" to trigger
in practice.

8 months agomp.c: Fix memory leak on parse errors.
Andre Noll [Thu, 10 Mar 2022 21:14:01 +0000 (22:14 +0100)]
mp.c: Fix memory leak on parse errors.

When mp_yyparse() fails we miss to free the abstract syntax tree.

Found by valgrind.

8 months agoserver: Loglevel adjustments for shutdown path.
Andre Noll [Thu, 21 Oct 2021 19:48:24 +0000 (21:48 +0200)]
server: Loglevel adjustments for shutdown path.

The log output of the server is rather verbose with respect to
shutdown messages. These not only occur when the server exits but
also in command handler context each time a command is executed.

This patch quietens these messages in two ways. For some messages we
simply reduce the log level. For others we print the message only in
server context, i.e. when process_is_command_handler() returns false.

8 months agoReject empty mood arguments.
Andre Noll [Fri, 11 Mar 2022 23:42:06 +0000 (00:42 +0100)]
Reject empty mood arguments.

Currently we crash in mood.c due to an assertion.

9 months agoMerge tag 'v0.6.4'
Andre Noll [Tue, 22 Feb 2022 08:17:23 +0000 (09:17 +0100)]
Merge tag 'v0.6.4'

paraslash-0.6.4

9 months agoMerge branch 'refs/heads/t/sha256'
Andre Noll [Mon, 21 Feb 2022 14:52:42 +0000 (15:52 +0100)]
Merge branch 'refs/heads/t/sha256'

A couple of changes which start to eliminate the use of sha1 in favor
of sha256. This series is only the first step, though, as we need to
keep sha1 for the time being to provide backward compatibility.

Cooking for four months.

* refs/heads/t/sha256:
  manual: Avoid sha1.
  upgrade_db: Add copyright and purpose to upgrade_db.c.
  web: Add link to the para_upgrade_db(1) man page.
  afs: Switch to sha256 and change default database path.
  Add para_upgrade_db.
  Use sha256 for the challenge response.
  Introduce hash2 (sha256).
  Assume sideband and aes_ctr128 are always supported/requested.

9 months agoaudiod: Avoid delay in status transition from standby to on.
Andre Noll [Thu, 25 Nov 2021 19:25:43 +0000 (20:25 +0100)]
audiod: Avoid delay in status transition from standby to on.

In close_receiver() we invalidate the current audio format value
to prevent the receiver from re-starting too quickly. See commit
2d61a8005c74 (audiod: Invalidate current audio format on close)
from seven years ago for details.

This invalidation happens in particular when the receiver is closed
because audiod was told to enter standby mode. If audiod is switched
back on shortly thereafter, no receiver can't be started because we
forgot the current audio format and must wait up to one minute until
the next status item dump arrives from the server which re-initializes
the current audio format value.

Fix this flaw by only invalidating the current audio format when
audiod operates in "on" mode.

Fixes: 2d61a8005c74

9 months agoweb: Add horizontal space between table columns.
Andre Noll [Sun, 14 Nov 2021 19:16:01 +0000 (20:16 +0100)]
web: Add horizontal space between table columns.

This tweaks the style sheet to make the rows of the tables easier to
read. It affects the topic list on the development page and the list
of files on the doxygen API page.

9 months agoweb: Clean up style sheet.
Andre Noll [Tue, 9 Nov 2021 17:16:15 +0000 (18:16 +0100)]
web: Clean up style sheet.

This removes some unnecessary cruft. The pages still look as before,
at least in elinks and firefox.

9 months agoweb: Avoid bold face on main page.
Andre Noll [Sun, 7 Nov 2021 22:21:03 +0000 (23:21 +0100)]
web: Avoid bold face on main page.

There is no reason to highlight this particular word.

9 months agoweb: Add padding to all pages.
Andre Noll [Sat, 6 Nov 2021 20:48:32 +0000 (21:48 +0100)]
web: Add padding to all pages.

Wrap text paragraphs in <p>...</p> and increase the margin value for
the html body in the css file. This looks much better in firefox.

No change to the contents.

9 months agoweb: Move navigation into top level table.
Andre Noll [Sat, 6 Nov 2021 20:28:10 +0000 (21:28 +0100)]
web: Move navigation into top level table.

Now the pages no longer look as if they were made in the nineties. More
like 2001 :)

9 months agoweb: Link to "./index.html" rather than to ".".
Andre Noll [Sat, 6 Nov 2021 20:31:59 +0000 (21:31 +0100)]
web: Link to "./index.html" rather than to ".".

When browsing the generated html locally, clicking the logo or the
about link may cause the browser to open the directory view, which
is annoying. This commit makes these links open the main page instead.

9 months agoweb: Remove selected APIs page.
Andre Noll [Sat, 6 Nov 2021 19:43:30 +0000 (20:43 +0100)]
web: Remove selected APIs page.

This list was outdated and not well maintained. Change the link on
the documentation page to point to the list of files instead.

9 months agoweb: Avoid "integrated" in feature list of main page.
Andre Noll [Sun, 14 Nov 2021 19:06:39 +0000 (20:06 +0100)]
web: Avoid "integrated" in feature list of main page.

It's not clear what this is supposed to mean.

9 months agoweb: Shorten slogan.
Andre Noll [Sun, 31 Oct 2021 22:52:02 +0000 (23:52 +0100)]
web: Shorten slogan.

The two removed words do not convey any additional information.

9 months agoweb: The new logo for paraslash-0.7.
Andre Noll [Sun, 31 Oct 2021 22:07:06 +0000 (23:07 +0100)]
web: The new logo for paraslash-0.7.

This new svg logo was not made with a graphical editor but with a
plain text exitor. Unfortunately, ImageMagick is unable to create
a working .ico file from the svg source. Therefore we continue to
carry a manually created .ico file in the repo.

The new logo is less wide than the previous one, and it does not look
nice when it floats to the right side of its tabular cell. Fortunately
this can be fixed by removing some code from the style sheet.

9 months agomanual: Avoid vertical space between coding style items.
Andre Noll [Mon, 8 Nov 2021 10:43:49 +0000 (11:43 +0100)]
manual: Avoid vertical space between coding style items.

For some reason markdown inserts vertical space between adjacent
items if the code example is not enclosed in <pre>...</pre>.

9 months agoMerge branch 'refs/heads/t/rm_v1_moods'
Andre Noll [Thu, 10 Feb 2022 15:34:19 +0000 (16:34 +0100)]
Merge branch 'refs/heads/t/rm_v1_moods'

A single commit which removes a long obsolete feature.

Cooking for almost a year.

* refs/heads/t/rm_v1_moods:
  Remove support for version 1 moods.

10 months agoUpdate copyright year.
Andre Noll [Mon, 3 Jan 2022 19:32:18 +0000 (20:32 +0100)]
Update copyright year.

Happy new year.

10 months agoMerge branch 'refs/heads/t/rm_task_subcmd'
Andre Noll [Mon, 3 Jan 2022 19:18:08 +0000 (20:18 +0100)]
Merge branch 'refs/heads/t/rm_task_subcmd'

A single commit containing an incompatible change for 0.7.0.

Cooking for almost a year.

* refs/heads/t/rm_task_subcmd:
  Remove obsolete server subcommand "task".

10 months agoweb: Add Download link for master tarballs.
Andre Noll [Fri, 3 Dec 2021 15:43:51 +0000 (16:43 +0100)]
web: Add Download link for master tarballs.

This link was deleted by accident when v0.6.2 was released.

11 months agomanual: Avoid sha1.
Andre Noll [Sat, 4 Dec 2021 16:41:34 +0000 (17:41 +0100)]
manual: Avoid sha1.

The section on client-server authentication is no longer accurate
because it depends on the version of client and server whether sha1
or sha256 is used for authentication. Since the hash function is an
implementation detail which should not be mentioned in the manual,
let's avoid the term and talk about cryptographic hash functions
instead.

12 months agoMerge branch 'refs/heads/t/ubsan'
Andre Noll [Sun, 28 Nov 2021 17:10:11 +0000 (18:10 +0100)]
Merge branch 'refs/heads/t/ubsan'

A single patch which adds an option to ./configure to instruct the
compiler to activate its undefined behaviour checker.

Cooking for two months.

* refs/heads/t/ubsan:
  New configure option: --enable-ubsan.

12 months agodaemon: Improve documentation of daemon_set_log_color_or_die().
Andre Noll [Tue, 19 Oct 2021 19:36:54 +0000 (21:36 +0200)]
daemon: Improve documentation of daemon_set_log_color_or_die().

The function receives a severity string, not a log level number.

12 months agoMerge branch 'refs/heads/t/duration-keyword'
Andre Noll [Mon, 22 Nov 2021 13:39:27 +0000 (14:39 +0100)]
Merge branch 'refs/heads/t/duration-keyword'

A single patch which was created already in 2018 but did not propagate
to next and master for no good reason.

Cooking for a month.

* refs/heads/t/duration-keyword:
  mood: Make "duration" a new keyword for the mood grammar.

12 months agoerror.h: Document the error code enumeration.
Andre Noll [Mon, 15 Nov 2021 20:35:25 +0000 (21:35 +0100)]
error.h: Document the error code enumeration.

Doxygen complains about the missing comment.

12 months agocommand.c: Remove unused includes.
Andre Noll [Wed, 20 Oct 2021 21:48:28 +0000 (23:48 +0200)]
command.c: Remove unused includes.

None of these are needed. Tested on Linux, FreeBSD and NetBSD.

12 months agoMerge branch 'refs/heads/t/score-formula'
Andre Noll [Thu, 18 Nov 2021 15:23:20 +0000 (16:23 +0100)]
Merge branch 'refs/heads/t/score-formula'

A single patch which modifies the score function of the audio file
selector to re-scale the two quantities that comprise the score value.

Cooking for six months.

* refs/heads/t/score-formula:
  Rework score formula.

12 months agoMerge branch 'refs/heads/t/list-cleanups'
Andre Noll [Mon, 15 Nov 2021 18:31:41 +0000 (19:31 +0100)]
Merge branch 'refs/heads/t/list-cleanups'

A bunch of simple patches which streamline the macros and inline
functions of list.h which were taken from linux long ago without
adjusting the coding style.

Cooking for three months.

* refs/heads/t/list-cleanups:
  list.h: Convert INIT_LIST_HEAD macro to inline function.
  list.h: Fix parameter doc of iterators.
  list.h: Move list_is_singular() up.
  list.h: Rename argument of list_move().
  list.h: Don't use "new" as an variable name.
  list.h: Open-code LIST_POISON1 and LIST_POISON2.
  list.h: Get rid of internal helpers.
  list.h: Trivial cleanups.
  list.h: Remove unused list_for_each_entry_safe_reverse.

12 months agoi9e: Fix typo: s/ie9/i9e.
Andre Noll [Mon, 18 Oct 2021 20:25:01 +0000 (22:25 +0200)]
i9e: Fix typo: s/ie9/i9e.

It's weird that this was not noticed for so long.

Fixes: e541d7bea7febed8cb9f8a65ae4bd9bdd1b5c8a0
Fixes: 3e3d8e1b48bbd8dbf46adf517c311b5e78dc820f

12 months agoupgrade_db: Add copyright and purpose to upgrade_db.c.
Andre Noll [Tue, 9 Nov 2021 18:11:21 +0000 (19:11 +0100)]
upgrade_db: Add copyright and purpose to upgrade_db.c.

12 months agoRemove unused error code E_OGG_STREAM_FLUSH.
Andre Noll [Sun, 7 Nov 2021 21:39:16 +0000 (22:39 +0100)]
Remove unused error code E_OGG_STREAM_FLUSH.

The last user was removed four years ago.

Fixes: c8466194a4842e319abdedfb38ddc93630e25471

12 months agoweb: Fix typo on documentation page.
Andre Noll [Sun, 7 Nov 2021 15:37:41 +0000 (16:37 +0100)]
web: Fix typo on documentation page.

This typo was introduced four years ago when para_fade was renamed
to para_mixer.

Fixes: 7031f4e380b429e9c0eea1b8e5484dabdbb12ae5

12 months agoMerge branch 'refs/heads/t/writer_improvements'
Andre Noll [Sun, 7 Nov 2021 16:07:24 +0000 (17:07 +0100)]
Merge branch 'refs/heads/t/writer_improvements'

A small set of patches which fix some minor issues, notably a problem
related to the handling of early EOF which affects all paraslash
writers.

* refs/heads/t/writer_improvements:
  alsa_write: Avoid spurious underrun warnings.
  Teach writers to abort gracefully on early EOF.
  alsa_write: Simplify alsa_post_select().

12 months agoweb: Add link to the para_upgrade_db(1) man page.
Andre Noll [Sun, 7 Nov 2021 15:42:50 +0000 (16:42 +0100)]
web: Add link to the para_upgrade_db(1) man page.

Of course this should have been done in the commit which added the
new executable.

Fixes: 56e54c8c75d59a8e5072451941aa56c727d99d26

12 months agoweb: Fix internal link to manual.
Andre Noll [Thu, 4 Nov 2021 20:14:57 +0000 (21:14 +0100)]
web: Fix internal link to manual.

This must have worked at some point...

Looks like it's no good idea to assume that these identifiers are
stable across markdown versions...

12 months agoparaslash 0.6.4
Andre Noll [Thu, 4 Nov 2021 16:15:26 +0000 (17:15 +0100)]
paraslash 0.6.4

Another v0.6 release to mark the destination of the maint branch
which will now be fast-forwarded to this commit. Subsequent merges
of topic branches into master will incorporate incompatible changes,
so from now on the master branch should be regarded as pre-0.7.

12 months agoparaslash 0.6.4 v0.6.4
Andre Noll [Thu, 4 Nov 2021 16:15:26 +0000 (17:15 +0100)]
paraslash 0.6.4

12 months agoMerge branch 'maint'
Andre Noll [Thu, 4 Nov 2021 16:10:34 +0000 (17:10 +0100)]
Merge branch 'maint'

* maint:
  paraslash 0.5.9

12 months agoparaslash 0.5.9
Andre Noll [Thu, 4 Nov 2021 15:56:56 +0000 (16:56 +0100)]
paraslash 0.5.9

A final v0.5-release to record the tip of the maint branch when
paraslash-0.5 became EOL. Shortly after this commit the 0.6.4 release
will be drawn from master, and maint will be fast-forwarded to point
to the v0.6.4 commit.

12 months agocommand.c: Make aux_info_cb() static.
Andre Noll [Thu, 28 Oct 2021 12:44:25 +0000 (14:44 +0200)]
command.c: Make aux_info_cb() static.

It is only used in command.c.

13 months agostring: Remove malloc attribute from para_realloc().
Andre Noll [Tue, 26 Oct 2021 18:19:58 +0000 (20:19 +0200)]
string: Remove malloc attribute from para_realloc().

Quoting from the corresponding section of the gcc-10 manual:

This tells the compiler that a function is 'malloc'-like, i.e.,
that the pointer P returned by the function cannot alias any other
pointer valid when the function returns, and moreover no pointers to
valid objects occur in any storage addressed by P.

Using this attribute can improve optimization. Compiler predicts
that a function with the attribute returns non-null in most cases.
Functions like 'malloc' and 'calloc' have this property because they
return a pointer to uninitialized or zeroed-out storage. However,
functions like 'realloc' do not have this property, as they can return
a pointer to storage containing pointers.

Found by code inspection, the unpached code never caused problems.
Also, the function definition in string.c does not contain the
attribute.

13 months agoalsa_write: Avoid spurious underrun warnings.
Andre Noll [Fri, 27 Aug 2021 13:52:43 +0000 (15:52 +0200)]
alsa_write: Avoid spurious underrun warnings.

Receiving EPIPE does not necessarily mean that an underrun occurred. We
also have to check the pcm state.

13 months agoTeach writers to abort gracefully on early EOF.
Andre Noll [Sun, 5 Sep 2021 18:16:59 +0000 (20:16 +0200)]
Teach writers to abort gracefully on early EOF.

For very short streams it may happen that the receiver and decoder
unregister themselves from the buffer tree before the writer had a
chance to query the information from the decoder which it needs to
open the audio device. This leads to errors such as

Aug 25 14:24:51 schubert (5) get_btr_value: cmd sample_rate: Operation not supported
Aug 25 14:24:51 schubert (5) get_btr_value: cmd channels: Operation not supported
Aug 25 14:24:51 schubert (5) get_btr_value: cmd sample_format: Operation not supported
Aug 25 14:24:51 schubert (4) alsa_init: channels count not available: Invalid argument

This may happen with all receivers, audio formats and writers,
although it is most common with ogg streams.

This commit changes get_btr_sample_rate() and friends to return a
standard error code rather than assuming success. The alsa, ao and
oss writers are patched to check the return value and fail gracefully
if one of these functions fails.

13 months agoalsa_write: Simplify alsa_post_select().
Andre Noll [Sun, 5 Sep 2021 16:57:28 +0000 (18:57 +0200)]
alsa_write: Simplify alsa_post_select().

Clean up the private alsa data structure in alsa_close() and teach
this function to DTRT even if pad->handle is NULL because alsa_init()
failed.

13 months agoafs: Switch to sha256 and change default database path.
Andre Noll [Fri, 13 Mar 2020 18:44:00 +0000 (19:44 +0100)]
afs: Switch to sha256 and change default database path.

This switches the hash column of the audio file table to use sha256
rather than sha1. Due to the preparations implemented in the previous
patches this is a mechanical conversion that could have been performed
by a script.

The only non-trivial thing is the rename of the audio file table
directory, which changes from "audio_files" to "audio-files" because
this is the name the conversion script introduced in the previous
commit picked.

13 months agoAdd para_upgrade_db.
Andre Noll [Fri, 6 Mar 2020 13:26:39 +0000 (14:26 +0100)]
Add para_upgrade_db.

This new executable is required to convert existing paraslash databases
to the new format. Only the layout of the audio file table changes
due to the switch from sha1 to sha256. The command creates and opens
a new audio file table and copies over each row, using an arbitrary
(incorrect) hash value for the values in hash column of the destination
table.

After all rows have been copied in this way, the old table is deleted
and the database directory is renamed from afs-database-0.4 to
afs-database-0.7, which will be the default path in paraslash-0.7.

Subsequent patches will modify para_server to load the database
from the new path and use sha256 instead of sha1 for the hash that
identifies the audio file. The user must then start the thusly patched
para_server and force-add all audio files to correct the hashes. This
approach keeps para_upgrade_db minimal and shortens its running time.

13 months agoUse sha256 for the challenge response.
Andre Noll [Thu, 12 Mar 2020 22:13:17 +0000 (23:13 +0100)]
Use sha256 for the challenge response.

sha1 is broken and should no longer be used. This commit introduces
the new server feature "sha256". It is announced during the handshake
with the client. The client code is patched to detect whether the
server supports the feature and uses sha256 if it does.

This change is backwards compatible. That is, old clients can still
connect to a new server (using sha1). Also new clients can connect
to an old server (and also use sha1 in this case).

13 months agoIntroduce hash2 (sha256).
Andre Noll [Thu, 12 Mar 2020 15:06:12 +0000 (16:06 +0100)]
Introduce hash2 (sha256).

This adds a second hash function which will replace sha1. Both openssl
and libgcrypt support sha256, so it is easy to do. There are no users
of the new functions so far, so this patch has no effect yet.

13 months agoAssume sideband and aes_ctr128 are always supported/requested.
Andre Noll [Thu, 12 Mar 2020 21:08:16 +0000 (22:08 +0100)]
Assume sideband and aes_ctr128 are always supported/requested.

Sideband connections and the AES-based stream cipher have become
mandatory in paraslash-0.6.

The server no longer needs to annouce the feature as 0.6.x clients
request it, regardless of whether it was announced or not. It needs
to still accept the option, though.

On the client side, we don't need to request the features any more
as the server just ignores the request.

13 months agoclient_common: Improve documentation of client_{pre,post}_select().
Andre Noll [Tue, 5 Oct 2021 19:31:06 +0000 (21:31 +0200)]
client_common: Improve documentation of client_{pre,post}_select().

It was outdated and not worded very well. In particular, it referred
to a task pointer although the argument is named "context". Also,
client_open() does not return a "client data structure", so talk
about the effect of that function rather than its return value.

13 months agocompress: Demote log level of clip message.
Andre Noll [Sat, 9 Oct 2021 17:42:52 +0000 (19:42 +0200)]
compress: Demote log level of clip message.

These messages trigger from time to time without having much audible
effect, so they are usually not serious.

13 months agomood: Make "duration" a new keyword for the mood grammar.
Andre Noll [Thu, 30 Aug 2018 20:09:54 +0000 (22:09 +0200)]
mood: Make "duration" a new keyword for the mood grammar.

The new keyword makes it possible to define moods which impose a
constraint on the duration of the admissible files. For example,
the expression

duration < 120 * 1000

selects only files which are shorter than two minutes.

The implementation is pretty simple.

13 months agoRemove ->fd of struct audio file data.
Andre Noll [Sun, 5 Sep 2021 20:38:41 +0000 (22:38 +0200)]
Remove ->fd of struct audio file data.

This structure contains information about the next audio file. It
is stored in a shared memory area, and a reference to this area is
sent through a pipe from the afs process to the server process. The
file descriptor of the next audio file, however, must be passed via
Unix socket magic (SCM_RIGHTS) and thus does not need to be part of
the structure.

Moreover, it's easier to define the afd structure in
open_and_update_audio_file() of aft.c rather than in its caller,
open_next_audio_file() of afs.c, because the caller only needs the
fd of the audio file and the shared memory ID but not the audio file
data structure itself.

Expand the documentation of open_and_update_audio_file() a bit while
at it.

13 months agoaudiod: Use para_malloc().
Andre Noll [Sat, 9 Oct 2021 15:20:06 +0000 (17:20 +0200)]
audiod: Use para_malloc().

If the allocation fails we feed NULL to strncpy(3) which should
result in a segfault. With para_malloc() a clear error message will
be shown instead.

Found by gcc's -fanalyzer option.

14 months agosync_filter: Silence noisy warning.
Andre Noll [Sun, 19 Sep 2021 15:08:40 +0000 (17:08 +0200)]
sync_filter: Silence noisy warning.

It's perfectly fine to fail here due to EOF, so don't print a log
message in this case.

14 months agoafs: Improve error diagnostics if no admissible files are found.
Andre Noll [Sun, 5 Sep 2021 19:36:55 +0000 (21:36 +0200)]
afs: Improve error diagnostics if no admissible files are found.

Currently, the server prints something like the following with
loglevel error:

Sep 05 21:19:42 (4) (31845) open_next_audio_file: key not found in rbtree
Sep 05 21:19:42 (4) (31844) recv_afs_result: afs code: 1, expected: 0
Sep 05 21:19:42 (4) (31844) recv_afs_result: did not receive open fd from afs

This is both cryptic and scary, given that this error is handled by
simply clearing the play bit from the vss status flags.

This commit changes the code to only print one clear log message with
loglevel notice:

Sep 05 21:34:45 (3) (5233) log_statistics: no admissible files

14 months agoplay: Avoid gcc warning when compiling without readline.
Andre Noll [Tue, 28 Sep 2021 20:05:45 +0000 (22:05 +0200)]
play: Avoid gcc warning when compiling without readline.

The EXPORT_PLAY_CMD_HANDLER() macro is only used when READLINE support
is enabled, which leads to

play.c:112: warning: macro "EXPORT_PLAY_CMD_HANDLER" is not used [-Wunused-macros]

Define the macro only when HAVE_READLINE is defined by moving down
the definition to the first user, com_quit().

14 months agoserver: Wait for command handler exit also when afs dies.
Andre Noll [Tue, 21 Sep 2021 12:09:39 +0000 (14:09 +0200)]
server: Wait for command handler exit also when afs dies.

When para_server is running in foreground mode in a terminal session,
and gets signalled by hitting CTRL+C, it is unspecified whether the
server or the afs process receive the resulting SIGINT first. It may
even happen that the afs process dies first, and that the server sees
the resulting SIGCHLD *before* the SIGINT.

In this case we currently don't wait for the command handlers to exit
but proceed right away with the shutdown, closing the signal pipe and
destroying the shared memory area which contains the mmd structure.
This leads to error messages on shutdown such as

Sep 21 12:38:18 (5) (29166) para_semop: semaphore set 12648470 was removed
Sep 21 12:38:18 (6) (29166) para_semop: fatal semop error Invalid argument: pid 29166
Sep 21 12:38:18 (6) (29161) generic_signal_handler: Bad file descriptor
Sep 21 12:38:18 (6) (29164) para_semop: fatal semop error Invalid argument: pid 29164
Sep 21 12:38:18 (6) (29165) command_handler_sighandler: terminating on signal 15
Sep 21 12:38:18 (6) (29165) para_semop: fatal semop error Invalid argument: pid 29165

This commit avoids the issue by letting the server wait for all
its children also in the SIGCHILD case when we exit because the afs
process has terminated.

14 months agovss: Avoid double free on exit.
Andre Noll [Mon, 20 Sep 2021 18:42:37 +0000 (20:42 +0200)]
vss: Avoid double free on exit.

When para_server fails to receive the fd for the next audio file,
the memory pointed to by mmd->afd.afhi.chunk_table is freed but the
pointer is not set to NULL. If the failure was due to server and afs
receiving SIGINT, the subsequent signal handling code might attempt
to free the chunk table again. This double free error is detected by
glibc, which results resulting in messages such as

free(): invalid size
Aborted

Fortunately, this is easy to fix.

14 months agovss: Handle empty chunk groups gracefully.
Andre Noll [Mon, 20 Sep 2021 17:15:12 +0000 (19:15 +0200)]
vss: Handle empty chunk groups gracefully.

The assertion in compute_group_size() triggers if the stream is
positioned right at the end of the file when the next chunk group
starts. This was observed with an mp3 file and the udp sender when
pausing and restarting the stream at the end of the file.

Fix this braino by returning EOF instead of aborting if the next
chunk group happens to be empty.

14 months agoupd sender: Don't send FEC EOF from command handler context.
Andre Noll [Mon, 20 Sep 2021 18:28:42 +0000 (20:28 +0200)]
upd sender: Don't send FEC EOF from command handler context.

Without this, the EOF packet might be sent twice: once by the command
handler which stopped the stream and once by the server process. This
does not hurt, but results in additional unnecessary network traffic,
so return early from udp_close_target() when we're running in command
handler context.

14 months agoNew configure option: --enable-ubsan.
Andre Noll [Sat, 15 Jul 2017 12:34:29 +0000 (14:34 +0200)]
New configure option: --enable-ubsan.

If the new option is given, -fsanitize=undefined is passed to the
compiler to enable the undefined behavior detector.

We don't check whether this option is supported because the configure
option is off by default.

15 months agoafh: Avoid memory leak at exit.
Andre Noll [Thu, 26 Aug 2021 17:31:25 +0000 (19:31 +0200)]
afh: Avoid memory leak at exit.

We missed to free the lopsub parse result. This is not a real leak
because it's a one-off allocation and we are about to exit anyway. It's
worth to fix nevertheless, though, because with the patch applied,
valgrind says "no leaks are possible". Hence any other output means
we have introduced a new memory leak.

15 months agoaac_afh: Fix check of return value of mp4ff_meta_update().
Andre Noll [Fri, 13 Aug 2021 19:47:47 +0000 (21:47 +0200)]
aac_afh: Fix check of return value of mp4ff_meta_update().

This function returns zero on failure, and one on success, so the
current check for a negative return value is incorrect. The call to
mp4ff_meta_get_by_index() suffers from the same mistake.