paraslash.git
9 years agoKill afhi->header_offset.
Andre Noll [Wed, 6 Oct 2010 20:21:07 +0000 (22:21 +0200)]
Kill afhi->header_offset.

It is always zero, and stored _twice_ in the osl table. Since we must
not change the layout of the on-disk afhi structure, rename these
fields to AFHI_UNUSED1_OFFSET and AFHI_UNUSED2_OFFSET, and set them
to zero in save_afhi().

9 years agoafh: Fix a memory leak in stream mode.
Andre Noll [Sun, 3 Oct 2010 10:43:00 +0000 (12:43 +0200)]
afh: Fix a memory leak in stream mode.

We need to free the audio format handler info in both info and stream mode.

9 years agovss: Initialize all bytes of the FEC header.
Andre Noll [Sun, 24 Oct 2010 17:23:25 +0000 (19:23 +0200)]
vss: Initialize all bytes of the FEC header.

Does not matter much but makes valgrind happier.

9 years agoTrivial comment cleanups.
Andre Noll [Sat, 11 Sep 2010 23:58:52 +0000 (01:58 +0200)]
Trivial comment cleanups.

9 years agoNEWS update.
Andre Noll [Sat, 29 Jan 2011 10:40:40 +0000 (11:40 +0100)]
NEWS update.

9 years agoMerge branch 't/command_util_replace_tabs'
Andre Noll [Sat, 29 Jan 2011 10:37:34 +0000 (11:37 +0100)]
Merge branch 't/command_util_replace_tabs'

9 years agoMerge branch 't/color_cleanups'
Andre Noll [Sat, 29 Jan 2011 10:34:18 +0000 (11:34 +0100)]
Merge branch 't/color_cleanups'

9 years agoMerge branch 't/aacdec_cleanups'
Andre Noll [Sat, 29 Jan 2011 10:01:30 +0000 (11:01 +0100)]
Merge branch 't/aacdec_cleanups'

9 years agoChange year in version string to 2011.
Andre Noll [Thu, 27 Jan 2011 07:44:57 +0000 (08:44 +0100)]
Change year in version string to 2011.

9 years agoNEWS update.
Andre Noll [Sat, 22 Jan 2011 16:02:19 +0000 (17:02 +0100)]
NEWS update.

9 years agoMerge branch 't/periodic_header_optional'
Andre Noll [Sat, 22 Jan 2011 15:57:02 +0000 (16:57 +0100)]
Merge branch 't/periodic_header_optional'

9 years agoSuppress non-fatal bitstream error message at BOF/EOF.
Andre Noll [Sat, 18 Dec 2010 12:05:25 +0000 (13:05 +0100)]
Suppress non-fatal bitstream error message at BOF/EOF.

These errors occur frequently for 64KBit aac files. So print an
error message only if the error occurs in the middle of the stream.

9 years agoaacdec: Print error message only once.
Andre Noll [Sat, 18 Dec 2010 12:00:04 +0000 (13:00 +0100)]
aacdec: Print error message only once.

No need to call NeAACDecGetErrorMessage() twice.

9 years agoaacdec: Remove useless call to btr_get_input_queue_size().
Andre Noll [Sat, 18 Dec 2010 11:54:11 +0000 (12:54 +0100)]
aacdec: Remove useless call to btr_get_input_queue_size().

The returned input queue size is never used.

9 years agocommand_util: Replace tabs by \t.
Andre Noll [Wed, 24 Nov 2010 22:36:01 +0000 (23:36 +0100)]
command_util: Replace tabs by \t.

Probably does not matter much...

9 years agoReplace 2010 in copyright message by 2011.
Andre Noll [Mon, 17 Jan 2011 07:50:18 +0000 (08:50 +0100)]
Replace 2010 in copyright message by 2011.

9 years agoogg_afh_common.c: Remove pointless loop counter.
Andre Noll [Fri, 14 Jan 2011 18:51:22 +0000 (19:51 +0100)]
ogg_afh_common.c: Remove pointless loop counter.

The integer variable i is not used at all in the loop body.

9 years agohttp_recv.c: Trivial typo fix.
Andre Noll [Mon, 10 Jan 2011 22:34:12 +0000 (23:34 +0100)]
http_recv.c: Trivial typo fix.

