paraslash.git
4 years agoLet afs_event() return int.
Andre Noll [Thu, 9 Apr 2015 12:51:49 +0000 (12:51 +0000)]
Let afs_event() return int.

It is usually a critical error if an afs event handler returns an
error. Currently we only print a log message an continue in this
case. The callers of afs_event() which trigger the event have no way
to tell that something went wrong, since this function returns void.

By returning int instead of void the callers can abort in the error
case. Most of the callers are afs callbacks. These can propagate the
error code to the command handler process, which will translate the
error code into a string and send it to the client.  All callbacks
are changed in this way.

4 years agoplaylist_check_callback(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 16:04:20 +0000 (16:04 +0000)]
playlist_check_callback(): Return negative on errors

4 years agomood_check_callback(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 15:48:57 +0000 (15:48 +0000)]
mood_check_callback(): Return negative on errors

4 years agocom_mvblob(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 15:34:39 +0000 (15:34 +0000)]
com_mvblob(): Return negative on errors

4 years agocom_addblob(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 15:18:48 +0000 (15:18 +0000)]
com_addblob(): Return negative on errors

4 years agocom_rmblob(): Return negative on errors, cleanup
Andre Noll [Sun, 5 Apr 2015 14:50:32 +0000 (14:50 +0000)]
com_rmblob(): Return negative on errors, cleanup

4 years agocom_catblob(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 14:43:59 +0000 (14:43 +0000)]
com_catblob(): Return negative on errors

4 years agocom_lsblob(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 14:40:44 +0000 (14:40 +0000)]
com_lsblob(): Return negative on errors

4 years agormatt: Cleanup callback.
Andre Noll [Sun, 5 Apr 2015 22:22:04 +0000 (22:22 +0000)]
rmatt: Cleanup callback.

Get rid of struct remove_attribute_action_data, since only the para
buffer is necessary. This changes the code to pass a pointer to the
para buffer itself as the data pointer for ->action() of the pattern
matching loop.

4 years agocom_rmatt(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 22:21:56 +0000 (22:21 +0000)]
com_rmatt(): Return negative on errors

4 years agocom_mvatt(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 14:12:42 +0000 (14:12 +0000)]
com_mvatt(): Return negative on errors

4 years agocom_lsatt(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 13:49:10 +0000 (13:49 +0000)]
com_lsatt(): Return negative on errors

4 years agocom_init(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 13:40:12 +0000 (13:40 +0000)]
com_init(): Return negative on errors

4 years agocom_check(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 13:35:13 +0000 (13:35 +0000)]
com_check(): Return negative on errors

4 years agocom_setatt(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 13:21:51 +0000 (13:21 +0000)]
com_setatt(): Return negative on errors

4 years agocom_cpsi(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 13:19:48 +0000 (13:19 +0000)]
com_cpsi(): Return negative on errors

4 years agocom_rm(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 13:12:18 +0000 (13:12 +0000)]
com_rm(): Return negative on errors

4 years agocom_touch(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 12:59:22 +0000 (12:59 +0000)]
com_touch(): Return negative on errors

4 years agocom_add(): Return negative on errors
Andre Noll [Sun, 5 Apr 2015 12:54:14 +0000 (12:54 +0000)]
com_add(): Return negative on errors

4 years agocom_select() callback: Return negative on errors.
Andre Noll [Sun, 5 Apr 2015 00:58:35 +0000 (00:58 +0000)]
com_select() callback: Return negative on errors.

4 years agoafs: Pass sideband error packet on callback failures.
Andre Noll [Sat, 4 Apr 2015 23:20:00 +0000 (23:20 +0000)]
afs: Pass sideband error packet on callback failures.

This changes the afs callback mechanism to honor negative return
values from a callback. We now send a special "callback failure"
sideband packet to the command handler in this case. This packet
contains the (negative) return value of the callback.

The dispatcher for afs callback results reads the error code and passes
it back via the callback request functions to the caller of the command
handler in handle_connect(). The latter already does the right thing:
It translates the error code into a string and sends this string to
the client.

This commit changes the callback of the ls command to return negative
on errors. With the patch applied the command

para_client ls /does/not/exist

now exits with status 1.

Other afs commands will make use of the new feature in subsequent
commits.

4 years agoLet afs callbacks return an error code.
Andre Noll [Sat, 4 Apr 2015 21:45:11 +0000 (21:45 +0000)]
Let afs callbacks return an error code.

