paraslash.git
2 years agoparaslash 0.4.14 v0.4.14
Andre Noll [Sat, 31 Dec 2016 19:40:51 +0000 (20:40 +0100)]
paraslash 0.4.14

3 years agoclient: Fix lsatt completer.
Andre Noll [Tue, 29 Mar 2016 16:16:33 +0000 (16:16 +0000)]
client: Fix lsatt completer.

The lsatt command does not take any non-option arguments. So it is wrong
to complete on the available attributes.

3 years agoplaylist: Do not update score if no playlist is open.
Andre Noll [Mon, 21 Mar 2016 19:51:08 +0000 (19:51 +0000)]
playlist: Do not update score if no playlist is open.

This bug caused the score of the current audio file to be updated
twice in mood mode since both the mood event handler and playlist
event handler honor afs events and update the score.

We fix the bug by moving the check whether a playlist is open to the
top of the playlist event handler, similar to the check in the mood
event handler of mood.c. If no playlist is open, we now return early
from the playlist event handler, ignoring all afs events.

3 years agoAvoid member access within misaligned address for ancillary data buffer.
Andre Noll [Wed, 16 Mar 2016 22:05:56 +0000 (23:05 +0100)]
Avoid member access within misaligned address for ancillary data buffer.

For glibc-2.23, the CMSG_FIRSTHDR macro is defined as

#define CMSG_FIRSTHDR(mhdr) \
  ((size_t) (mhdr)->msg_controllen >= sizeof (struct cmsghdr)                 \
   ? (struct cmsghdr *) (mhdr)->msg_control : (struct cmsghdr *) 0)

In recv_cred_buffer(), pass_afd() and dispose_fds() the on-stack
ancillary data buffer is not necessarily aligned. The pointer is
cast to struct cmsghdr *, then dereferenced, resulting in undefined
behaviour due to the lack of alignment.

This patch asks the compiler to align the ancillary data buffers.

3 years agomood.c: Avoid overflow in update_quadratic_deviation().
Andre Noll [Wed, 16 Mar 2016 21:48:54 +0000 (22:48 +0100)]
mood.c: Avoid overflow in update_quadratic_deviation().

The last multiplication of the expression returned can overflow. This
patch replaces the expression by an equivalent one which avoids to
multiply large numbers.

3 years agomood.c: Avoid integer underflow.
Andre Noll [Tue, 15 Mar 2016 19:48:12 +0000 (20:48 +0100)]
mood.c: Avoid integer underflow.

The three variables x, s, n are all of unsigned type, and the
subtraction x - s / n may underflow, resulting in a very large positive
value. This should not matter since we square the difference, but on
the other hand, the underflow can easily be avoided. This patch uses
a temporary variable to do so.

3 years agomood.c: Avoid integer overflow.
Andre Noll [Tue, 15 Mar 2016 19:41:37 +0000 (20:41 +0100)]
mood.c: Avoid integer overflow.

The quadratic deviation qd can be very large, causing the
multiplication n * qd to overflow. The new code avoids the
multiplication at the cost of two calls to int_sqrt() instead of one.

3 years agoattribute: Avoid shifting 32 bit integers.
Andre Noll [Sun, 7 Feb 2016 16:33:15 +0000 (17:33 +0100)]
attribute: Avoid shifting 32 bit integers.

Doing "1UL << i" is wrong here, because the constant "1UL" is 32 bit
on 32 bit systems and we definitely need 64 bit quantities for the
attribute mask.

Fix this by using an uint64_t variable instead.

4 years agormblob: Generate proper BLOB_REMOVE_EVENT.
Andre Noll [Fri, 4 Sep 2015 07:24:12 +0000 (09:24 +0200)]
rmblob: Generate proper BLOB_REMOVE_EVENT.

com_rmblob_callback() of blob.c is responsible for removing blobs from
any of the four blob tables (images, lyrics, moods, playlists). After
a blob has been removed, the function generates an AFS event to tell
the other tables about the removal.

However, the call to afs_event() passes the event identifier
BLOB_RENAME rather than BLOB_REMOVE to the event handlers, which is
clearly incorrect. Fortunately, this does not matter because the only
event handler which cares about blob events is the one in mood.c,
and this handler treats the two events identically. Nevertheless,
it's is a bug that should be fixed.

This bug was introduced eight years ago when event handling was
introduced: commit 02d818d9 (Complete afs event handling).

4 years agolsatt: Fix sort order.
Andre Noll [Fri, 28 Aug 2015 20:33:57 +0000 (22:33 +0200)]
lsatt: Fix sort order.

The -i option had the opposite effect of what the documentation
says. That is, lsatt -i sorted the attribute list by name while the
default was to sort by id. This patch reverts the logic in com_lsatt()
to let the implementation match the documentation.

4 years agoserver: Avoid segfault in com_sender().
Andre Noll [Sat, 22 Aug 2015 14:21:53 +0000 (16:21 +0200)]
server: Avoid segfault in com_sender().

If exactly one argument is given to the sender command, and this
argument is the name of an existing sender, the sender command
segfaults due to the NULL pointer dereference. The problem is an
off-by-one bug in the check for the number of arguments.

This patch makes sure we never dereference argv[2] if it is NULL.

4 years agomood: Clear the score table on attribute changes.
Andre Noll [Sat, 20 Jun 2015 18:26:33 +0000 (20:26 +0200)]
mood: Clear the score table on attribute changes.