9 years agoMerge remote-tracking branch 'fml/master'
Andre Noll [Sat, 18 Dec 2010 11:07:36 +0000 (12:07 +0100)]
Merge remote-tracking branch 'fml/master'

9 years agonew codename, reset version to git
Andre Noll [Fri, 17 Dec 2010 16:10:49 +0000 (17:10 +0100)]
new codename, reset version to git

9 years agothe paraslash-0.4.5 release tarball
Andre Noll [Fri, 17 Dec 2010 16:06:25 +0000 (17:06 +0100)]
the paraslash-0.4.5 release tarball

9 years agoparaslash 0.4.5 v0.4.5
Andre Noll [Fri, 17 Dec 2010 16:02:14 +0000 (17:02 +0100)]
paraslash 0.4.5

9 years agoweb: Remove server side includes.
Andre Noll [Thu, 9 Dec 2010 10:37:17 +0000 (11:37 +0100)]
web: Remove server side includes.

The new apache on systemlinux.org does not have this feature activated.
We don't need it anyway.

9 years agoMerge branch 't/kill_udp_cq'
Andre Noll [Wed, 8 Dec 2010 12:45:42 +0000 (13:45 +0100)]
Merge branch 't/kill_udp_cq'

9 years agoclient: Kill superfluous label "err".
Andre Noll [Mon, 6 Dec 2010 22:05:28 +0000 (23:05 +0100)]
client: Kill superfluous label "err".

The position of this label is identical to the "out" label, and "out"
is more to the point as we jump there not only in case of an error. So
jump always to "out" and remove the "err" label.

9 years agocolor: Simplify color error handling.
Andre Noll [Sun, 21 Nov 2010 20:48:42 +0000 (21:48 +0100)]
color: Simplify color error handling.

We exit on errors anyway, so get rid of the return value of
color_parse() and daemon_set_log_color() and abort immediately rather
than returning -1. Add the familiar "_or_die" suffix to these functions
to make it clear that no error handling is necessary in the caller.

9 years agoNEWS update.
Andre Noll [Sun, 5 Dec 2010 17:52:19 +0000 (18:52 +0100)]
NEWS update.

9 years agoMerge branch 't/writer_cleanups'
Andre Noll [Sun, 5 Dec 2010 17:41:36 +0000 (18:41 +0100)]
Merge branch 't/writer_cleanups'

9 years agoMinor documentation cleanups.
Andre Noll [Sun, 5 Dec 2010 17:38:06 +0000 (18:38 +0100)]
Minor documentation cleanups.

9 years agofecdec: Use a fixed buffer pool size of 64K.
Andre Noll [Sun, 28 Nov 2010 21:43:45 +0000 (22:43 +0100)]
fecdec: Use a fixed buffer pool size of 64K.

With the new variable sized FEC slices, it may happen that the
number of bytes per slice of the first FEC group is very small. It is
therefore no longer appropriate to base the size of the buffer pool
on this quantity. It happened to be much too small (< 1000 bytes)
for one aac file which caused the fecdec filter to abort early due
to a full buffer pool.

This patch uses a fixed buffer pool size of 64K for the fecdec filter,
which ought to be enough for everybody.

9 years agowrite_common: Don't abort if btr_exec_up() failed.
Andre Noll [Sun, 28 Nov 2010 21:32:13 +0000 (22:32 +0100)]
write_common: Don't abort if btr_exec_up() failed.

btr_exec_up() failure is unusual but possible if the upper btr node dies just in
the right moment.  It happened for an aac file due to another bug in the fecdec
filter (fixed in a subsequent patch)  which caused the fecdec and hence the aacdec
btr nodes to unregister themselves due to a full buffer tree pool.

So replace the assertion by a fat error message. This sets the sample rate,
channel count and the sample format to zero which makes the writer unregister
itsself. This is better than aborting.

9 years agofd.c: Fix typo in comment.
Andre Noll [Sun, 28 Nov 2010 21:17:39 +0000 (22:17 +0100)]
fd.c: Fix typo in comment.