It was a design mistake that callbacks have no way to tell whether
they were successful. This commit changes the callback_function
typedef so that callbacks return int instead of void.

Naturally, every callback must be adjusted accordingly. Doing so would
make the patch a bit large, so as a first step we make all callbacks
(except path_brother_callback() and hash_sister_callback() which
are special) return zero. The return value is ignored at the moment,
so the changes of this commit have no effect yet.

4 years agoafs: Introduce flush_and_free_pb().
Andre Noll [Sun, 26 Apr 2015 16:58:22 +0000 (18:58 +0200)]
afs: Introduce flush_and_free_pb().

This kills some code which is duplicated in all command handlers.

4 years agoDo not check return value of WRITE_STATUS_ITEM().
Andre Noll [Thu, 7 May 2015 16:17:54 +0000 (18:17 +0200)]
Do not check return value of WRITE_STATUS_ITEM().

The previous commit removed error checking from para_printf(), but
one instance remains: the WRITE_STATUS_ITEM() macro which also calls
para_printf(). This patch removes the error checking code of this
macro and adjusts all callers.

4 years agoDo not check the return value of para_printf().
Andre Noll [Sun, 5 Apr 2015 02:30:10 +0000 (02:30 +0000)]
Do not check the return value of para_printf().

This function rarely fails, and if it does, we don't care too much. On
the other hand, checking the return value of each call to para_printf()
clutters the code considerably, especially in the error paths where
there is already an error code we have to keep.

This commit simply removes all error checking for para_printf(),
resulting in code which is easier to follow and less error-prone.

4 years agoblob: Simplify stdin_command().
Andre Noll [Sun, 5 Apr 2015 15:04:24 +0000 (15:04 +0000)]
blob: Simplify stdin_command().

This function has only a single caller which passes a pointer to
afs_cb_result_handler() as the result handler. Also the first and
last parameter are both pointers to the command_context structure.

This commit gets rid of the two arguments of the function and updates
the documentation accordingly. Since it is a static function, we
don't need doxygen comments for all arguments.

4 years agoImprove doxygen description of mood.c.
Andre Noll [Sat, 4 Apr 2015 21:34:22 +0000 (21:34 +0000)]
Improve doxygen description of mood.c.

4 years agoplaylist_check_callback(): Remove incorrect documentation.
Andre Noll [Sat, 4 Apr 2015 21:27:41 +0000 (21:27 +0000)]
playlist_check_callback(): Remove incorrect documentation.

The function returns void, so remove the text about its return value.

4 years agoafs.c: Remove outdated comment.
Andre Noll [Sat, 4 Apr 2015 21:02:01 +0000 (21:02 +0000)]
afs.c: Remove outdated comment.

The call_calback() does not write anything to the given fd.

4 years agocom_init: Remove pointless initialization.
Andre Noll [Sat, 4 Apr 2015 20:42:25 +0000 (20:42 +0000)]
com_init: Remove pointless initialization.

4 years agoUnify version command handlers.
Andre Noll [Sat, 4 Apr 2015 20:22:53 +0000 (20:22 +0000)]
Unify version command handlers.

para_audiod's version command supports the -v flag, but the one of
para_server does not and always prints the full version text. This
adds the -v flag to the server command and makes it behave identical
to its counterpart.

4 years agoaudiod: Document and fix command handler return values.
Andre Noll [Sat, 4 Apr 2015 19:51:25 +0000 (19:51 +0000)]
audiod: Document and fix command handler return values.

It has always been the case that a positive return value from a
command handler causes audiod to dump all audiod status items to all
clients. This behaviour is not documented though, and some command
handlers get it wrong. In fact, the help, stat, tasks, grab, version
commands all return positive values although those commands never
change any status items.

This comment documents the meaning of the return value and fixes up
the above command handlers.

4 years agocommand_util.bash: Kill make_proto().
Andre Noll [Tue, 13 Jan 2015 23:52:02 +0000 (00:52 +0100)]
command_util.bash: Kill make_proto().

It is always a bad idea to parse C code with a regex in a script. In
addition, it is completely unnecessary in this case.

This commit changes command_util.bash to define, in addition to the
old XXX_CMD_ARRAY, another preprocessor macro XXX__COMMAND_HANDLERS
containing the comma separated list of command handlers without
any type information instead of grepping the source files. A simple
typedef is used to declare all command handlers.