Although the comment in mood.c correctly states that the score table
must be cleared and reloaded if an attribute is added, removed or
renamed, the mood event handler misses to clear the score table.
Therefore, after reload_current_mood() has reloaded the mood object,
insertion of admissible files fails with

afs_event: table moods, event 0: key already exists in rbtree

This results in an unmodified set of admissible files.

The only other caller of reload_current_mood() clears the score table
right before the function call, so the easiest fix is to move the
call to clear_score_table() into reload_current_mood().

4 years agoFix help text of lsblob commands.
Andre Noll [Sun, 10 May 2015 17:58:41 +0000 (19:58 +0200)]
Fix help text of lsblob commands.

The help text of the lsblob commands talked about attributes but
of course the commands list arbitrary blobs (images, lyrics, moods,
playlists).

This bug was introduced eight years ago in commit 4b80a041 (afs.cmd:
Update help output).

4 years agogcrypt: Initialize key size of public ASN keys.
Andre Noll [Wed, 18 Feb 2015 15:40:40 +0000 (16:40 +0100)]
gcrypt: Initialize key size of public ASN keys.

This bug could make applications acccept short keys. Not a serious
problem since (a) ASN keys are deprecated in favor of ssh keys and
(b) encrypting a 256 byte buffer fails anyways on short keys. Let's
fix it anyway.

4 years agoaft: Generate a remove event when adding duplicate files.
Andre Noll [Wed, 24 Dec 2014 15:10:43 +0000 (15:10 +0000)]
aft: Generate a remove event when adding duplicate files.

Consider the following scenario:

add /foo.mp3
add /bar.mp3 # assume foo and bar differ
cp /foo.mp3 /bar.mp3
add /bar.mp3

In the last add command, the path being added (/bar.mp3) already exists
in the audio file table, and its hash also exists for a different path
(/foo.mp3).

The code in aft.c is smart enough to detect this. It first removes
the existing entry for /bar.mp3 and then considers the addition of
/bar.mp3 as a file rename /foo.mp3 -> /bar.mp3.

Unfortunately, we miss to generate the remove event in this case. This
patch should fix it. It also improves the message a bit.

4 years agocommand.c: Add missing items to EMPTY_STATUS_ITEMS.
Andre Noll [Tue, 23 Dec 2014 01:13:25 +0000 (01:13 +0000)]
command.c: Add missing items to EMPTY_STATUS_ITEMS.

Without this, para_gui shows stale information when no audio file
is open.

Add a doxygen comment to the macro while at it.

4 years agoaft.c: Don't call osl_close_disk_object() on failure.
Andre Noll [Sun, 21 Dec 2014 14:35:19 +0000 (14:35 +0000)]
aft.c: Don't call osl_close_disk_object() on failure.

If osl_open_disk_object() fails, we branch to the "err" label where
osl_close_disk_object() is called although no disk object exists. Fix
this by simply returning the osl error code in this case.

4 years agoAdd missing osl() wrapper calls.
Andre Noll [Sun, 21 Dec 2014 14:28:34 +0000 (14:28 +0000)]
Add missing osl() wrapper calls.

Some calls to osl library functions were missing the osl() wrapper
which transforms the osl error code into a paraslash error code.
This results in an incorrect error message on failure or worse.

print_chunk_table() and open_and_update_audio_file() of aft.c are
affected as well as the xxx_create() functions of blob.c.

4 years agofd.c: Add missing va_end().
Andre Noll [Thu, 1 Jan 2015 17:32:36 +0000 (17:32 +0000)]
fd.c: Add missing va_end().

This bug was introduced three years ago in commit 3e6bba77.

Found by cppcheck.

5 years agoplay: Fix next and prev on singular playlists.
Andre Noll [Fri, 5 Sep 2014 23:20:55 +0000 (01:20 +0200)]
play: Fix next and prev on singular playlists.

The next and prev commands do not reset the file position if
the playlist consists of one file only. This fixes it.

5 years agorecv: Don't segfault on invalid chunk values.
Andre Noll [Sun, 11 May 2014 19:47:03 +0000 (21:47 +0200)]
recv: Don't segfault on invalid chunk values.

If recv->open() fails, para_recv segfaults after

==10129== Conditional jump or move depends on uninitialised value(s)
==10129==    at 0x41BA18A: vfprintf (vfprintf.c:1634)
==10129==    by 0x41BE3D8: buffered_vfprintf (vfprintf.c:2311)
==10129==    by 0x41B975C: vfprintf (vfprintf.c:1289)
==10129==    by 0x804C6AF: stderr_log (in /home/maan/scm/paraslash/para_recv)
==10129==    by 0x804A193: main (recv.c:114)

This happens for example with the afh_receiver if the given start
chunk is bigger than the end chunk, e.g. when para_recv is executed as

./para_recv -r 'afh -b -80 -e -89 -f foo.opus'

The reason for the bug is we zero-out the stdout structure too late.
Fix this by initializing the stdout task structure upfront rather
than using memset().

This bug was introduced a year ago in commit b01605d7 (Avoid unwanted
log messages during startup) which moved down the call to memset().

5 years agogui: Don't ignore SIGHUP.
Andre Noll [Mon, 17 Feb 2014 21:18:43 +0000 (22:18 +0100)]
gui: Don't ignore SIGHUP.