10 years agovss: Avoid large FEC parameters for the DCCP transport.
Andre Noll [Mon, 1 Nov 2010 15:30:22 +0000 (16:30 +0100)]
vss: Avoid large FEC parameters for the DCCP transport.

Now that for DCCP streams the audio file header is sent only once at client
connect time, we can go one step further and send the header as its own FEC
group. Then all subsequent FEC groups contain data slices only, hence the
maximal required size for a FEC group reduces from

header_size + largest_chunk_size

to

max(header_size, largest_chunk_size)

This patch introduces a new helper function need_data_slices() which returns
false only at the beginning of a DCCP stream. In this case FEC group 0 consists
of the header only and an arbitrary time interval of 200ms is used for this
group.

10 years agovss: Let the transport specify whether periodic headers are necessary.
Andre Noll [Sun, 17 Oct 2010 09:45:57 +0000 (11:45 +0200)]
vss: Let the transport specify whether periodic headers are necessary.

This adds the new flag need_periodic_header to struct fec_client_parms. It is
true for the UDP transport and false for the DCCP transport.

vss checks this flag in need_audio_header() and does not send the header
periodically if the flag is false.

10 years agowriters: Kill ->open.
Andre Noll [Sat, 6 Nov 2010 10:48:12 +0000 (11:48 +0100)]
writers: Kill ->open.

Now that ->open is a dummy function for all writers, we may safely remove
this method from struct writer and kill the dummy functions.

10 years agoosx_write: Make osx_write_open() a no-op.
Andre Noll [Sat, 6 Nov 2010 10:43:59 +0000 (11:43 +0100)]
osx_write: Make osx_write_open() a no-op.

Move the allocation of the private_osx_write_data struct to core_audio_init()
and adjust the the check whether core audio has been initialized accordingly.

10 years agooss: Make oss_open() a no-op.
Andre Noll [Fri, 5 Nov 2010 18:45:00 +0000 (19:45 +0100)]
oss: Make oss_open() a no-op.

Move the allocation of the private_oss_write_data struct to oss_init() and
adjust the the check whether oss has been initialized accordingly.

10 years agofile_writer: Make file_write_open() a no-op.
Andre Noll [Fri, 5 Nov 2010 18:28:24 +0000 (19:28 +0100)]
file_writer: Make file_write_open() a no-op.

Move the allocation of the private_file_write_data struct to ->post_select()
and adjust the the check whether the output file has already been opened
accordingly.

If the output file has just been opened, pfwd->fd will never be set in
the write fd set of the scheduler, so we can skip this test.

10 years agoalsa: Make alsa_open() a no-op.
Andre Noll [Fri, 5 Nov 2010 18:07:39 +0000 (19:07 +0100)]
alsa: Make alsa_open() a no-op.

Move the allocation of the private_alsa_write_data struct to
->post_select() and adjust the the check whether alsa has
been initialialized accordingly.

10 years agowriters: Unify ->pre_select().
Andre Noll [Fri, 5 Nov 2010 07:45:20 +0000 (08:45 +0100)]
writers:  Unify ->pre_select().

Always treat the easy cases "nothing to do", "error", and "not yet initialized"
first. For the alsa writer,  this change fixes two minor bugs:

First, if data is available but alsa has not yet been initialized, we return
from ->pre_select() without setting a delay. This is wrong, we should init
the alsa handle ASAP in this case.

Second,  on errors we wait 20ms which is both ugly and unnecessary.
Requesting a minimal delay is the right thing to do here as well.

10 years agoChange the ->open method of writers to void.
Andre Noll [Thu, 4 Nov 2010 22:54:23 +0000 (23:54 +0100)]
Change the ->open method of writers to void.

These methods always succeed. Add missing documentation of the public
register_writer_node() function while we're at it.

10 years agofile_write: Fix NULL pointer dereference.
Andre Noll [Thu, 4 Nov 2010 22:34:33 +0000 (23:34 +0100)]
file_write: Fix NULL pointer dereference.

register_writer_node() is the only caller of the ->open method for paraslash
writers. It does not check its return value, which is OK for alsa, oss and osx
as these writer's ->open method always succeeds. However, the ->open() method
of the file writer may fail, for example because the output file could not be
opened. This error will be ignored and the writer node task is registered as
usual with ->fd being initialized to zero.