Avoiding all the grep/sed calls reduces the (warm cache) make dep
time by ~10%.

4 years agoaudiod: Clean up fd closing logic in command handlers.
Andre Noll [Tue, 13 Jan 2015 22:53:28 +0000 (23:53 +0100)]
audiod: Clean up fd closing logic in command handlers.

audiod_command.c contains this comment:

/*
 * command handlers don't close their fd on errors (ret < 0) so that
 * its caller can send an error message. Otherwise (ret >= 0) it's up
 * to each individual command to close the fd if necessary.
 */

This is a somewhat weird rule and this commit gets rid of it. Instead,
from now on the command handlers must not close their file descriptor
and handle_connect() closes it unconditionally.

The grab and stat commands need special treatment, which was the reason
for imposing the above rule. They need to keep the file descriptor open
to send the status items or the grabbed stream to the client. This
patch makes these two handlers create a copy of the descriptor with
dup(2). The new approach is simpler and less error-prone.

4 years agoserver: Remove command pointer from struct command_context.
Andre Noll [Tue, 13 Jan 2015 23:00:41 +0000 (00:00 +0100)]
server: Remove command pointer from struct command_context.

Command handlers should not know about this implementation detail.

This commit also changes parse_sb_command() to not only parse the
command line but to actually run the command in case the caller has
sufficient permissions. The function is renamed to run_command()
to reflect this change.

We use the opportunity to clean up the allocation and freeing of
the command buffer and the argument vector. Both are now freed in
the same function they were allocated, which is considered good
programming practice.

4 years agoserver: Reduce scope of struct server_command.
Andre Noll [Tue, 13 Jan 2015 23:00:05 +0000 (00:00 +0100)]
server: Reduce scope of struct server_command.

This structure is only needed in command.c, so we can make it local
to this file. As structures defined in .c files are not included
in doxygen, the reference to struct server_command in server.h must
be removed.

4 years agoaudiod: Reduce scope of struct audiod_command.
Andre Noll [Tue, 13 Jan 2015 23:49:52 +0000 (00:49 +0100)]
audiod: Reduce scope of struct audiod_command.

This structure is only used in audiod_command.c, so move the
declaration from audiod.h to audiod_command.c.

4 years agoaction_if_pattern_matches(): Always count matching rows.
Andre Noll [Sun, 5 Apr 2015 14:07:28 +0000 (14:07 +0000)]
action_if_pattern_matches(): Always count matching rows.

We missed to do this if PM_NO_PATTERN_MATCHES_EVERYTHING is set.

4 years agobuild: Unify spelling of commands and directories.
Andre Noll [Wed, 27 Aug 2014 21:33:41 +0000 (23:33 +0200)]
build: Unify spelling of commands and directories.

Always spell commands in upper case and directories in lower case.

This is a first step to auto-generate Makefile.in.

4 years agoclient: Add details to --key-file option.
Andre Noll [Mon, 30 Mar 2015 21:39:13 +0000 (21:39 +0000)]
client: Add details to --key-file option.

The new text lists the locations where para_clients looks for
private keys.

4 years agoclient.m4: Adjust coding style of ggo options.
Andre Noll [Mon, 30 Mar 2015 21:30:02 +0000 (21:30 +0000)]
client.m4: Adjust coding style of ggo options.

All other commands use one line per directive of each option, which
improves the readability of the ggo file. This commit changes the
.m4 file for the client options to match this style of specifying
the gengetopt options.

No real changes.

4 years agodaemon: Improve daemon_log_welcome().
Andre Noll [Mon, 15 Dec 2014 19:20:44 +0000 (20:20 +0100)]
daemon: Improve daemon_log_welcome().

This function is used within para_server and para_audiod. Adding the
"para_" prefix in daemon_log_welcome() rather than its callers avoids
the duplication. Also rename the "whoami" parameter to "name" and add
documentation for it. Finally, the build date is not really important
in the log message, so simply remove it.

4 years agosetatt: Print meaningful error message.
Andre Noll [Mon, 19 Jan 2015 19:39:21 +0000 (20:39 +0100)]
setatt: Print meaningful error message.

An attempt to set or unset a non-existing attribute currently results in

key not found in rbtree

which is a bit cryptic. This commit adds another message that clarifies
what went wrong.