It results in para_gui eating 100% CPU if the terminal window is
closed. Not ignoring the signal makes para_gui die instead.

This bug was present since "day 1".

6 years agoconfigure.ac: Fix definition of recv_ldflags.
Andre Noll [Sun, 1 Dec 2013 00:41:16 +0000 (01:41 +0100)]
configure.ac: Fix definition of recv_ldflags.

This cut+paste typo in the libflac section causes the build system
to link para_recv against the wrong set of libraries.

6 years agoMerge branch 't/for-maint' into maint
Andre Noll [Sun, 17 Nov 2013 15:49:35 +0000 (16:49 +0100)]
Merge branch 't/for-maint' into maint

A single fix that was cooking for some weeks.

0a7cad build: Fix --with-opus-libs.

6 years agobuild: Fix --with-opus-libs.
Andre Noll [Sat, 28 Sep 2013 14:27:56 +0000 (16:27 +0200)]
build: Fix --with-opus-libs.

Due to a silly cut & paste bug this option sets $speex_libs rather
than $opus_libs. Since the assignment comes after speex detection,
the option is effectively a no-op.

6 years agoplay: Fix prev command.
Andre Noll [Sun, 22 Sep 2013 14:45:10 +0000 (16:45 +0200)]
play: Fix prev command.

The command handler for com_prev() stores the number of the
previous (valid) file in ->next_file of struct play_task and sets
->rq to CRT_FILE_CHANGE. The real work is supposed to be done in
load_next_file() which is called from the main post_select() function.

However, load_next_file() ignores ->next_file and computes the number
of the next file itself. Hence the "prev" command acts exactly as
"next", i.e. it selects the next rather than the previous file.

Fix this by ignoring ->next_file only if ->rq is CRT_NONE, which
indicates a normal end of file condition.

6 years agobash_completion: Run client/audioc from PATH.
Andre Noll [Wed, 11 Sep 2013 00:46:26 +0000 (00:46 +0000)]
bash_completion: Run client/audioc from PATH.

This way it works for everybody.

6 years agoconfigure: Print opus audio file handler if opus lib was found.
Andre Noll [Wed, 4 Sep 2013 04:35:38 +0000 (04:35 +0000)]
configure: Print opus audio file handler if opus lib was found.

The variable is called $audio_format_handlers rather than
$server_audio_formats. The value is only used to display the supported
audio formats, so this bug is rather benign.

6 years agobtr: splice-out fix.
Andre Noll [Fri, 6 Sep 2013 23:21:44 +0000 (23:21 +0000)]
btr: splice-out fix.

When splicing out a node we set the ->parent pointer of all child
nodes to the parent of the given node, and move each child to the
->children list of the parent.

Except when there is no parent. If the given node was a root node
(or an internal node whose parent vanished), we leave the ->children
list untouched. In this case the assertion a few lines later triggers
and aborts the program. Fix this by removing the nodes from the list.
Such nodes have become root nodes themselves, so they should not be
on any list of children.

6 years agofecdec: Avoid fecdec output buffer overruns.
Andre Noll [Sun, 1 Sep 2013 17:48:46 +0000 (17:48 +0000)]
fecdec: Avoid fecdec output buffer overruns.

The size of the buffer tree pool of the amp filter is 64K, which
is smaller than BTRN_MAX_PENDING of buffer_tree.c (96K). The latter
value is used in btr_node_status() as follows. If input for a buffer
tree node is available and there is less than BTRN_MAX_PENDING bytes
in the output queue of the node, the function returns 1 to indicate
that the node should continue to process its input.

This can result in a fatal error condition when the buffer tree pool
fills up completely. Avoid this by increasing the pool size to 128K.

6 years agovss: Avoid assertion "i == g->num_header_slices - 1".
Andre Noll [Sun, 1 Sep 2013 15:54:25 +0000 (15:54 +0000)]
vss: Avoid assertion "i == g->num_header_slices - 1".

This fixes a very old bug in the timing code for fec slices. Depending
on the number of bytes to submit and on the number of slices of the
current fec group it may happen that we reserve more slices than
needed for the audio file header. This is not a problem per se, but the
above assertion hits in this case and aborts para_server.

Fix this by removing the assertion, and make sure that all reserved
header slices are properly initialized.

6 years agoconfigure: Fix restoring LDFLAGS.
Andre Noll [Thu, 29 Aug 2013 22:20:30 +0000 (22:20 +0000)]
configure: Fix restoring LDFLAGS.

The configure script saves the values of CPPFLAGS, CFLAGS and LDLFAGS
before each check and restores the old values afterwards. However,
for LDLFAGS this was broken at several places, since we used LD_FLAGS
(with an underscore) for saving the value and LDFLAGS for restoring it
(or vice versa).

This bug had the effect that a user-specified value of LDFLAGS was
ignored in some tests because LDFLAGS was restored to the empty
value LD_FLAGS during an earlier test.

This patch changes all occurrences of LD_FLAGS in configure.ac to
LDFLAGS.

6 years agoserver/gcrypt: Fix sending the empty status items.
Andre Noll [Mon, 29 Jul 2013 22:44:37 +0000 (00:44 +0200)]
server/gcrypt: Fix sending the empty status items.

When para_server enters the "stopped" state, it needs to inform all
clients which are executing the stat command that no audio file is
currently available.

