diff options
| author | Max Kellermann <max@duempel.org> | 2009-11-14 22:27:04 +0100 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2009-11-14 22:27:04 +0100 |
| commit | ff70dbd316c12bf8c1edc140f0a73f0c53b81e43 (patch) | |
| tree | e718964e59b69949274ab6c3dec5c0649229f9bb | |
| parent | dd4625ce13bfbd296f69bc9795561cfbfb63c210 (diff) | |
| download | mpd-ff70dbd316c12bf8c1edc140f0a73f0c53b81e43.tar.gz mpd-ff70dbd316c12bf8c1edc140f0a73f0c53b81e43.tar.xz mpd-ff70dbd316c12bf8c1edc140f0a73f0c53b81e43.zip | |
decoder/modplug: check ModPlug_Read() < 0
Negative return values are not documented here, but since the function
prototype is signed, let's be sure.
| -rw-r--r-- | src/decoder/modplug_plugin.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/src/decoder/modplug_plugin.c b/src/decoder/modplug_plugin.c index 1d373a466..4c3302383 100644 --- a/src/decoder/modplug_plugin.c +++ b/src/decoder/modplug_plugin.c @@ -139,10 +139,8 @@ mod_decode(struct decoder *decoder, struct input_stream *is) do { ret = ModPlug_Read(f, audio_buffer, MODPLUG_FRAME_SIZE); - - if (ret == 0) { + if (ret <= 0) break; - } total_time += ret * sec_perbyte; cmd = decoder_data(decoder, NULL, |