4 years agovss: Perform stream change actions in vss_post_select().
Andre Noll [Sun, 21 Dec 2014 19:05:47 +0000 (19:05 +0000)]
vss: Perform stream change actions in vss_post_select().

The pre_select methods should only set the scheduler timeouts and
manipulate fd sets.

Clean up the documentation a bit while at it.

4 years agoaft.c: Remove condition which is always false.
Andre Noll [Mon, 22 Dec 2014 01:15:19 +0000 (01:15 +0000)]
aft.c: Remove condition which is always false.

aft_open() returns early if ret is non-negative, so ret is known to
be negative at this point.

Upgrade the log message severity on open failure from INFO to NOTICE
while at it.

4 years agoaft.c: Silence noisy warnings in com_add().
Andre Noll [Wed, 24 Dec 2014 02:55:52 +0000 (02:55 +0000)]
aft.c: Silence noisy warnings in com_add().

If the -a option to com_add() is not given, we ignore paths for which
guess_audio_format() returns negative. Currently the add command sends
a warning to the client for each skipped file, which may result in
very verbose output. This commit silences the warning.

4 years agoweb: fix htaccess description of snapshot tarball.
Andre Noll [Sun, 3 May 2015 10:51:13 +0000 (12:51 +0200)]
web: fix htaccess description of snapshot tarball.

Without this, the snapshot tarball file on the download web page is
annotated as "release tarball" rather than "current master snapshot".

Fix this by adding two additional patterns to the .htaccess file.

4 years agobuild: Add -Wunused-macros to CPPFLAGS on Linux.
Andre Noll [Wed, 1 Apr 2015 02:50:27 +0000 (02:50 +0000)]
build: Add -Wunused-macros to CPPFLAGS on Linux.

We already specify -Wunused which combines all -Wunused options but
does not activate the -Wunused-macros cpp warning.

4 years agoMerge branch 'maint'
Andre Noll [Sat, 30 May 2015 09:54:47 +0000 (11:54 +0200)]
Merge branch 'maint'

4 years agoMerge branch 'refs/heads/t/web-diet'
Andre Noll [Sun, 10 May 2015 18:27:35 +0000 (20:27 +0200)]
Merge branch 'refs/heads/t/web-diet'

Was cooking in next for three weeks.

* web: Remove screenshots, license and credits page.
* web: Integrate Contact page into about.
* web: Make FEATURES the new homepage, rename it to "about".
* web: Shorten feature list

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 agomanual: Improve section on decoders.
Andre Noll [Wed, 1 Apr 2015 13:55:46 +0000 (13:55 +0000)]
manual: Improve section on decoders.

Decoders are also part of para_play but only para_filter and
para_audiod are mentioned in the text.  Let's just talk about
"executables" here.

4 years agomanual: Fix paragraph on addblob commands.
Andre Noll [Wed, 1 Apr 2015 13:48:16 +0000 (13:48 +0000)]
manual: Fix paragraph on addblob commands.

These days we have the AWAITING_DATA sideband designator to distinguish
between normal and addblob commands.

4 years agoMerge branch 'refs/heads/t/audiod_acl'
Andre Noll [Sun, 3 May 2015 09:51:22 +0000 (11:51 +0200)]
Merge branch 'refs/heads/t/audiod_acl'

Cooking for about one month.

* audiod: Allow to specify usernames in --user-allow.

4 years agoAllow to start server and audiod as daemon with no logfile.
Andre Noll [Tue, 7 Oct 2014 14:02:07 +0000 (14:02 +0000)]
Allow to start server and audiod as daemon with no logfile.

Currently para_audiod and para_server won't start in the background
if no logfile is specified, so "-dL /dev/null" must be given to force
this. This is a bit tedious to type, so this commit makes "/dev/null"
the default.

To achive this, we can simply remove the gengetopt "dependon" statement
from daemon.m4. This works because if no logfile was given, log output
is written to stderr, which is redirected to /dev/null in case -d
was also given. We need to open /dev/null in read-write mode though,
but no other changes are required.

4 years agoerror.h: Fix typo in error text of E_BAD_AFSI.
Andre Noll [Mon, 6 Apr 2015 00:10:54 +0000 (00:10 +0000)]
error.h: Fix typo in error text of E_BAD_AFSI.

