aboutsummaryrefslogtreecommitdiffstats
path: root/src/decoder/SndfileDecoderPlugin.cxx
diff options
context:
space:
mode:
authorMarcello Desantis <marcello@casadesantis.it>2014-04-09 23:58:56 +0200
committerMax Kellermann <max@duempel.org>2014-04-09 23:58:56 +0200
commit95ac6071b9fdaa543bd0c8ab7665d262db708683 (patch)
treee8bd8f2548f2f4840596660cb8bf04f9b24d4cc1 /src/decoder/SndfileDecoderPlugin.cxx
parent3a4e667078e13f27b3d196d3b3a56f1f39be2c75 (diff)
downloadmpd-95ac6071b9fdaa543bd0c8ab7665d262db708683.tar.gz
mpd-95ac6071b9fdaa543bd0c8ab7665d262db708683.tar.xz
mpd-95ac6071b9fdaa543bd0c8ab7665d262db708683.zip
decoder/sndfile: work around libsndfile bug on partial read
Diffstat (limited to 'src/decoder/SndfileDecoderPlugin.cxx')
-rw-r--r--src/decoder/SndfileDecoderPlugin.cxx26
1 files changed, 20 insertions, 6 deletions
diff --git a/src/decoder/SndfileDecoderPlugin.cxx b/src/decoder/SndfileDecoderPlugin.cxx
index 3360cd1bf..77b132962 100644
--- a/src/decoder/SndfileDecoderPlugin.cxx
+++ b/src/decoder/SndfileDecoderPlugin.cxx
@@ -55,14 +55,28 @@ sndfile_vio_read(void *ptr, sf_count_t count, void *user_data)
{
InputStream &is = *(InputStream *)user_data;
+ sf_count_t total_bytes = 0;
Error error;
- size_t nbytes = is.LockRead(ptr, count, error);
- if (nbytes == 0 && error.IsDefined()) {
- LogError(error);
- return -1;
- }
- return nbytes;
+ /* this loop is necessary because libsndfile chokes on partial
+ reads */
+
+ do {
+ size_t nbytes = is.LockRead((char *)ptr + total_bytes,
+ count - total_bytes, error);
+ if (nbytes == 0) {
+ if (error.IsDefined()) {
+ LogError(error);
+ return -1;
+ }
+
+ break;
+ }
+
+ total_bytes += nbytes;
+ } while (total_bytes < count);
+
+ return total_bytes;
}
static sf_count_t