From a1df060a933522234ab3064ea904a28e58c70a36 Mon Sep 17 00:00:00 2001 From: Thom Johansen Date: Wed, 2 Nov 2005 18:05:31 +0000 Subject: [PATCH] Allow bigger delay setting in LAME header. git-svn-id: svn://svn.rockbox.org/rockbox/trunk@7730 a1c6a512-1295-4272-9138-f99709370657 --- firmware/mp3data.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/firmware/mp3data.c b/firmware/mp3data.c index 3c9264b7cc..52aee11e2a 100644 --- a/firmware/mp3data.c +++ b/firmware/mp3data.c @@ -442,7 +442,9 @@ int get_mp3file_info(int fd, struct mp3info *info) i += 21; info->enc_delay = (vbrheader[i] << 4) | (vbrheader[i + 1] >> 4); info->enc_padding = ((vbrheader[i + 1] & 0x0f) << 8) | vbrheader[i + 2]; - if (!(info->enc_delay >= 0 && info->enc_delay <= 1152 && + /* TODO: This sanity checking is rather silly, seeing as how the LAME + header contains a CRC field that can be used to verify integrity. */ + if (!(info->enc_delay >= 0 && info->enc_delay <= 2880 && info->enc_padding >= 0 && info->enc_padding <= 2*1152)) { /* Invalid data */