4 years agoMerge branch 'refs/heads/t/immediate-si-update'
Andre Noll [Sun, 26 Apr 2015 12:11:56 +0000 (14:11 +0200)]
Merge branch 'refs/heads/t/immediate-si-update'

Cooking in next for four weeks.

* aft: Store resolved paths when adding files.
* aft: Update mtime and file size on afhi changes.
* aft: Update status items on afs events.
* aft.c: Pass struct ls_data pointer to make_status_items().
* afs: Simplify open_next_audio_file().

Conflicts:
aft.c

4 years agoversion.c: Update copyright year to range 2002-2015.
Andre Noll [Tue, 7 Apr 2015 18:34:05 +0000 (18:34 +0000)]
version.c: Update copyright year to range 2002-2015.

Although we don't update the year in each source file any more,
version.c contains the text that is printed with --version. So this
text should be up to date, and it should be a range. This commit
changes it from "2014" to "2002-2015".

2002 seems appropriate for the first year in the range since the
initial cvs commit was in 2002. Some code is older than that, but
that's probably not important.

4 years agotouch, rm, cpsi, init: Fix initialization of para_buffer.
Andre Noll [Tue, 7 Apr 2015 23:56:11 +0000 (23:56 +0000)]
touch, rm, cpsi, init: Fix initialization of para_buffer.

Three years ago, in commit 68cb0aef (Introduce
afs_max_size_handler_data and afs_max_size_handler()) the afs callbacks
were converted to pass a pointer to an afs_max_size_handler_data
structure to the dispatcher. This structure is defined as

struct afs_max_size_handler_data {
int fd;
uint8_t band;
};

However, we missed to convert the callbacks of the three commands
mentioned in the subject. All these commands except init pass
a pointer to an int as they did before commit 68cb0aef. Since
afs_max_size_handler_data stores one additional byte (the band
designator) after the file descriptor, the dispatcher will read one
byte past the allocated space.

This bug is benign because the max size handler is usually not called
for the affected commands, since they never have more than SHMMAX
bytes of output. For com_init() we even set the private_data pointer
to NULL, so the max size handler will never be called.

Let's fix it anyway.

4 years agoweb: Remove screenshots, license and credits page.
Andre Noll [Sat, 3 Jan 2015 02:50:12 +0000 (02:50 +0000)]
web: Remove screenshots, license and credits page.

The server and audiod log files provided little information and
were constantly outdated, so this patch removes them. The para_gui
screenshot is turned into a link in the feature list where para_gui
is mentioned. This gets rid of the screenshots page.

The paraslash license is mentioned on the main page, so we don't
need a dedicated page for it. The CREDITS file stays in the git repo
of course but we don't need a web page for this either. It's fairly
static anyway.

4 years agoweb: Integrate Contact page into about.
Andre Noll [Sat, 3 Jan 2015 02:39:57 +0000 (02:39 +0000)]
web: Integrate Contact page into about.

This page contains only an email address and one sentence. The main
page is a better place for this information.

4 years agoweb: Make FEATURES the new homepage, rename it to "about".
Andre Noll [Sat, 3 Jan 2015 02:23:18 +0000 (02:23 +0000)]
web: Make FEATURES the new homepage, rename it to "about".

This commit moves the FEATURES file to web/about.html.in, switching
the syntax from grutatxt to plain html. The new file is translated into
index.html while the old main page is now available as the "News" page.

4 years agoweb: Shorten feature list
Andre Noll [Sat, 3 Jan 2015 01:58:23 +0000 (01:58 +0000)]
web: Shorten feature list

This list was a bit too verbose for a web page. This patch reduces
the level of detain to mention only the most prominent features.

4 years agoaft.c: Simplify load_afd().
Andre Noll [Wed, 24 Dec 2014 02:52:03 +0000 (02:52 +0000)]
aft.c: Simplify load_afd().

Remove the pointless variable "buf".

4 years agoMerge branch 't/lib_arg_with'
Andre Noll [Sun, 19 Apr 2015 11:49:54 +0000 (13:49 +0200)]
Merge branch 't/lib_arg_with'

Was cooking in next for three weeks.