To this aim the stat command handler calls empty_status_items(), which
creates a buffer containing empty values for most status items. This
function tries to be smart by computing the buffer only once. It
saves a reference in a static variable so that on subsequent calls
it can simply return the same buffer.

This works fine when para_server is compiled against the openssl
crypto library. However, it fails when libgcrypt is used because with
libgcrypt the send buffer is encrypted in-place. Hence on subsequent
calls the already encrypted buffer will be encrypted again, resulting
in garbage being sent to the client.

This patch avoids this bug by using a fresh buffer each time the
empty status items are sent.

6 years agonew codename, reset version to git
Andre Noll [Mon, 29 Jul 2013 13:08:32 +0000 (15:08 +0200)]
new codename, reset version to git

6 years agoparaslash 0.4.13 v0.4.13
Andre Noll [Mon, 29 Jul 2013 12:59:50 +0000 (14:59 +0200)]
paraslash 0.4.13

6 years agotime.c: Make declarations match definitions.
Andre Noll [Sat, 6 Jul 2013 13:02:10 +0000 (15:02 +0200)]
time.c: Make declarations match definitions.

Also fix some whitespace issues.

6 years agoAdd two missing GPL and doxygen file headers.
Andre Noll [Sun, 21 Jul 2013 14:40:56 +0000 (16:40 +0200)]
Add two missing GPL and doxygen file headers.

It's a pity that doxygen doesn't warn about this.

6 years agosched.h: Remove outdated comment.
Andre Noll [Sun, 21 Jul 2013 14:30:23 +0000 (16:30 +0200)]
sched.h: Remove outdated comment.

The term "these functions" is undefined here, and setting t->error
is deprecated.  The post_select() functions are supposed to return
an error code instead, but this is already mentioned further down in
the file, so let's just remove the comment.

6 years agoFix documentation of filter->post-select().
Andre Noll [Sun, 21 Jul 2013 14:25:10 +0000 (16:25 +0200)]
Fix documentation of filter->post-select().

The ->post_select() methods return an integer these days and should
*not* set the ->error field of the task struct directly.

6 years agoipc.c: Fix typo in comment.
Andre Noll [Sat, 20 Jul 2013 19:07:06 +0000 (21:07 +0200)]
ipc.c: Fix typo in comment.

6 years agoaft.c: Remove a stray "*" in a comment.
Andre Noll [Sun, 14 Jul 2013 11:13:03 +0000 (13:13 +0200)]
aft.c: Remove a stray "*" in a comment.

6 years agoMove com_select_callback() closer to com_select().
Andre Noll [Tue, 16 Jul 2013 05:38:33 +0000 (07:38 +0200)]
Move com_select_callback() closer to com_select().

It's considered good style to have the command handler and it's
callback next to each other. No actual changes.

6 years agobuffer_tree.c: Add documentation of btr_parent().
Andre Noll [Sat, 6 Jul 2013 18:25:17 +0000 (20:25 +0200)]
buffer_tree.c: Add documentation of btr_parent().

This function was introduced in commit 37e0dfe7 (check_wav: Ask parent
nodes before falling back to defaults) without documentation.

6 years agoggo.c: Document return value of printf_or_die().
Andre Noll [Sat, 6 Jul 2013 18:23:04 +0000 (20:23 +0200)]
ggo.c: Document return value of printf_or_die().

Commit b59a3c41 (filter: Wrap lines in the available filter list)
changed the return value of this function from void to int but missed
to update the documentation.

6 years agoMood: Document return value of moods_event_handler().
Andre Noll [Sat, 6 Jul 2013 18:21:19 +0000 (20:21 +0200)]
Mood: Document return value of moods_event_handler().

Just to make doxygen happy.

6 years agonet.c: Add documentation of flowopt_new().
Andre Noll [Sat, 6 Jul 2013 18:20:41 +0000 (20:20 +0200)]
net.c: Add documentation of flowopt_new().

It's not obvious that the returned structure does not need to
be freed by the caller, so let's document this fact.

6 years agonet: Remove unused flowopt_add_bool().
Andre Noll [Sat, 6 Jul 2013 18:00:57 +0000 (20:00 +0200)]
net: Remove unused flowopt_add_bool().

6 years agorecv.h: Fix comment indentation.
Andre Noll [Sat, 6 Jul 2013 20:02:56 +0000 (22:02 +0200)]
recv.h: Fix comment indentation.

6 years agoRemove some duplicate semicolons.
Andre Noll [Sun, 7 Jul 2013 16:47:36 +0000 (18:47 +0200)]
Remove some duplicate semicolons.

6 years agoRemove some unused error codes.
Andre Noll [Sun, 7 Jul 2013 19:14:21 +0000 (21:14 +0200)]
Remove some unused error codes.

Quite a few error codes have become unused, but the stale entries
of error.h remained. Unfortunately, we have no way to spot these
automatically.

6 years agoDoxyfile: Predefine HAVE_CLOCK_GETTIME.
Andre Noll [Sat, 6 Jul 2013 18:26:59 +0000 (20:26 +0200)]
Doxyfile: Predefine HAVE_CLOCK_GETTIME.

Fixes the following doxygen warning:

time.c:227: Warning: include file sys/time.h not found, perhaps you forgot to add its directory to INCLUDE_PATH?