Fix this bug by splitting the ->open method of the file writer into the part which
merely allocates the private_file_write_data structure, hence always succeeds,
and the part which actually opens the output file. This second part is called later
from ->post_select as soon as there is data available to be written.

After this patch the ->open methods of all writers always succeed and we
may change its return value to void which is done in the next patch.

10 years agowrite_common: Remove check which is never true.
Andre Noll [Thu, 4 Nov 2010 08:12:57 +0000 (09:12 +0100)]
write_common: Remove check which is never true.

All writers have a config parser. This check was bogus anyway: If there was a writer
without config parser and we call it without arguments (i.e. "c" is NULL), this would lead
to a NULL pointer dereference.

10 years agowrite: Simplify config parsers.
Andre Noll [Thu, 4 Nov 2010 07:59:57 +0000 (08:59 +0100)]
write: Simplify config parsers.

These functions all call the gengetopt parser which aborts on errors. It is therefore
pointless to check the return value. Document this fact and make it explicit by renaming
->parse_config of struct writer to ->parse_config_or_die().

10 years agoMinor makesock() fixes.
Andre Noll [Sun, 10 Oct 2010 16:51:17 +0000 (18:51 +0200)]
Minor makesock() fixes.

On getaddrinfo() or setsockopt() errors, we leak the flowopts and/or
the local/remote address info structure. Fix this by jumping to the
cleanup section at the bottom of makesock() rather than returning
early without cleaning up.

Moreover, we can not rely on errno containing a valid error code in the
cleanup part of the function because flowopt_cleanup() calls free()
which usually resets errno. So use the error code provided via "rc"
if possible and fall back to the new -E_MAKESOCK if rc is non-negative
to make sure we return a negative value on errors.

10 years agoNEWS update.
Andre Noll [Sun, 14 Nov 2010 14:35:33 +0000 (15:35 +0100)]
NEWS update.

10 years agoMerge branch 't/server_optional'
Andre Noll [Sun, 14 Nov 2010 14:32:45 +0000 (15:32 +0100)]
Merge branch 't/server_optional'

10 years agomanual: Add a section on rebuild of the "next" branch.
Andre Noll [Sun, 31 Oct 2010 12:20:30 +0000 (13:20 +0100)]
manual: Add a section on rebuild of the "next" branch.

10 years agoNEWS update.
Andre Noll [Sun, 31 Oct 2010 11:59:11 +0000 (12:59 +0100)]
NEWS update.

10 years agoMake build of para_server optional.
Andre Noll [Sun, 12 Sep 2010 21:30:59 +0000 (23:30 +0200)]
Make build of para_server optional.

para_server is the only executables which requires libosl, so do not consider
it a fatal error if libosl could not be located. This allows to build the
paraslash package for example on client-only machines which do not need
para_server anyway.

10 years agochunk_queue: Remove cq_force_enqueue().
Andre Noll [Wed, 6 Oct 2010 22:08:48 +0000 (00:08 +0200)]
chunk_queue: Remove cq_force_enqueue().

It was only used by the UDP sender which does not use chunk queueing
any more.

10 years agoudp: Remove chunk queueing.
Andre Noll [Wed, 6 Oct 2010 22:08:31 +0000 (00:08 +0200)]
udp: Remove chunk queueing.

This was broken beyond repair for several reasons:

First of all, not each write to a UDP socket with no listener on
the remote leads to a write error. For a local connection on Linux,
only each second write yields ECONNREFUSED while all others seem to
succeed. So we would only queue each second FEC slice which is next
to useless.

Secondly, only buffer references are stored in the chunk queue, the
buffer contents are not copied for performance reasons. This works fine
if the buffers point to the read-only memory map of the audio file,
which is the case for the HTTP sender, but not for the UDP sender.
In fact, for UDP the buffer is always the same, namely ->enc_buf of
the FEC client struct for the UDP target.

Finally, it is not clear that the buffer references stored in the chunk
queue are still valid when the chunk queue is emptied by sending the
buffers since vss.c might have realloced the enc_buf.

So remove that broken code and be happy.