* t/lib_arg_with:
  build: Fix curses detection.
  build: Convert curses detection to new macros.
  build: Convert libreadline detection to new macros.
  build: Convert oss detection to new macros.
  build: Convert libsamplerate detection to new macros.
  build: Convert libao detection to new macros.
  build: Convert alsa detection to new macros.
  build: Convert flac detection to new macros.
  build: Convert libid3tag detection to new macros.
  build: Convert mad detection to new macros.
  build: Convert faad detection to new macros.
  build: Convert crypto detection/selection to new macros.
  build: Convert opus detection to new macros.
  build: Convert speex detection to new macros.
  build: convert vorbis detection to new macros.
  build: Convert ogg detection to new macros.
  build: Reduce redundancy in configure.ac, convert osl detection.

4 years agoMerge branch 'maint'
Andre Noll [Sun, 19 Apr 2015 11:42:14 +0000 (13:42 +0200)]
Merge branch 'maint'

5 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.

5 years agogcrypt: Some trivial fixes.
Andre Noll [Wed, 18 Feb 2015 15:40:01 +0000 (16:40 +0100)]
gcrypt: Some trivial fixes.

Some whitespace issues, but also a typo and two instances of
superfluous parentheses.

5 years agoaft: Rename label "err" of open_and_update_audio_file().
Andre Noll [Sun, 21 Dec 2014 14:54:18 +0000 (14:54 +0000)]
aft: Rename label "err" of open_and_update_audio_file().

We also execute this on success.

5 years agoaudiod: Make struct slot_info local to audiod.c.
Andre Noll [Sun, 4 Jan 2015 04:55:43 +0000 (04:55 +0000)]
audiod: Make struct slot_info local to audiod.c.

Thanks to the previous commit there is only one user of struct
slot_info left in audiod_command.c: decoder_flags(). Moving this
function to audiod.c allows to make the structure private to audiod.c,
along with the slot array and the MAX_STREAM_SLOTS and FOR_EACH_SLOT
macros.

Since audiod_command still needs the decoder flags for the stat command,
the function is made public and renamed to audiod_get_decoder_flags().

5 years agoaudiod: Move get_play_time_slot_num() to audiod.c.
Andre Noll [Sun, 4 Jan 2015 04:39:15 +0000 (04:39 +0000)]
audiod: Move get_play_time_slot_num() to audiod.c.

audiod_status_dump() sends the play time string that corresponds
to the current slot to all connected stat clients. It first calls
the get_play_time_slot_num() helper to obtain the slot number, then
calls get_time_string(slot_num).

Currently the helper function is part of audiod_command.c, hence
we expose the slot_info structure to audiod_command.c. Moving the
helper from audiod_command.c to audiod.c is a first step to make this
structure local to audiod.c.  It also allows to remove the slot_num
parameter from get_time_string().

5 years agoalsa: Avoid alloca().
Andre Noll [Fri, 2 Jan 2015 20:10:56 +0000 (20:10 +0000)]
alsa: Avoid alloca().

It is not in POSIX.1-2001, and its use is discouraged. Fortunately,
the alsa library provides also variants which call ordinary malloc(),
so use these instead.

5 years agocommand: Make struct command_context->cmd_ptr const.
Andre Noll [Mon, 5 Jan 2015 03:28:58 +0000 (03:28 +0000)]
command: Make struct command_context->cmd_ptr const.

The fields of struct server command never need to be modified so they
can be const.

5 years agoafs.c: Trivial whitespace cleanup.
Andre Noll [Wed, 1 Apr 2015 01:34:26 +0000 (01:34 +0000)]
afs.c: Trivial whitespace cleanup.

5 years agoserver.h: Trivial whitespace/language cleanup.
Andre Noll [Wed, 1 Apr 2015 00:25:50 +0000 (00:25 +0000)]
server.h: Trivial whitespace/language cleanup.

5 years agobuild: Fix curses detection.
Andre Noll [Tue, 31 Mar 2015 21:11:10 +0000 (21:11 +0000)]
build: Fix curses detection.

We need to append to the $curses_ldflags variable rather than
overwriting it. Without this fix, --with-curses-libs is effectively
ignored.

5 years agoaudiod: Allow to specify usernames in --user-allow.
Andre Noll [Mon, 19 Jan 2015 23:25:18 +0000 (00:25 +0100)]
audiod: Allow to specify usernames in --user-allow.

It's more convenient to specify usernames rather than the UIDs of
the users who may execute an audiod command. This patch allows to
do so. For backwards compatibility we still need to accept numerical
UIDs though.