6 years agoaudiod: Close filters in reverse order.
Andre Noll [Sat, 6 Jul 2013 21:00:14 +0000 (23:00 +0200)]
audiod: Close filters in reverse order.

Otherwise, a subsequent filter might refer to a buffer reference of
the previous filter, accessing freed memory. This is not a bug right
now, as audiod only closes filters after the full buffer tree has
become inactive, i.e. after all buffer tree nodes have unregistered
their task. It's cleaner to reverse the loop though, and it has no
additional cost.

6 years agoopusdec: Get rid of opusdec_pre_select().
Andre Noll [Sun, 7 Jul 2013 15:59:07 +0000 (17:59 +0200)]
opusdec: Get rid of opusdec_pre_select().

The generic one should do just fine and the additional 100ms timeout
should not be necessary.

6 years agoopusdec: avoid __STDC_VERSION__ warning.
Andre Noll [Fri, 12 Jul 2013 14:08:41 +0000 (16:08 +0200)]
opusdec: avoid __STDC_VERSION__ warning.

opusdec_filter.c indirectly includes opus_types.h which checks whether
__STDC_VERSION__ >= 199901L. However, at least some gcc versions
don't define __STDC_VERSION__ which results in

warning: "__STDC_VERSION__" is not defined

For example, this happens on Ubuntu lucid, which ships gcc-4.4.3.

This patch gets rid of the warning by defining __STDC_VERSION__
if necessary prior to including the opus_types header.

6 years agoafh: Initialize audio format handlers only once.
Andre Noll [Sun, 7 Jul 2013 11:22:48 +0000 (13:22 +0200)]
afh: Initialize audio format handlers only once.

afc.c calls afh_init() twice. This does not really hurt as this
function is idempotent, but it causes the initialization log messages
to be printed twice.

This gets rid of the additional call to afh_init() which crept in
in commit 042767ce (Use self-made help to avoid recompilations on
version changes).

6 years agomp3_afh: Use symbolic constants for id3 tag identifiers.
Andre Noll [Sun, 7 Jul 2013 08:44:50 +0000 (10:44 +0200)]
mp3_afh: Use symbolic constants for id3 tag identifiers.

6 years agoMerge branch 't/opus'
Andre Noll [Sat, 6 Jul 2013 00:24:48 +0000 (02:24 +0200)]
Merge branch 't/opus'

Was cooking for two weeks with no problems.

273756 The opus decoder.
7007ae The opus audio format handler.
8bcf75 ogg/opus: Infrastructure.
53133e speex: Don't export spx_ctl().

6 years agoMerge branch 't/versioning_improvements'
Andre Noll [Mon, 1 Jul 2013 19:06:48 +0000 (21:06 +0200)]
Merge branch 't/versioning_improvements'

Cooking for ~2 weeks.

15e99a version.c: Mark version_git() as const.
9bdebf Remove CODENAME macro.
be2f6b gui: Use version_single_line().
d60dae Improve man page layout.
5dbc9a afh/play: Include supported audio formats in help output.
042767 Use self-made help to avoid recompilations on version changes.
48f1fc Provide "purpose" texts.
aa74a9 Revamp ggo help.
9f7a49 afh_recv: Replace ggo text section by description.
b59e0e Make gengetopt descriptions work.
06b3e7 Introduce version.c to limit recompilation on version changes.
75feac Make all commands print git version and improve version string.
b01605 Avoid unwanted log messages during startup.
625fdb Don't check return value of command line parsers unnecessarily.
533b03 Build receivers, filters and writers without -h and -V support.
b59a3c filter: Wrap lines in the available filter list.
9e56d3 audioc: Print config file errors.
e5264d doc: Rewrite udp sender description
5ec373 client: Fix typo in comment.
6d5159 client: Remove duplicate include.

6 years agoversion.c: Mark version_git() as const.
Andre Noll [Mon, 1 Jul 2013 19:02:56 +0000 (21:02 +0200)]
version.c: Mark version_git() as const.

Fixes the following warning on gcc-4.8.1:

version.c:13:13: warning: function might be candidate for attribute 'const' [-Wsuggest-attribute=const]

6 years agoi9e: Fix memory leak in clear_bottom_line().
Andre Noll [Sat, 22 Jun 2013 15:20:15 +0000 (17:20 +0200)]
i9e: Fix memory leak in clear_bottom_line().

In clear_bottom_line() we call readline's rl_copy_text() which
allocates a buffer for the given range of the current input line. But
we never free this buffer, which results in a memory leak. This patch
plugs the leak.

6 years agoHandle empty command lines properly.
Andre Noll [Sat, 22 Jun 2013 14:59:04 +0000 (16:59 +0200)]
Handle empty command lines properly.

Both para_client and para_audioc create an argument vector from the
given command line using create_argv(). If the command line contains
only whitespace characters, this vector has length zero, and argv[0]
is NULL.

We missed to check for this at at least three places in audioc.c,
client.c and audiod_command.c, which resulted in crashes due to NULL
pointer dereferences or failed assertions.

These bugs can easily be triggered by starting para_client or
para_audioc in interactive mode, entering a single space character
and hitting return.

This patch adds the missing checks to prevent the crashes.

6 years agoMerge branch 't/interactive'
Andre Noll [Sat, 22 Jun 2013 13:58:29 +0000 (15:58 +0200)]
Merge branch 't/interactive'

Was cooking for almost two weeks.