10 years agoudp: Also send the EOF packet when a target is removed.
Andre Noll [Wed, 6 Oct 2010 21:38:17 +0000 (23:38 +0200)]
udp: Also send the EOF packet when a target is removed.

This notifies the client of the EOF condition and causes it to remove
its buffer tree.  Without this patch, the client keeps listening
which can lead to problems when the target is added later while a
different audio file is being streamed.

Also there is no point to check twice whether the chunk queue exists.

10 years agoRemove unused field "sc" of struct udp_target.
Andre Noll [Wed, 6 Oct 2010 21:23:19 +0000 (23:23 +0200)]
Remove unused field "sc" of struct udp_target.

All functions get/pass a struct sender_client which contains a struct
udp_target as its private data, but we never need to get the struct
sender_client from a struct udp_target. So the sc pointer is unnecessary.

10 years agoudp_send: Add/remove the udp socket fd only once.
Andre Noll [Wed, 6 Oct 2010 20:56:23 +0000 (22:56 +0200)]
udp_send: Add/remove the udp socket fd only once.

Currently, we remove this fd from the close-on-fork list in
udp_close_target() and re-add it in udp_init_session(). This is wrong
because we want the fd being closed in the command handlers also if
the child process for the command is spawned while the udp sender
is inactive.

So add/delete the fd when the udp target is added/removed rather than
on a per session basis.

10 years agoFix a fd leak in the udp sender.
Andre Noll [Wed, 6 Oct 2010 20:51:30 +0000 (22:51 +0200)]
Fix a fd leak in the udp sender.

When deleting the UDP target, we forgot to close the socket fd.

10 years agovss: Avoid potential double free of FEC parameters.
Andre Noll [Mon, 16 Aug 2010 07:29:59 +0000 (09:29 +0200)]
vss: Avoid potential double free of FEC parameters.

If an error occurs in initialize_fec_client() we free the FEC parameter
structure but do not invalidate it. It's both easier and safer to
free this struct just before allocating it.

10 years agoIntroduce per group slice sizes.
Andre Noll [Sat, 7 Aug 2010 16:36:45 +0000 (18:36 +0200)]
Introduce per group slice sizes.

While the FEC parameters k and n are fixed, the size of a FEC slice may
be different for each FEC group. This patch exploits this freedom and
implements variable sized FEC slices for the DCCP and UDP transports.

Two new functions, compute_group_size() and compute_slice_size(),
are introduced which try to compute an optimal size for the entire
FEC group and the slice size of the group respectively.

The group size is chosen such that the group duration is approximately
150ms. Larger values cause too much latency while smaller groups use
the available bandwidth ineffectively. Several contraints such as
the maximal packet size are taken into account when computing the
group size.

Once the group size is known, a suitable slice size is chosen. It
should be as small as possible to avoid unnecessary FEC calculations
but must be large enough to guarantee that the k data slices suffice
to encode the header (if needed) and the data chunk(s).

10 years agovss: need_audio_header(): Change return value to bool.
Andre Noll [Sat, 7 Aug 2010 16:33:04 +0000 (18:33 +0200)]
vss: need_audio_header(): Change return value to bool.

Obviously an equivalent transformation.

10 years agoCompute the maximal chunk size only once.
Andre Noll [Sat, 7 Aug 2010 15:46:44 +0000 (17:46 +0200)]
Compute the maximal chunk size only once.

This patch adds the new field max_chunk_size to struct audio_file_data.
It is initialized in the afs process when the chunk table is saved,
just before the audio_file_data struct is passed to the server process.

The vss code benefits from this change since it can just use the
new information rather than calling afh_get_largest_chunk_size()
to recompute the maximal chunk size from scratch whenever a new
client connects.

Since vss.c was the only user of afh_get_largest_chunk_size(), we
may kill this function.

10 years agoReplace ROUND_UP by DIV_ROUND_UP and use it everywhere.
Andre Noll [Sat, 7 Aug 2010 15:07:12 +0000 (17:07 +0200)]
Replace ROUND_UP by DIV_ROUND_UP and use it everywhere.

10 years agoRemove FEC parameter max_slice_bytes.
Andre Noll [Sat, 24 Jul 2010 18:11:08 +0000 (20:11 +0200)]
Remove FEC parameter max_slice_bytes.