On startup we first try to convert each given --user-allow argument
to a number and regard this number as a UID. Only if the conversion
fails, we translate the argument to a username with getpwnam().
In order to not perform this conversion on each command, we allocate
a UID whitelist at startup and populate it with the UIDs derived from
both types of --user allow arguments. A pointer to the whitelist is
passed as an additional argument to handle_connect().

The documentation is updated to reflect this change.

5 years agoaft: Store resolved paths when adding files.
Andre Noll [Wed, 24 Dec 2014 03:07:48 +0000 (03:07 +0000)]
aft: Store resolved paths when adding files.

Currently the add command performs some sanity checks on the given
paths, but stores them unmodified in the audio file table if they
pass the checks. This commit removes the checks in favor of a call to
realpath(3) to get the canonicalized absolute pathname to be stored
in the audio file table.

The new code relies on POSIX.1-2008 semantics. That is, it calls
realpath() with second argument NULL to let the function allocate a
suitably sized buffer. This should not be a problem, since the old
POSIX.1-2001 version is broken by design, and all moderately recent
systems support the new semantics.

This change breaks t0004 which expects the ls -p command to print
the same paths that were used earlier to add audio files, which is
no longer true. We fix the test by simply running ls without the
-p option.

5 years agoaft: Update mtime and file size on afhi changes.
Andre Noll [Tue, 23 Dec 2014 12:52:03 +0000 (12:52 +0000)]
aft: Update mtime and file size on afhi changes.

If the current audio file is re-added to the osl database, for example
because meta tags have changed, file size and modification time change.
However, the paraslash stat command still shows the old values because
currently the virtual streaming system calls fstat(2) only once when
the audio file is opened and keeps reporting the resulting mtime and
file size values until the file is closed.

This commit introduces make_inode_status_items() which is called
from make_status_items() on AFSI_CHANGE and AFHI_CHANGE events for
the current audio file. The new function calls stat(2) on the path
obtained from the current aft row. We can't use fstat(2) since (a)
we don't have an open file descriptor any more and (b) this would
not work anyway if the original file was overwritten.

Due to the new helper, mmd->mtime can be removed. We must keep
mmd->size though, since we need the original file size for the call
to munmap().

5 years agoaft: Update status items on afs events.
Andre Noll [Mon, 22 Dec 2014 18:05:38 +0000 (18:05 +0000)]
aft: Update status items on afs events.

When a paraslash command changes the afs or afh info structures of the
current audio file, for example by re-adding the file with modified
meta tags, the stat output still reports the old values. With this
patch applied, the status items are updated immediately.

This is achieved by letting aft.c honor the AFSI_CHANGE and AFHI_CHANGE
events. If the audio file which triggered the event is the one
currently being streamed, the event handler recreates the status items.
We need two new global static variables in aft.c to track the row of
the audio file table which corresponds to the current file.

5 years agoaft.c: Pass struct ls_data pointer to make_status_items().
Andre Noll [Sun, 21 Dec 2014 18:07:40 +0000 (18:07 +0000)]
aft.c: Pass struct ls_data pointer to make_status_items().

We essentially already do that, with each field of the structure
spelled out in the argument list of make_status_items().

5 years agoafs: Simplify open_next_audio_file().
Andre Noll [Sun, 21 Dec 2014 15:24:18 +0000 (15:24 +0000)]
afs: Simplify open_next_audio_file().

This function of afs.c is called whenever the virtual streaming
system requests a new audio file. It determines the row of the audio
file table that corresponds to the audio file with highest score,
according to the current mood or playlist. The function passes this
information to open_and_update_audio_file() of aft.c which sets up
an afd structure to be passed back to the server process.

Letting open_and_update_audio_file() determine which file to
open is shorter, and it allows to get rid of two arguments to
open_and_update_audio_file().

5 years agobuild: Convert curses detection to new macros.
Andre Noll [Sun, 28 Sep 2014 15:38:35 +0000 (15:38 +0000)]
build: Convert curses detection to new macros.

5 years agobuild: Convert libreadline detection to new macros.
Andre Noll [Sun, 21 Sep 2014 14:48:01 +0000 (16:48 +0200)]
build: Convert libreadline detection to new macros.

5 years agobuild: Convert oss detection to new macros.
Andre Noll [Mon, 25 Aug 2014 21:50:01 +0000 (23:50 +0200)]
build: Convert oss detection to new macros.

