aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* pcm_convert: use GError for error handlingMax Kellermann2009-07-2311-116/+181
| | | | | Don't abort the whole MPD process when the conversion fails. This has been a denial-of-service attack vector for years.
* pcm_resample_fallback: don't include glib.hMax Kellermann2009-07-221-1/+0
| | | | This library does not use GLib directly.
* pcm_resample_fallback: removed G_GNUC_UNUSED attributeMax Kellermann2009-07-222-6/+3
|
* pcm_channels: num_channels is unsignedMax Kellermann2009-07-222-12/+12
| | | | | You cannot have a negative number of channels, let's pass it as uint8_t instead of int8_t.
* pcm_byteswap: converted NULL checks to assertionsMax Kellermann2009-07-222-10/+5
| | | | | It is illegal to pass a NULL buffer to pcm_byteswap_X(). The result of this is that pcm_byteswap_X() never returns NULL.
* pcm_convert: added pcm_convert_state.byteswap_bufferMax Kellermann2009-07-222-3/+8
| | | | | | 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.
* filter/volume: check the flag audio_format.reverse_endianMax Kellermann2009-07-221-0/+7
| | | | The volume plugin does not work for reverse_endian samples.
* audio_format: added API documentationMax Kellermann2009-07-221-0/+48
|
* audio_format: initialize reverse_endian in audio_format_init()Max Kellermann2009-07-221-0/+1
| | | | | This line was missing in the reverse_endian patch, and led to undefined values and crashes in that attribute.
* daemon: daemonize_close_stdin() optimised.Michal Nazarewicz2009-07-222-12/+7
| | | | | | | | | | | | | | | | | | 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()]
* cmdline: renamed options.stderr to options.log_stderrEugeny N Dzhurinsky2009-07-223-6/+6
| | | | | | | On FreeBSD, "stderr" is a macro, and using this name for a struct member breaks the build. [mk: renamed _stderr to log_stderr]
* Merge branch 'master' of git://git.infradead.org/users/dwmw2/mpdMax Kellermann2009-07-1926-81/+254
|\ | | | | | | | | | | Conflicts: Makefile.am
| * Support wrong-endian ALSA outputDavid Woodhouse2009-07-191-2/+50
| |
| * Add reverse_endian field to struct audio_format and handle conversionDavid Woodhouse2009-07-197-6/+154
| |
| * Add audio_format_init() functionDavid Woodhouse2009-07-1919-73/+50
| | | | | | | | | | 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.
* | tag_ape: simplified the apeItems arrayMax Kellermann2009-07-191-22/+12
| | | | | | | | | | Make "enum tag_type" the array index, and convert apeItems to a sparse array.
* | tag_ape: moved code to tag_ape_import_item()Max Kellermann2009-07-191-12/+22
| | | | | | | | Improve code readability.
* | tag_ape: converted apeItems and tagItems to global varsMax Kellermann2009-07-191-20/+20
| | | | | | | | Don't initialize those arrays each time tag_ape_load() is called.
* | client: return "enum command_return" instead of "int"Max Kellermann2009-07-191-16/+24
| | | | | | | | | | Several functions work with the wrong return type, this patch fixes them.
* | removed buffer2array.cMax Kellermann2009-07-193-166/+0
| |
* | command: use the tokenizer libraryMax Kellermann2009-07-191-4/+50
| |
* | conf: use the tokenizer libraryMax Kellermann2009-07-192-70/+90
| |
* | tokenizer: new library replacing buffer2array()Max Kellermann2009-07-193-0/+237
| | | | | | | | | | The new code is more robust and more flexible. It provides detailed error information in GError objects.
* | daemon: Moved empty Windows version functions to header fileMichal Nazarewicz2009-07-192-21/+36
| | | | | | | | | | | | | | 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.
* | conf: replaced gcc "const" attribute with "pure"Michal Nazarewicz2009-07-191-13/+22
| | | | | | | | | | | | | | | | | | | | | | | | 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.
* | daemon: added "group" configuration optionMichal Nazarewicz2009-07-196-18/+42
|/ | | | | | | | | | | | | 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).
* Merged release 0.15.1 from branch 'v0.15.x'Max Kellermann2009-07-1611-32/+245
|\
| * mpd version 0.15.1release-0.15.1Avuton Olrich2009-07-152-2/+2
| |
| * Makefile.am: disable test/run_encoder without pluginsMax Kellermann2009-07-151-1/+3
| | | | | | | | | | If the encoder plugin API is disabled at compile time, don't compile test/run_encoder.c.
| * NEWS: fixed typoMax Kellermann2009-07-141-1/+1
| |
| * configure.ac: fail when ffmpeg is enabled, but not foundMax Kellermann2009-07-141-7/+5
| |
| * output/httpd: removed duplicate sys/types.h includeMax Kellermann2009-07-141-4/+0
| | | | | | | | | | The first patch by Patrick didn't work, because his "#ifdef HAVE_OSX" line would have required config.h.
| * doc: documented the "shout" output plugin settingsMax Kellermann2009-07-061-0/+123
| |
| * doc: documented the "pulse" output plugin settingsMax Kellermann2009-07-061-0/+34
| |
| * doc: documented the "command" setting of the "pipe" output pluginMax Kellermann2009-07-061-0/+22
| |
| * decoder/flac: fix assertion failure in tag_free() callMax Kellermann2009-07-062-2/+3
| | | | | | | | | | | | 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.
| * output/httpd: include sys/types.hMax Kellermann2009-07-062-0/+3
| | | | | | | | | | | | 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.
| * song: initialize mtime in song_alloc()Max Kellermann2009-07-061-0/+1
| |
| * log: fix double free() bug during shutdownMax Kellermann2009-07-052-6/+3
| | | | | | | | | | Don't free an internal configuration value in log_init(). Call config_get_path() instead of manually calling parsePath().
| * database: fixed NULL pointer dereference after charset changeMax Kellermann2009-06-302-4/+6
| | | | | | | | | | | | | | 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.
| * doc/user: added introductionEnrico Mioso2009-06-301-3/+33
| | | | | | | | | | | | | | - 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...
| * doc/protocol: clarified "idle database"Max Kellermann2009-06-301-1/+1
| |
| * output_thread: don't play next chunk after command==PAUSEMax Kellermann2009-06-291-1/+5
| | | | | | | | | | When the PAUSE loop ends, re-check the next command before calling ao_play() again.
| * output_all: don't resume playback when stopping during pauseMax Kellermann2009-06-292-2/+2
| | | | | | | | | | | | | | | | When MPD was paused, and the client sent the "stop" command (or "clear"), a glitch caused MPD to continue playback for a split second. This was because audio_output_all_cancel() calls audio_output_all_update(), which reopens all output devices, and re-ignites the playback loop.
| * doc: fix wording for option "follow_inside_symlinks"Enrico Mioso2009-06-261-2/+2
| |
| * configure.ac: fix the --disable-ffmpeg help textMax Kellermann2009-06-251-1/+1
| |
| * NEWS: updated NEWS file for 0.15.1Max Kellermann2009-06-252-1/+4
| |
* | mapper: use g_file_test() instead of stat()Max Kellermann2009-07-151-20/+2
| | | | | | | | The GLib functions are more portable.
* | mapper: pass music and playlist directory to mapper_init()Max Kellermann2009-07-153-19/+23
| | | | | | | | Added another glue function in main().
* | main: moved complex initialization code to glue functionsMax Kellermann2009-07-151-17/+36
| | | | | | | | | | | | The glue_*() functions act as a glue between MPD's main() function and its libraries. They handle disabled features, and pass validated configuration options.