It is no longer a FEC parameter as its value is determined at runtime
from the MPS of the connection.

So replace ->max_slice_bytes of struct fec_client_parms by the new
->mps field of struct fec_client.

10 years agoSimplify vss_send().
Andre Noll [Sat, 24 Jul 2010 16:52:37 +0000 (18:52 +0200)]
Simplify vss_send().

10 years agoReplace check for stream_chunk < 0 by state == NONE.
Andre Noll [Sat, 24 Jul 2010 16:46:25 +0000 (18:46 +0200)]
Replace check for stream_chunk < 0 by state == NONE.

The two conditions are equivalent but the latter form is more readable.

10 years agoFEC initialization cleanups.
Andre Noll [Sat, 7 Aug 2010 14:50:59 +0000 (16:50 +0200)]
FEC initialization cleanups.

Initialization was performed in two steps: During vss_send and
during compute_next_fec_slice(). This patch moves the init code
for fec clients to the single location.

10 years agovss: Cleanup num_slices().
Andre Noll [Sat, 7 Aug 2010 14:45:41 +0000 (16:45 +0200)]
vss: Cleanup num_slices().

This makes num_slices() take ints rather than uint8_t arguments which
avoids nasty integer overflows and allows to drop the result parameter
since the result is now contained in the return value.

Moreover, num_slices() now takes the number of redundant slices as
an additional argument, so that this value has to be comuted only
once in the caller.

10 years agoNEWS update.
Andre Noll [Sun, 24 Oct 2010 11:58:12 +0000 (13:58 +0200)]
NEWS update.

10 years agoMerge branch 't/sched_cleanups'
Andre Noll [Sun, 17 Oct 2010 11:04:55 +0000 (13:04 +0200)]
Merge branch 't/sched_cleanups'

10 years agopara.h, recv.h: Move parts marked with \cond to bottom.
Andre Noll [Mon, 11 Oct 2010 20:38:38 +0000 (22:38 +0200)]
para.h, recv.h: Move parts marked with \cond to bottom.

This makes doxygen much happier as it does no longer spit out
tons of messages like

warning: documentation for unknown define VERSION_TEXT found.

10 years agoMerge remote branch 'refs/remotes/fml/master'
Andre Noll [Mon, 11 Oct 2010 19:21:57 +0000 (21:21 +0200)]
Merge remote branch 'refs/remotes/fml/master'

10 years agoudp_recv: Add missing line break in log messages.
Andre Noll [Mon, 11 Oct 2010 19:15:12 +0000 (21:15 +0200)]
udp_recv: Add missing line break in log messages.

Also reduce the line length a bit and fix a whitespace issue.

10 years agopara.h: Fix typo in comment.
Andre Noll [Mon, 11 Oct 2010 13:38:12 +0000 (15:38 +0200)]
para.h: Fix typo in comment.

10 years agoweb/download.html: Recommend downloading the snapshot tarball.
Andre Noll [Sun, 10 Oct 2010 20:02:14 +0000 (22:02 +0200)]
web/download.html: Recommend downloading the snapshot tarball.

Mention that the git tarball is a snapshot of the master branch and
is expected to be more stable than the released versions.

10 years agoNEWS update.
Andre Noll [Sun, 10 Oct 2010 19:43:52 +0000 (21:43 +0200)]
NEWS update.

10 years agodccp_send: Fix com_off().
Andre Noll [Sun, 10 Oct 2010 19:37:03 +0000 (21:37 +0200)]
dccp_send: Fix com_off().

This command made para_server sefgault due to the FEC client structure being
freed, but still used from vss.c. Fix this serious bug by first calling
dccp_shutdown_clients() which removes all FEC clients.

10 years agoaudiod: Handle crashes of para_server more robustly.
Andre Noll [Sun, 10 Oct 2010 19:32:58 +0000 (21:32 +0200)]
audiod: Handle crashes of para_server more robustly.

In case para_server dies unexpectedly, para_audiod may still have a partial status
item in the buffer tree node of the client task which can lead to a busy loop.

Fix this by flushing the input queue and invalidate the current audio format to prevent
recreating a buffer tree before the status task resumes.

