Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | lib/nfs/Connection: add debug flag "in_destroy" | Max Kellermann | 2014-12-15 | 2 | -0/+11 | |
| | | ||||||
* | | lib/nfs/Connection: add assertion | Max Kellermann | 2014-12-14 | 1 | -0/+2 | |
| | | ||||||
* | | lib/nfs/Connection: fix reconnect after mount failure | Max Kellermann | 2014-12-14 | 2 | -1/+2 | |
| | | | | | | | | | | | | | | When mounting had not yet finished, SocketMonitor::IsDefined() was always false, due to the workaround at the beginning of the function that calls SocketMonitor::Steal(). This commit drops the IsDefined() check because it was never necessary and breaks reconnect. | |||||
* | | lib/nfs/Connection: fix typo in code comment | Max Kellermann | 2014-12-14 | 1 | -1/+1 | |
| | | ||||||
* | | lib/nfs/Connection: fix memory leak (and assertion failure) | Max Kellermann | 2014-12-14 | 4 | -0/+29 | |
| | | | | | | | | | | | | | | | | | | | | | | nfs_destroy_context() will invoke all pending callbacks with err==-EINTR. In CancellableCallback::Callback(), this will invoke NfsConnection::DeferClose(), which however is only designed to be called from nfs_service(). In non-debug mode, this will leak memory because nfs_close_async() is never called. Workaround: before nfs_destroy_context(), invoke nfs_close_async() on all pending file handles. | |||||
* | | lib/nfs/Connection: move code to Service() | Max Kellermann | 2014-12-14 | 2 | -12/+28 | |
| | | ||||||
* | | lib/nfs/Connection: add assertions | Max Kellermann | 2014-12-14 | 1 | -0/+2 | |
| | | ||||||
* | | lib/nfs/Connection: move code to method InternalClose() | Max Kellermann | 2014-12-14 | 2 | -3/+17 | |
| | | ||||||
* | | lib/nfs/Connection: make in_service and in_event debug-only flags | Max Kellermann | 2014-12-14 | 2 | -0/+11 | |
| | | ||||||
* | | lib/nfs/Connection: fix crash while canceling a failing Open() | Max Kellermann | 2014-12-14 | 2 | -2/+6 | |
| | | | | | | | | | | | | | | The method NfsConnection::CancellableCallback::Callback() will always invoke NfsConnection::Close() on the file handle, even if the void pointer is not a nfsfh. This can happen if the Open() was not successful, e.g. when the file does not exist. | |||||
* | | event/Loop: read the "again" flag while holding mutex | Max Kellermann | 2014-12-14 | 1 | -1/+2 | |
| | | ||||||
* | | lib/nfs/Connection: add assertions | Max Kellermann | 2014-12-14 | 1 | -0/+31 | |
| | | ||||||
* | | util/HugeAllocator: disable MEM_LARGE_PAGES on Windows | Max Kellermann | 2014-12-12 | 2 | -1/+3 | |
| | | | | | | | | | | | | MEM_LARGE_PAGES does not appear to work. Instead, MEM_RESERVE appears to be necessary. Until I figure this out, this large pages are disabled. | |||||
* | | input/Init: eliminate double colon from log message | Max Kellermann | 2014-12-12 | 1 | -1/+1 | |
| | | ||||||
* | | playlist/Print: don't skip non-existent songs in "listplaylist" | Max Kellermann | 2014-12-09 | 2 | -6/+9 | |
| | | | | | | | | | | | | | | | | Skipping those songs silently will confuse the client, because commands specifying the song index within a playlist (e.g. playlistdelete) will be out of sync. This copies spl_print()'s behavior to playlist_file_print(). | |||||
* | | configure.ac: prepare for 0.19.7 | Max Kellermann | 2014-12-09 | 2 | -2/+4 | |
| | | ||||||
* | | release v0.19.6 | Max Kellermann | 2014-12-08 | 2 | -3/+3 | |
| | | ||||||
* | | android/build.py: update FFmpeg to 2.5 | Max Kellermann | 2014-12-08 | 2 | -2/+3 | |
| | | ||||||
* | | Merge tag 'v0.18.20' into v0.19.x | Max Kellermann | 2014-12-08 | 3 | -2/+15 | |
|\| | ||||||
| * | release v0.18.20 | Max Kellermann | 2014-12-08 | 1 | -1/+1 | |
| | | ||||||
| * | decoder/ffmpeg: support FFmpeg 2.5 | Max Kellermann | 2014-12-08 | 2 | -0/+7 | |
| | | | | | | | | | | | | Version 2.5 fixed an API oddity, however it broke API compatibility, at least with C++. Disable the workaround when a libavformat version is detected that is recent enough. | |||||
| * | thread/Id: drop "::" prefix before pthread function names | Max Kellermann | 2014-12-08 | 2 | -2/+3 | |
| | | | | | | | | | | | | The "::" to explicitly refer to the global namespace appeared like a good idea in C++, but it breaks with C libraries that implement standard functions using macros (e.g. musl). | |||||
| * | configure.ac: prepare for 0.18.20 | Max Kellermann | 2014-12-08 | 2 | -2/+4 | |
| | | ||||||
* | | Output: start with a null mixer. | Nix | 2014-12-01 | 1 | -0/+1 | |
| | | | | | | | | | | | | | | | | | | | | | | There are code paths (mostly error cases) in which it is possible to initialize an AudioOutput and then kill it without ever calling audio_output_new(). In such a case, its destructor will attempt to free a mixer that was never initialized, leading to an attempt to take out a lock on a mutex that was similarly never initialized, which hangs forever. Fix by always initializing the mixer appropriately. | |||||
* | | android/build.py: update ffmpeg 2.4.3, curl 7.39 | Max Kellermann | 2014-11-28 | 1 | -4/+4 | |
| | | ||||||
* | | android: update libFLAC to 1.3.1 | Max Kellermann | 2014-11-28 | 2 | -2/+4 | |
| | | | | | | | | Due to security vulnerabilities. | |||||
* | | .gitignore: ignore /lib/ | Max Kellermann | 2014-11-28 | 1 | -0/+2 | |
| | | ||||||
* | | configure.ac: prepare for 0.19.6 | Max Kellermann | 2014-11-28 | 2 | -2/+4 | |
| | | ||||||
* | | release v0.19.5 | Max Kellermann | 2014-11-26 | 2 | -3/+3 | |
| | | ||||||
* | | Makefile.am: distribute Android sources | Max Kellermann | 2014-11-26 | 1 | -0/+7 | |
| | | ||||||
* | | Merge tag 'v0.18.19' into v0.19.x | Max Kellermann | 2014-11-26 | 2 | -4/+11 | |
|\| | ||||||
| * | release v0.18.19 | Max Kellermann | 2014-11-26 | 1 | -1/+1 | |
| | | ||||||
| * | archive/zzip: fix inverted error handler | Max Kellermann | 2014-11-24 | 2 | -4/+7 | |
| | | | | | | | | | | Set the Error when zzip_seek()==-1 and not on success. Fixes a crash after seeking. | |||||
| * | configure.ac: prepare for 0.18.19 | Max Kellermann | 2014-11-24 | 2 | -2/+4 | |
| | | ||||||
* | | lib/nfs/FileReader: clean up on disconnect | Max Kellermann | 2014-11-25 | 1 | -1/+1 | |
| | | | | | | | | | | Avoids crash because Close() invokes a call on a destructed NfsConnection. | |||||
* | | lib/nfs/FileReader: move code to CancelOrClose() | Max Kellermann | 2014-11-25 | 2 | -0/+16 | |
| | | ||||||
* | | lib/nfs/FileReader: reset state in OnNfsConnectionFailed() | Max Kellermann | 2014-11-25 | 1 | -0/+2 | |
| | | | | | | | | | | Avoid calling NfsConnection::RemoveLease(), because the lease has been removed already. | |||||
* | | lib/nfs/FileReader: update "state" in OnNfsError() | Max Kellermann | 2014-11-25 | 1 | -0/+24 | |
| | | | | | | | | | | | | Clean up the "state" to indicate that there is no longer any asynchronous operation. Fixes another NFS-related crash due to cleanup of a non-existing asynchronous operation. | |||||
* | | lib/nfs/Manager: defer NfsConnection destruction | Max Kellermann | 2014-11-25 | 3 | -9/+56 | |
| | | | | | | | | | | Avoids a crash that occurs when NfsConnection::OnSocketReady() dereferences itself before returning. | |||||
* | | lib/nfs/Connection: broadcast error before closing connection | Max Kellermann | 2014-11-25 | 1 | -4/+4 | |
| | | | | | | | | | | | | During the NfsLease::OnNfsConnectionFailed() call, the old (defunct) nfs_context may be used to close file handles. Such code does not yet exist, but will be added soon to fix other bugs. | |||||
* | | lib/nfs/Connection: cancel DeferredMonitor on disconnect | Max Kellermann | 2014-11-25 | 1 | -0/+4 | |
| | | | | | | | | | | Fixes potential second mount attempt after the old connection to the NFS server was shut down. | |||||
* | | lib/nfs/FileReader: include Compiler.h for "final" fallback | Max Kellermann | 2014-11-25 | 1 | -0/+1 | |
| | | ||||||
* | | decoder/mp4v2: remove because of incompatible license | Max Kellermann | 2014-11-25 | 7 | -394/+1 | |
| | | | | | | | | | | | | | | | | | | libmp4v2 is licensed under MPL 1.1, which is incompatible with GPLv2. Unfortunately, this means that we must remove the plugin. More information can be found in the Debian bug report: http://bugs.debian.org/767504 | |||||
* | | event/DeferredMonitor: include cleanup | Max Kellermann | 2014-11-25 | 2 | -3/+2 | |
| | | ||||||
* | | test/run_input: make variables more local | Max Kellermann | 2014-11-25 | 1 | -12/+8 | |
| | | ||||||
* | | decoder/{dsdiff,dsf,opus}: fix deadlock while seeking | Max Kellermann | 2014-11-24 | 3 | -3/+5 | |
| | | ||||||
* | | doc/protocol: mention that "count" can have multiple filters | Max Kellermann | 2014-11-24 | 1 | -0/+1 | |
| | | ||||||
* | | configure.ac: prepare for 0.19.5 | Max Kellermann | 2014-11-24 | 2 | -2/+4 | |
| | | ||||||
* | | android: release v0.19.4 | Max Kellermann | 2014-11-24 | 1 | -2/+2 | |
| | | | | | | | | Android releases were missing since 0.19.1. | |||||
* | | android: switch to gcc 4.9 / llvm 3.5 (NDK r10c) | Max Kellermann | 2014-11-24 | 1 | -2/+2 | |
| | |