diff options
Diffstat (limited to 'firmware')
-rw-r--r-- | firmware/id3.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/firmware/id3.c b/firmware/id3.c index 3f2ba23684..5dfb3fbe50 100644 --- a/firmware/id3.c +++ b/firmware/id3.c | |||
@@ -1000,7 +1000,11 @@ static int getsonglength(int fd, struct mp3entry *entry) | |||
1000 | 1000 | ||
1001 | if(filetime == 0) | 1001 | if(filetime == 0) |
1002 | { | 1002 | { |
1003 | filetime = (entry->filesize - bytecount) / (info.bitrate / 8); | 1003 | /* Prevent a division by zero */ |
1004 | if (info.bitrate < 8) | ||
1005 | filetime = 0; | ||
1006 | else | ||
1007 | filetime = (entry->filesize - bytecount) / (info.bitrate / 8); | ||
1004 | /* bitrate is in kbps so this delivers milliseconds. Doing bitrate / 8 | 1008 | /* bitrate is in kbps so this delivers milliseconds. Doing bitrate / 8 |
1005 | * instead of filesize * 8 is exact, because mpeg audio bitrates are | 1009 | * instead of filesize * 8 is exact, because mpeg audio bitrates are |
1006 | * always multiples of 8, and it avoids overflows. */ | 1010 | * always multiples of 8, and it avoids overflows. */ |