This also adds the missing oss_cppflags autoconf variable and the
corresponding make rule for building the oss writer.

5 years agobuild: Convert libsamplerate detection to new macros.
Andre Noll [Sun, 24 Aug 2014 22:24:29 +0000 (00:24 +0200)]
build: Convert libsamplerate detection to new macros.

5 years agobuild: Convert libao detection to new macros.
Andre Noll [Sun, 24 Aug 2014 22:00:51 +0000 (00:00 +0200)]
build: Convert libao detection to new macros.

Splits the old check into two checks for pthread and libao.

5 years agobuild: Convert alsa detection to new macros.
Andre Noll [Sun, 24 Aug 2014 20:57:29 +0000 (22:57 +0200)]
build: Convert alsa detection to new macros.

Adds alsa_cppflags.

5 years agobuild: Convert flac detection to new macros.
Andre Noll [Wed, 30 Jul 2014 08:11:15 +0000 (10:11 +0200)]
build: Convert flac detection to new macros.

5 years agobuild: Convert libid3tag detection to new macros.
Andre Noll [Tue, 3 Jun 2014 17:42:52 +0000 (19:42 +0200)]
build: Convert libid3tag detection to new macros.

5 years agobuild: Convert mad detection to new macros.
Andre Noll [Tue, 3 Jun 2014 17:03:32 +0000 (19:03 +0200)]
build: Convert mad detection to new macros.

5 years agobuild: Convert faad detection to new macros.
Andre Noll [Sun, 4 May 2014 17:22:22 +0000 (19:22 +0200)]
build: Convert faad detection to new macros.

5 years agobuild: Convert crypto detection/selection to new macros.
Andre Noll [Fri, 2 May 2014 21:01:53 +0000 (23:01 +0200)]
build: Convert crypto detection/selection to new macros.

5 years agobuild: Convert opus detection to new macros.
Andre Noll [Sun, 20 Apr 2014 23:27:19 +0000 (23:27 +0000)]
build: Convert opus detection to new macros.

5 years agobuild: Convert speex detection to new macros.
Andre Noll [Sun, 20 Apr 2014 23:08:24 +0000 (23:08 +0000)]
build: Convert speex detection to new macros.

5 years agobuild: convert vorbis detection to new macros.
Andre Noll [Mon, 2 Jun 2014 20:24:50 +0000 (22:24 +0200)]
build: convert vorbis detection to new macros.

Also introduces NEED_OGG_OBJECTS() and NEED_VORBIS_OBJECTS(). These
two macros are identical at the moment, but they will start to differ
when opus and speex will be converted.

5 years agobuild: Convert ogg detection to new macros.
Andre Noll [Sun, 20 Apr 2014 22:18:25 +0000 (22:18 +0000)]
build: Convert ogg detection to new macros.

5 years agobuild: Reduce redundancy in configure.ac, convert osl detection.
Andre Noll [Sun, 20 Apr 2014 04:33:11 +0000 (04:33 +0000)]
build: Reduce redundancy in configure.ac, convert osl detection.

We've had a fair number of bugs in configure.ac due to the fact that
most parts of it were created by copy+paste from an existing part.
This commit is an attempt to reduce this redundancy by factoring out
the reappearing parts to common macros. This will also reduce the
number of lines of configure.ac considerably, although this first
patch naturally adds more lines than it removes.

This commit introduces four new macros in configure.ac:

* AC_ARG_WITH: handles --with-xxx-headers and --with-xxx-libs
options for configure. The generated options are identical
to the old ones, so there are no compatibility issues.

* LIB_SUBST_FLAGS: creates preprocessor defines and make
variables

* STASH_FLAGS: saves copies of the CPPFLAGS, LDFLAGS and
LIBS variables

* UNSTASH_FLAGS restores stashed values

Only libosl detection is converted to use the new macros.  Subsequent
patches will convert other libraries.

5 years ago.gitignore: Ignore *~ and *.rej files.
Andre Noll [Mon, 23 Feb 2015 08:12:15 +0000 (09:12 +0100)]
.gitignore: Ignore *~ and *.rej files.

These should never be under version control.

5 years agoerror.h: Change type of return value of is_errno() to bool.
Andre Noll [Sun, 4 Jan 2015 02:48:08 +0000 (02:48 +0000)]
error.h: Change type of return value of is_errno() to bool.

There is only a single caller in afs.c which is OK with this change.