f8931d Simplify i9e_line_handler.
f99dbe i9e: Fix a prompt display issue
cf39e4 audioc: Abstract out connection code.

6 years agoImprove text on the download web page.
Andre Noll [Sun, 19 May 2013 12:53:31 +0000 (14:53 +0200)]
Improve text on the download web page.

Enumerate the possible ways to download paraslash and explain when
each option is most suitable.

6 years agogui: Fix off-by-one in add_spaces().
Andre Noll [Sun, 16 Jun 2013 12:22:06 +0000 (14:22 +0200)]
gui: Fix off-by-one in add_spaces().

Commit e90c6c0a (Speed up add_spaces().) changed add_spaces() to print
space characters in chunks rather than one at a time. It introduced
space[], an array of whitespace characters, which is written entirely
if there are more spaces to print than the size of the array. However,
in the calculation of how much was printed so far, we missed the fact
that sizeof(space) includes the terminating NULL byte, so this number
is in fact the number of space characters *plus one*.

Consequently, we printed too few space characters. This resulted in
parts of the previous string still being visible in the top window
of para_gui.

6 years agoMerge branch 't/compress'
Andre Noll [Sun, 16 Jun 2013 11:09:20 +0000 (13:09 +0200)]
Merge branch 't/compress'

Well tested an cooked.

0a6c0e compress: Further optimize inner loop.
be761d compress: Remove log statement in inner loop.
15a54f compress: Avoid PARA_ABS and PARA_MAX in inner loop.
28c03c compress: Compile with -O3.

6 years agoMerge branch 't/oss_error_message_fix'
Andre Noll [Thu, 13 Jun 2013 16:34:04 +0000 (18:34 +0200)]
Merge branch 't/oss_error_message_fix'

9e32ff oss mixer: Improve error message.

6 years agoRemove CODENAME macro.
Andre Noll [Mon, 20 May 2013 20:30:53 +0000 (22:30 +0200)]
Remove CODENAME macro.

This macro is no longer used, so we may remove it from Makefile.in.

6 years agogui: Use version_single_line().
Andre Noll [Mon, 20 May 2013 20:12:40 +0000 (22:12 +0200)]
gui: Use version_single_line().

No need to reinvent the wheel.

6 years agoImprove man page layout.
Andre Noll [Mon, 20 May 2013 17:29:58 +0000 (19:29 +0200)]
Improve man page layout.

This changes the output of --help, --detailed-help and --version
so that the man pages generated from this output by help2man look
much nicer.

Specifically, name and purposes are printed in a single line and
the version text now contains also build data, build system and the
compiler version.

Also the codename is no longer printed in version_single_line(),
since this caused the top and bottom labels of the man pages to
look quite ugly.

6 years agoafh/play: Include supported audio formats in help output.
Andre Noll [Sun, 7 Apr 2013 00:24:13 +0000 (00:24 +0000)]
afh/play: Include supported audio formats in help output.

This is useful information, and it is simple to do, so include it in
the help.

6 years agoUse self-made help to avoid recompilations on version changes.
Andre Noll [Fri, 12 Apr 2013 12:24:54 +0000 (14:24 +0200)]
Use self-made help to avoid recompilations on version changes.

This provides print_help() for all remaining commands so that we can
get rid of the gengetopt generated help output completely.

Hence the *.cmdline.[ch] files no longer depend on git-version.h
which saves a lot of recompilations.

6 years agoProvide "purpose" texts.
Andre Noll [Sun, 7 Apr 2013 02:31:51 +0000 (02:31 +0000)]
Provide "purpose" texts.

This text will be printed right after the program name in the --help
output. For para_afh, the inline text becomes redundant, so this text
is removed.

Each executable gets a purpose text, and for receivers, filters,
writers the purpose will be printed as part of the help output of
para_audiod, para_filter. para_recv and para_write.

6 years agoRevamp ggo help.
Andre Noll [Fri, 12 Apr 2013 11:54:37 +0000 (13:54 +0200)]
Revamp ggo help.

This adds usage and description fields to struct ggo_help and
changes ggo_print_help() to optionally print these.

The boolean detailed_help flag of ggo_print_help() is replaced by a
bitmask which lets the caller specify what to print. Four pre-defined
masks are used to print the normal help, the detailed help, the help
for modules (receivers, filters, writers) and the detailed module help.

The new macro DEFINE_GGO_HELP can be employed to create a struct
ggo_help from a gengetopt structure.

6 years agoafh_recv: Replace ggo text section by description.
Andre Noll [Sat, 6 Apr 2013 22:27:51 +0000 (22:27 +0000)]
afh_recv: Replace ggo text section by description.

This way the text shows up only when --detailed-help is given.

6 years agoMake gengetopt descriptions work.
Andre Noll [Fri, 12 Apr 2013 11:54:12 +0000 (13:54 +0200)]
Make gengetopt descriptions work.

With gengetopt's text and description options it is not possible to
print a text only when --detailed-help was given, but not when only
--help is given.

Moreover, headers generated with old versions of gengetopt do not
export the description text.  However, declaring it unconditionally
causes compiler warnings on new systems.

To circumvent these problems, we introduce a new test for configure
which checks whether the description string is declared. If it is not,
we simply append the declaration to the *.cmdline.h files.

This change allows to move the description string of para_play to
the gengetopt source file.