There is already flush_input_queue() in buffer_tree.c but that can't be called by audiod.c
as it is a static function. Make it public and rename it to btr_drain() which is shorter and
more to the point and has the usual btr_ prefix.

10 years agoAdd missing documentation of moods_event_handler() and btr_free_node().
Andre Noll [Sun, 10 Oct 2010 16:17:53 +0000 (18:17 +0200)]
Add missing documentation of moods_event_handler() and btr_free_node().

10 years agoconnect_local_socket(): Silence noisy debug message.
Andre Noll [Tue, 5 Oct 2010 06:10:54 +0000 (08:10 +0200)]
connect_local_socket(): Silence noisy debug message.

10 years agosched: Use built-in instead of open-coded timeout helpers.
Andre Noll [Sun, 1 Aug 2010 17:56:10 +0000 (19:56 +0200)]
sched: Use built-in instead of open-coded timeout helpers.

In many places, sched_min_delay() or sched_request_timeout() are
open-coded.  Use the corresponding helpers from sched.c instead and
rename the "timeout" parameter of sched_request_timeout() to avoid
warnings on old gcc versions.

10 years agoImprove filter help text.
Andre Noll [Mon, 23 Aug 2010 13:25:07 +0000 (15:25 +0200)]
Improve filter help text.

- Mention that "efficient way" means zero-copy,

- Replace "filter options" by "options for a particular filter",

- Replace -f by --filter.

10 years agonet: Rename create_remote_socket() connect_local_socket().
Andre Noll [Mon, 23 Aug 2010 13:14:48 +0000 (15:14 +0200)]
net: Rename create_remote_socket() connect_local_socket().

It's all about local unix domain sockets, so avoid the word
"remote". Also, this function does not create the socket special
in /var/paraslash.

10 years agofade: Print the mixer device.
Andre Noll [Mon, 23 Aug 2010 11:39:16 +0000 (13:39 +0200)]
fade: Print the mixer device.

Otherwise the error "no such file" can be misleading.

10 years agoafh: Chunk numbers are zero-based.
Andre Noll [Tue, 17 Aug 2010 20:11:24 +0000 (22:11 +0200)]
afh: Chunk numbers are zero-based.

This makes the numbers match what para_server logs.

10 years agoFix an invalid-free-bug in the ogg audio format handler code.
Andre Noll [Mon, 9 Aug 2010 22:15:04 +0000 (00:15 +0200)]
Fix an invalid-free-bug in the ogg audio format handler code.

In process_ogg_packets(), if ogg_sync_pageout() fails, we jmp to
the out label where "stream" is being freed by ogg_stream_clear()
without being initialized.

This causes para_afh and para_server to segfault in libogg:

*** glibc detected *** para_afh: munmap_chunk(): invalid pointer: 0x6f890d42 ***
======= Backtrace: =========
/usr/lib/glibc/lib/libc.so.6(+0x66e5a)[0x6f712e5a]
/usr/lib/glibc/lib/libc.so.6(+0x68021)[0x6f714021]
/usr/local/lib/libogg.so.0(ogg_stream_clear+0x2f)[0x6f891adf]
para_afh[0x804cb52]
para_afh[0x804cbeb]
para_afh[0x804ce59]
para_afh[0x804be71]
para_afh[0x804a97c]
/usr/lib/glibc/lib/libc.so.6(__libc_start_main+0xd9)[0x6f6c2c79]
para_afh[0x8049991]

Fix this bug by returning the error code directly rather than jumping
to "out".

10 years agonew codename, reset version to git
Andre Noll [Fri, 6 Aug 2010 12:13:56 +0000 (14:13 +0200)]
new codename, reset version to git

10 years agothe paraslash-0.4.4 release tarball
Andre Noll [Fri, 6 Aug 2010 12:09:52 +0000 (14:09 +0200)]
the paraslash-0.4.4 release tarball

10 years agoparaslash 0.4.4 v0.4.4
Andre Noll [Fri, 6 Aug 2010 12:05:04 +0000 (14:05 +0200)]
paraslash 0.4.4

10 years agomanual: Minor cleanups.
Andre Noll [Mon, 2 Aug 2010 07:12:16 +0000 (09:12 +0200)]
manual: Minor cleanups.

