From ae9b5a75d9560ec5ce88f93e89efccdf64594100 Mon Sep 17 00:00:00 2001 From: Darren Salt Date: Sat, 10 Oct 2009 12:12:07 +0100 Subject: MOD reported zero-length workaround (avoids divide-by-0). --- ChangeLog | 1 + src/demuxers/demux_mod.c | 2 ++ 2 files changed, 3 insertions(+) diff --git a/ChangeLog b/ChangeLog index 84c51e994..7fa1dd398 100644 --- a/ChangeLog +++ b/ChangeLog @@ -19,6 +19,7 @@ xine-lib (1.1.17) 2009-??-?? * Add a user agent & protocol hack ("qthttp://...") to allow direct viewing of Apple film trailers. * Fixed int-to-float conversion in the JACK output plugin. + * Work around MOD files with reported length == 0. xine-lib (1.1.16.3) 2009-04-03 * Security fixes: diff --git a/src/demuxers/demux_mod.c b/src/demuxers/demux_mod.c index 34b8ebabe..de3e29ca8 100644 --- a/src/demuxers/demux_mod.c +++ b/src/demuxers/demux_mod.c @@ -178,6 +178,8 @@ static int open_mod_file(demux_mod_t *this) { this->copyright = strdup(""); this->mod_length = ModPlug_GetLength(this->mpfile); + if (this->mod_length < 1) + this->mod_length = 1; /* avoids -ve & div-by-0 */ return 1; } -- cgit v1.2.3