6 years agoIntroduce version.c to limit recompilation on version changes.
Andre Noll [Fri, 12 Apr 2013 12:23:45 +0000 (14:23 +0200)]
Introduce version.c to limit recompilation on version changes.

Currently version.h includes git-version.h which changes whenever
a different commit is checked out or the working tree becomes dirty
because a file has been modified. Consequently, all .c files that
include this header must be recompiled in this case.

To limit the number of recompilations, this commit introduces
version.c, which contains functions that return the various types of
the version string. It is now the only file that includes version.h,
so only version.o must be rebuilt if git-version.h changes.

This also adds the build date, OS and CC version to the version output.

6 years agoMake all commands print git version and improve version string.
Andre Noll [Tue, 26 Mar 2013 23:30:49 +0000 (23:30 +0000)]
Make all commands print git version and improve version string.

Currently the format of the first line of the help output varies
between executables, for example:

para_afh git (0.4.12.12.g15a4: spectral gravity)
para_audioc git

This difference comes from the fact that para_afh has its own
->print_help method which uses the VERSION_SINGLE_LINE macro of
version.h while para_audioc relies on gengetopt's help output.

The latter uses the make variable PACKAGE_VERSION which gets
initialized at configure time through the second argument in AC_INIT
of configure.ac. This value is either the version number for or the
string "git".

It's a good thing to have the abbreviated git version encoded in
all executables, so this commit changes the argument of gengetopt's
--set-version to the git version string including the codename. With
the patch applied, the output of all commands looks like this if
--version was given:

para_filter 0.4.12.12.g11d7 (spectral gravity)

To make sure things stay consistent, the patch introduces the
VERSION_SINGLE_LINE macro and changes all commands to use it instead
of open coding the version string.

6 years agoAvoid unwanted log messages during startup.
Andre Noll [Mon, 1 Apr 2013 22:52:20 +0000 (22:52 +0000)]
Avoid unwanted log messages during startup.

Regardless of the given loglevel, para_recv currently prints log
messages like these at startup:

afh_init: initializing mp3 handler
afh_init: initializing ogg handler
afh_init: initializing aac handler
afh_init: initializing wma handler
afh_init: initializing spx handler
afh_init: initializing flac handler

That's because recv_init() is called before the command line arguments
are parsed, so the loglevel has not been set at this point. Other
programs have similar problems.

Fix these problems by always setting the loglevel right after a parser
has been called so that the init functions run *after* loglevel has
been set.

6 years agoDon't check return value of command line parsers unnecessarily.
Andre Noll [Sat, 6 Apr 2013 18:59:12 +0000 (18:59 +0000)]
Don't check return value of command line parsers unnecessarily.

All gengetopt parsers except the one for para_client are generated
without the --no-handle-error option, i.e. these parsers exit on errors.
Hence it is pointless to check return values.

6 years agoBuild receivers, filters and writers without -h and -V support.
Andre Noll [Fri, 5 Apr 2013 18:58:03 +0000 (18:58 +0000)]
Build receivers, filters and writers without -h and -V support.

This is not needed since the help is printed by the driving application.
It allows to get rid of an ugly "p += 3" statement in ggo.c.

6 years agofilter: Wrap lines in the available filter list.
Andre Noll [Sat, 6 Apr 2013 19:25:18 +0000 (19:25 +0000)]
filter: Wrap lines in the available filter list.

Currently para_filter -h and para_audiod -h print the available
filters in a single line which exceeds 80 characters if many
filters are supported.

This patch makes printf_or_die() return the number of characters
printed.  This allows to add line breaks to format the filter list.

6 years agoaudioc: Print config file errors.
Andre Noll [Sun, 7 Apr 2013 01:05:56 +0000 (01:05 +0000)]
audioc: Print config file errors.

Currently para_audioc fails silently if the config file contains
errors. This patch makes it print a proper error message in this case.

6 years agodoc: Rewrite udp sender description
Andre Noll [Sun, 7 Apr 2013 03:06:39 +0000 (03:06 +0000)]
doc: Rewrite udp sender description

This text was rather badly written. Replace it with better English.

6 years agoclient: Fix typo in comment.
Andre Noll [Sat, 6 Apr 2013 19:43:53 +0000 (19:43 +0000)]
client: Fix typo in comment.

6 years agoclient: Remove duplicate include.
Andre Noll [Fri, 12 Apr 2013 11:30:17 +0000 (13:30 +0200)]
client: Remove duplicate include.

The client.cmdline.h header file was included twice.

6 years agoMerge branch 't/stdin_stdout_fixes'
Andre Noll [Tue, 11 Jun 2013 18:41:49 +0000 (20:41 +0200)]
Merge branch 't/stdin_stdout_fixes'

Was cooking for almost a month.

b99b4a stdin/stdout: Restore fd flags on shutdown.

Conflicts:
stdin.c
stdout.c

6 years agoFix help text of com_touch().
Andre Noll [Tue, 11 Jun 2013 18:36:23 +0000 (20:36 +0200)]
Fix help text of com_touch().

It is the amp filter rather than the compress filter that makes use
of the amplification value stored in the audio file table.

6 years agoSimplify i9e_line_handler.
Andre Noll [Tue, 2 Apr 2013 14:30:06 +0000 (14:30 +0000)]
Simplify i9e_line_handler.