- Mention that OGG/Speex also needs periodic audio file headers, reword
this sentence.

- para_write now understands formats != 16 bit, so remove this restriction also
from the documentation. Also mention that the sample format is being read from
the wav header, if present.

- Typo fix.

10 years agoFix typo in write_common.h.
Andre Noll [Sun, 1 Aug 2010 17:48:30 +0000 (19:48 +0200)]
Fix typo in write_common.h.

This makes doxygen happy again.

10 years agoRename some variables to avoid warnings on old gcc versions.
Andre Noll [Sun, 1 Aug 2010 17:46:32 +0000 (19:46 +0200)]
Rename some variables to avoid warnings on old gcc versions.

Fortunately, gcc-4 no longer warns on these. However, gcc-3.3 is still supported
for compiling paraslash, so rename "index" to "idx" and "conf" to "config" to get
rid of warnings of the form

wmadec_filter.c:1013: warning: declaration of `index' shadows a global declaration

Add documentation of ->free_config() while we're at it.

10 years agoNEWS update.
Andre Noll [Sun, 1 Aug 2010 17:38:33 +0000 (19:38 +0200)]
NEWS update.

10 years agoMerge branch 't/speex'
Andre Noll [Sun, 1 Aug 2010 11:26:42 +0000 (13:26 +0200)]
Merge branch 't/speex'

10 years agoMerge branch 't/error2'
Andre Noll [Sun, 1 Aug 2010 11:25:59 +0000 (13:25 +0200)]
Merge branch 't/error2'

10 years agoNEWS update
Andre Noll [Thu, 29 Jul 2010 18:55:12 +0000 (20:55 +0200)]
NEWS update

10 years agoMerge branch 't/bits_per_sample'
Andre Noll [Thu, 29 Jul 2010 18:52:06 +0000 (20:52 +0200)]
Merge branch 't/bits_per_sample'

10 years agoalsa: Add a missing "goto again" statement.
Andre Noll [Mon, 26 Jul 2010 11:02:56 +0000 (13:02 +0200)]
alsa: Add a missing "goto again" statement.

Without this the return value of btr_node_status() might be incorrect
as the wrong minimal input queue size was used.

10 years agoAdd a reference to the speex home page.
Andre Noll [Thu, 29 Jul 2010 16:25:41 +0000 (18:25 +0200)]
Add a reference to the speex home page.

Also mention the name of the speex Debian package.

10 years agoAdd missing documentation of rfds parameter of accept_sender_client().
Andre Noll [Tue, 27 Jul 2010 21:19:31 +0000 (23:19 +0200)]
Add missing documentation of rfds parameter of accept_sender_client().

Fixes

/home/work/maan/scm/paraslash/send_common.c:260: warning: The following parameters of accept_sender_client(struct sender_status *ss, fd_set *rfds) are not documented:
parameter 'rfds'

10 years agoaacdec: Fix a brown paper bag bug.
Andre Noll [Mon, 26 Jul 2010 11:05:42 +0000 (13:05 +0200)]
aacdec: Fix a brown paper bag bug.

Oops, we were using PARA_MAX() instead of PARA_MIN() to determine
the number of bytes to convert in the next run.  This silly bug could
lead to a segfault in the aacdec filter. Moreover, the assertion in
buffer_tree.c that would have caught the bug was _also_ b0rken.

But it's not my fault. I blame Grover of the Sesame street for both
bugs. He did explain the difference between near and far, although
this knowledge is next to useless for a programmer. And he did not
say a single word about MIN and MAX, or true and false. No wonder
such bugs happen.

10 years agoMake autoconf-2.66 happy.
Andre Noll [Mon, 5 Jul 2010 21:42:31 +0000 (23:42 +0200)]
Make autoconf-2.66 happy.

This removes more lines than it adds and avoids the following warning;

configure.ac:689: warning: AC_DEFINE_UNQUOTED: `
configure.ac:689:       result=
configure.ac:689:       for i in $status_items; do
configure.ac:689:               result="$result SI_$(echo $i | tr 'a-z' 'A-Z')' is not a valid preprocessor define value