| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
The soure file client.c has nearly 1000 lines, time for splitting it
into smaller pieces to improve readability.
|
|
|
|
| |
Prepare splitting client.c into several sources.
|
|
|
|
|
|
| |
Added a patch to flush out the last.fm input plugin slightly. It
basically turns it into a wrapper for the appropriate plugin. Most
notably metadata is now extracted.
|
|
|
|
|
|
|
|
|
|
|
| |
Instead of hard-coding the path "/etc/mpd.conf", use the configured
$(sysconfdir) path. This can be set with:
./configure --sysconfdir=/etc
Note that this changes the default path to "/usr/local/etc/mpd.conf",
given the default prefix "/usr/local". This is actually more correct
than the old default.
|
|
|
|
| |
Make the client list management a separate sub-library.
|
|
|
|
|
| |
The client code uses portable GLib I/O functions and doesn't need the
OS specific socket headers.
|
|
|
|
|
|
|
| |
Calculate the total play time with the audio_format object each time,
using audio_format_time_to_size(). The function
audioFormatSizeToTime() is not needed anymore, and will be removed
with this patch.
|
| |
|
|
|
|
|
| |
Don't abort the whole MPD process when the conversion fails. This has
been a denial-of-service attack vector for years.
|
|
|
|
| |
This library does not use GLib directly.
|
| |
|
|
|
|
|
| |
You cannot have a negative number of channels, let's pass it as
uint8_t instead of int8_t.
|
|
|
|
|
| |
It is illegal to pass a NULL buffer to pcm_byteswap_X(). The result
of this is that pcm_byteswap_X() never returns NULL.
|
|
|
|
|
|
| |
Currently, byteswapping is performed on the format_buffer. This can
go wrong when this buffer is used twice during one run. Add a
separate buffer for swapping the byte order.
|
|
|
|
| |
The volume plugin does not work for reverse_endian samples.
|
| |
|
|
|
|
|
| |
This line was missing in the reverse_endian patch, and led to
undefined values and crashes in that attribute.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Changed function to first close standard input (this may
fail but we don't care) and then try to open /dev/null (this
may fail but it shouldn't on Unix platforms plus we don't
know what to do in such case anyways). Since standard input
has the "zeroth" descriptor number next "open" will use it.
Since there is no "/dev/null" on Windows (It's not even
a valid path!) the second step is skipped if WIN32 is
defined.
As a final touch, since the function consists of merely two
function calls it has been moved to header file and declared
static inline.
[mk: un-inline daemonize_close_stdin()]
|
|
|
|
|
|
|
| |
On FreeBSD, "stderr" is a macro, and using this name for a struct
member breaks the build.
[mk: renamed _stderr to log_stderr]
|
|\
| |
| |
| |
| |
| | |
Conflicts:
Makefile.am
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
It makes no difference right now, but we're about to add an endianness
flag and will want to make sure it's correctly initialised every time.
|
| |
| |
| |
| |
| | |
Make "enum tag_type" the array index, and convert apeItems to a sparse
array.
|
| |
| |
| |
| | |
Improve code readability.
|
| |
| |
| |
| | |
Don't initialize those arrays each time tag_ape_load() is called.
|
| |
| |
| |
| |
| | |
Several functions work with the wrong return type, this patch fixes
them.
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
The new code is more robust and more flexible. It provides detailed
error information in GError objects.
|
| |
| |
| |
| |
| |
| |
| | |
On Windows only daemonize_close_stdin() function does
something. Other functions are either empty or generate an
error. Those have been moved to header file and declared
static inline so compiler can remove the call all together.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The functions were not "const" (they examined values pointed
by arguments passed to them, quoting gcc's doc: "Note that
a function that has pointer arguments and examines the data
pointed to must _not_ be declared 'const'.") but rather
"pure" and still not all of them.
Note also, that even some of the functions declared "pure"
are not pure, however, due to reasons stated in source code
the attribute has been kept.
|
|/
|
|
|
|
|
|
|
|
|
|
|
| |
The "group" configuration option is similar to "user" as it
sets user set what group MPD shall run as. With "user"
option, MPD changed GID to the GID of the user, however,
more control could be desired.
Moreover, the patch changes the way of checking whether no
setuid(2)/setgid(2) is required -- previously user names
were compered, now UID and GIDs are compered (ie. the one we
already have (getuid(2)/getgid(2)) with the one we want to
change to).
|
|\ |
|
| | |
|
| |
| |
| |
| |
| | |
If the encoder plugin API is disabled at compile time, don't compile
test/run_encoder.c.
|
| | |
|
| | |
|
| |
| |
| |
| |
| | |
The first patch by Patrick didn't work, because his "#ifdef HAVE_OSX"
line would have required config.h.
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| | |
Initialize flac_data.tag right after flac_data_init(). This way, the
"goto fail" won't jump to the point where tag_free(NULL) can be
called.
|
| |
| |
| |
| |
| |
| | |
On Mac OS X, the httpd plugin cannot be compiled, because OS X's
system headers do nto include sys/types.h, although they use
u_int32_t.
|
| | |
|
| |
| |
| |
| |
| | |
Don't free an internal configuration value in log_init(). Call
config_get_path() instead of manually calling parsePath().
|
| |
| |
| |
| |
| |
| |
| | |
When the filesystem_charset is changed in mpd.conf, MPD should discard
the old database. In this error branch, MPD did not fill the GError
object properly, and logged a warning message instead, which caused a
segmentation fault.
|
| |
| |
| |
| |
| |
| |
| | |
- introduce a section explaining the mpd.conf format, as done in the man page:
is it better to re-explain it here or ointing the user to the man page,
avoiding information dupplication?
- reorganizze some sections of the manual to give them a linear aspect...
|
| | |
|