aboutsummaryrefslogtreecommitdiffstats
path: root/src (unfollow)
Commit message (Collapse)AuthorFilesLines
2014-11-18QueueCommands: workaround for buggy clients that send "add /"Max Kellermann1-1/+10
2014-11-18Client: assume uid==0 is local socketMax Kellermann2-2/+2
A negative uid value means it's not a "local socket" (PF_LOCAL). uid==0 means user "root" connected.
2014-11-18event/ServerSocket: fix get_remote_uid() error valueMax Kellermann1-1/+1
Must return -1 on error, not 0. 0 is root.
2014-11-12decoder/opus: add MIME types audio/ogg and application/oggMax Kellermann1-0/+7
2014-11-11Construct a Null AllocatedPath if the filename conversion into UTF8 failedFlorent Le Coz1-1/+5
2014-11-11decoder/opus: support chained streamsMax Kellermann1-1/+36
2014-11-11decoder/opus: move code to HandleEOS()Max Kellermann1-1/+8
2014-11-11decoder/opus: improved error loggingMax Kellermann1-4/+11
2014-11-11decoder/opus: fix mistyped LoadEOSPacket() return valueMax Kellermann1-1/+1
2014-11-11decoder/opus: eliminate flag "found_opus"Max Kellermann1-5/+3
Check opus_decoder!=nullptr instead.
2014-11-11decoder/opus: add constexpr output_buffer_framesMax Kellermann1-8/+10
2014-11-11Main: fix compilation on OS X using non-Apple compilersMisty De Meo1-2/+2
Commit d42c0f1dc5063d50a62817b63a1c2a4507c46071 added an OS X-specific method of calling mpd_main_after_fork(), which uses Grand Central Dispatch. Since this uses a block literal, it breaks compilation on compilers which don't support the block extension, e.g. non-Apple compilers. This affects users on older OS X releases with GCD (which depend on older Clang releases, or Apple GCCs, which don't support the C++11 features MPD needs); or which don't support GCD at all (10.5 and lower). This patch changes the #ifdef so that the non-GCD code is used as it was on OS X before this patch if blocks aren't available, via checking __BLOCKS__ macro.
2014-11-10pcm/SoxrResampler: round output buffer size upMax Kellermann1-1/+2
The old formula calculates the output buffer size with "regular" rounding (to the nearest integer), however sometimes, that is insufficient and the last sample cannot be resampled. This causes audible distortions. By changing the formula to consider the worst case (always round up), this problem is eliminated.
2014-11-10decoder/ffmpeg: support opusMax Kellermann1-1/+2
2014-11-10decoder/audiofile: fix bit rate calculationMax Kellermann1-1/+1
2014-11-10input/curl: ignore ResponseBoundary() while seekingMax Kellermann1-0/+4
While seeking, metadata must not be updated. ResponseBoundary() was added in MPD 0.19.1, but I forgot to add the IsSeeking() check there. This caused the "seekable" flag to reset.
2014-11-08tag/Set: do AlbumArtist/Artist fallback only if AlbumArtist is not disabledMax Kellermann1-0/+2
On "list albumartist", songs that have no AlbumArtist tag will use the Artist tag. However, if AlbumArtist is disabled via "metadata_to_use", the TagBuilder::AddItem() call is ignored, and PrintUniqueTag() attempts to print a nullptr string. This commit fixes the problem by attempting the fallback only if AlbumArtist is not disabled.
2014-11-08db/Count: include cleanupMax Kellermann1-1/+1
2014-11-07decoder/ffmpeg: support opusMax Kellermann1-1/+2
2014-11-07db/upnp: fix valgrind warningMax Kellermann1-0/+1
2014-11-07lib/upnp/ContentDirectoryService: swap uri_apply_base() parametersMax Kellermann1-1/+1
When uri_apply_base() was moved from db/upnp/Util.cpp to util/UriUtil.cpp, the parameter order was changed, however without swapping the parameters in the ContentDirectoryService constructor.
2014-11-07input/AsyncInputStream: set Error when seeking unseekableMax Kellermann1-1/+4
Fixes crash in the "audiofile" decoder while logging the seek error.
2014-11-07input/Open: expose input_domainMax Kellermann3-2/+52
2014-11-02input/curl: forget Content-Length (and more) after redirectMax Kellermann1-0/+26
Fixes playback of redirected streams.
2014-11-02AsyncInputStream: add method ClearTag()Max Kellermann1-0/+4
2014-11-02InputStream: add method ClearMimeType()Max Kellermann1-0/+4
2014-11-02input/curl: fix curl_easy_setopt() parameter typesMax Kellermann1-4/+4
2014-11-02Decoder, Playlist: ignore URI query string for plugin detectionMax Kellermann2-6/+8
Use the new uri_get_suffix() overload that removes the query string.
2014-11-02util/UriUtil: add uri_get_suffix() overload that ignores query stringMax Kellermann2-0/+28
2014-11-02PlaylistFile: don't allow empty playlist nameMax Kellermann1-0/+4
2014-11-02playlist/m3u: recognize the file suffix ".m3u8"Max Kellermann2-1/+3
2014-11-02decoder/faad: remove workaround for ancient libfaad2 ABI bugMax Kellermann1-10/+2
Many years ago, FAAD had a serious ABI bug: the NeAACDecInit() prototype in its header declared the "samplerate" parameter to be "unsigned long *", but internally, the function assumed it was "uint32_t *" instead. On 32 bit machines, that was no difference, but on 64 bit, this left one portion of the return value uninitialized; and worse, on big-endian, the wrong word was filled. This bug had to be worked around in MPD (commit 9c4e97a6). A few months later, the bug was fixed in the FAAD CVS in commit 1.117 on file libfaad/decoder.c; the commit message was: "Use public headers internally to prevent duplicate declarations" The commit message was too brief at best; the problem was not duplicate declarations, but a prototype mismatch. No mention of the bug fix in the ChangeLog. The MPD project never learned about this bug fix, and so MPD would always pass a "uin32_t *" dressed up as a "unsigned long *". Nearly 6 years later, it's about time to fix this second ABI problem. Let's kill the workaround!
2014-11-02decoder/ffmpeg: recognize MIME type audio/aacpSteven OBrien1-0/+1
2014-11-02output/roar: remove unnecessary "volatile" keywordMax Kellermann1-1/+1
A mutex acts as a memory barrier, and thus "volatile" is not necessary.
2014-11-02TagString: use g_strndup() for unterminated stringMax Kellermann1-1/+1
Fixes buffer overflow bug.
2014-11-02lib/upnp/Discovery: add missing stdlib.h includeMax Kellermann1-0/+1
2014-11-01input/curl: fix curl_easy_setopt() parameter typesMax Kellermann1-3/+3
2014-11-01Decoder, Playlist: ignore URI query string for plugin detectionMax Kellermann3-5/+10
Use the new uri_get_suffix() overload that removes the query string.
2014-11-01util/UriUtil: add uri_get_suffix() overload that ignores query stringMax Kellermann2-0/+28
2014-10-31Main: run the OS X native event loop after forkingNanoTech1-0/+27
2014-10-31Revert "Main: run the OS X native event loop"NanoTech3-50/+0
This reverts commit f0be48ff90503d9ffa5b295fd4454eec753950ee (except for the NEWS entry). If libdispatch (GCD) is used before forking, it can't safely be used again after forking.
2014-10-31PlaylistFile: don't allow empty playlist nameMax Kellermann1-0/+4
2014-10-28decoder/mad: fix negative replay gain valuesMax Kellermann1-1/+1
Negating an unsigned integer does not work.
2014-10-28TextInputStream: don't ignore unterminated last lineMax Kellermann1-4/+16
2014-10-26playlist/m3u: recognize the file suffix ".m3u8"Max Kellermann2-0/+2
2014-10-25OutputThread: fall back to PCM if given DSD sample rate is not supportedMax Kellermann1-0/+30
Works around the "PCM conversion from f to dsd is not implemented" error message that prevents DSD playback.
2014-10-25decoder/faad: remove workaround for ancient libfaad2 ABI bugMax Kellermann2-20/+4
Many years ago, FAAD had a serious ABI bug: the NeAACDecInit() prototype in its header declared the "samplerate" parameter to be "unsigned long *", but internally, the function assumed it was "uint32_t *" instead. On 32 bit machines, that was no difference, but on 64 bit, this left one portion of the return value uninitialized; and worse, on big-endian, the wrong word was filled. This bug had to be worked around in MPD (commit 9c4e97a6). A few months later, the bug was fixed in the FAAD CVS in commit 1.117 on file libfaad/decoder.c; the commit message was: "Use public headers internally to prevent duplicate declarations" The commit message was too brief at best; the problem was not duplicate declarations, but a prototype mismatch. No mention of the bug fix in the ChangeLog. The MPD project never learned about this bug fix, and so MPD would always pass a "uin32_t *" dressed up as a "unsigned long *". Nearly 6 years later, it's about time to fix this second ABI problem. Let's kill the workaround!
2014-10-25decoder/ffmpeg: recognize MIME type audio/aacpSteven OBrien1-0/+1
2014-10-25fs/Charset: work around clang's -Wunused-const-variableMax Kellermann1-1/+2
MPD_PATH_MAX_UTF8 is only used by GLib-specific code currently.
2014-10-24OutputThread: close the output plugin after filter failureMax Kellermann1-0/+1
Fixes memory leak because ao_plugin_close() never gets called.