The line handlers of all three users of the i9e API (play, client
and audioc) return immediately if the passed line is NULL or the
empty string. Hence we may call the line handler only if there is a
non-empty line to dispatch.

Moving the check for a non-empty line to generic i9e code simplifies
the three line handlers a bit and avoids code duplication.

6 years agoi9e: Fix a prompt display issue
Andre Noll [Tue, 2 Apr 2013 04:05:59 +0000 (04:05 +0000)]
i9e: Fix a prompt display issue

If a line handler calls PARA_XXX_LOG() to write a log message before
attaching a buffer tree node, the prompt is not displayed properly.

Fix this by attaching a dummy btrn before the line handler is called.

6 years agoaudioc: Abstract out connection code.
Andre Noll [Tue, 2 Apr 2013 02:02:21 +0000 (02:02 +0000)]
audioc: Abstract out connection code.

This way we get a nice error message in interactive mode when
para_audiod is down. Before:

para_audioc> help
i9e_line_handler: No such file or directory

After:

para_audioc> help
connect_audiod: could not connect /var/paraslash/audiod_socket.schubert
i9e_line_handler: No such file or directory

6 years agoMerge branch 't/gui_improvements'
Andre Noll [Fri, 7 Jun 2013 19:34:51 +0000 (21:34 +0200)]
Merge branch 't/gui_improvements'

Was cooking for more than one month.

bf1831 string: Speed up xvasprintf().
bcc083 string: Add discard feature for for_each_line().
e3868d string: Simplify return value of for_each_line().
d1f0f0 string: Simplify for_each_line().
23b121 string: Clean up for_each_line() and related functions.
6256ed string: Replace the for_each_line_modes enum by a bitmap.
14e689 gui: Don't sleep before executing the status command.
e90c6c gui: Speed up add_spaces().
20e2c6 gui: Rename do_exit().
ef0508 gui: Remove superfluous cmd_died.
d7562b gui: Check stdin for readability.
a44fa6 gui: Discard overlong input lines.

Conflicts:
gui.c

6 years agoThe opus decoder.
Andre Noll [Sun, 14 Oct 2012 19:18:58 +0000 (21:18 +0200)]
The opus decoder.

Implementation is similar to the speex decoder, in particular meta
data handling is almost identical.

6 years agoThe opus audio format handler.
Andre Noll [Sun, 14 Oct 2012 16:44:57 +0000 (18:44 +0200)]
The opus audio format handler.

This fills the dummy files opus_afh.c and opus_common.c which were
introduced in the previous commit with content.  One function,
opus_parse_header(), will also be needed by the decoder to
be introduced in the next commit. So this function belongs to
opus_common.c and must be public.

6 years agoogg/opus: Infrastructure.
Andre Noll [Sun, 14 Oct 2012 14:17:44 +0000 (16:17 +0200)]
ogg/opus: Infrastructure.

This adds tests for libopus to configure.ac and minimal (non-working)
implementations of the ogg/opus decoder and audio format handler.

6 years agospeex: Don't export spx_ctl().
Andre Noll [Sun, 14 Oct 2012 14:27:13 +0000 (16:27 +0200)]
speex: Don't export spx_ctl().

This is only used from spx_common.c, so make it static and kill
the prototype in spx.h.

6 years agoMerge branch 't/doc'
Andre Noll [Sat, 25 May 2013 12:37:41 +0000 (14:37 +0200)]
Merge branch 't/doc'

Simple enough and cooking since 2013-05-05.

e82be0 manual: Add a new paragraph for installing dependencies.
1d6f5f manual: Add missing references of two optional packages.
4a6061 check_wav.c: Fix documentation of check_wav_post_select().
f9c625 Add two missing Copyright headers and file annotations.
16af9d Add documentation of public functions in check_wav.c.

6 years agoMerge remote-tracking branch 'mpi/master'
Andre Noll [Sun, 19 May 2013 21:05:00 +0000 (23:05 +0200)]
Merge remote-tracking branch 'mpi/master'

6 years agoFix compilation on Ubuntu precise.
Andre Noll [Sun, 19 May 2013 20:09:52 +0000 (22:09 +0200)]
Fix compilation on Ubuntu precise.

This distro ships glibc-2.15 where clock_gettime() is part of the
realtime library librt. So all executables which call clock_gettime()
must be linked with -lrt. The configure script detects this and adds
-lrt to LDFLAGS. However, -lrt must come *after* the objects to be
linked together, which is currently not the case.

Fix this by moving $(LDFLAGS) to the end of the recipe for the targets
of all executables.

6 years agofilter_common.c: Fix two typos in comment.
Andre Noll [Sun, 19 May 2013 19:25:13 +0000 (21:25 +0200)]
filter_common.c: Fix two typos in comment.

6 years agoalsa writer: Do not print uninitialized data.
Andre Noll [Wed, 15 May 2013 20:33:24 +0000 (22:33 +0200)]
alsa writer: Do not print uninitialized data.

ALSA's snd_output_buffer_string() returns the current size of valid
data in the returned data buffer, but this buffer is not guaranteed
to be zero-terminated.

Currently alsa_init() ignores this fact and prints the buffer up to
the first NULL byte.  Therefore it may print garbage that follows the
valid data in the buffer. If there is no zero byte after the data,
it may even segfault.

Fix this bug by using memchr() instead of strchr() and carefully
tracking the number of bytes processed.